[prev in list] [next in list] [prev in thread] [next in thread]
List: dm-devel
Subject: Re: [dm-devel] [PATCH] multipath-tools: replace multipath configuration output
From: Christophe Varoqui <christophe.varoqui () opensvc ! com>
Date: 2016-11-26 9:21:09
Message-ID: CABr-Gne32p_=xy7rrZ9ZgvPnEbkocYF=mded-8HEk37pofgTug () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
I also like the more structured wording you propose, and I can see how
greping "setting:" can help.
I'll wait a little longer to receive ack (not receive nack?) from
distributors, as they face the annoyed users whose scripts break when we
change our log strings format.
Thanks.
On Fri, Nov 25, 2016 at 10:00 AM, Martin Wilck <mwilck@suse.com> wrote:
> I am not sure I see the merit of these changes. If it's really
> necessary to change the wording of the log messages which people have
> got used to, the new ones should be really more self-explanatory than
> the old ones.
>
> The common "setting: " formatting is nice but IMHO this alone doesn't
> really justify overthrowing old habits.
>
> On Thu, 2016-11-24 at 23:44 +0100, Xose Vazquez Perez wrote:
> > sysfs setting -> setting: kernel sysfs
> >
> > detected setting -> setting: array autodetected
> >
> > controller setting -> setting: array configuration
>
> Is "array" really more understandable to users than "controller"?
> These settings come from hwentries, so in reality they're either part
> of the built-in hwtable or of the multipath.conf "devices" section, or
> am I wrong here?
>
> > internal default -> setting: multipath internal
> >
> > overrides setting -> setting: multipath.conf overrides section
> >
> > LUN setting -> setting: multipath.conf multipaths section
> >
> > config file default
> > config file setting -> setting: multipath.conf defaults/devices
> > section
>
> Have you double-checked if this is correct for all options? It would be
> helpful for users to be able to distinguish which config file section
> the option was taken from (defaults/devices/multipaths/overrides). The
> same argument applies to the hardware enries - being able to
> distinguish built in hardware defaults from config file "devices"
> settings would be really helpful, but this patch doesn't provide that
> functionality.
>
> Regards
> Martin
>
>
>
>
> >
> >
> > Cc: Benjamin Marzinski <bmarzins@redhat.com>
> > Cc: Hannes Reinecke <hare@suse.de>
> > Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> > Cc: device-mapper development <dm-devel@redhat.com>
> > Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> > ---
> > libmultipath/propsel.c | 66 +++++++++++++++++++++++++---------------
> > ----------
> > 1 file changed, 33 insertions(+), 33 deletions(-)
> >
> > diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c
> > index d1db7c3..c0bc616 100644
> > --- a/libmultipath/propsel.c
> > +++ b/libmultipath/propsel.c
> > @@ -41,28 +41,28 @@ do {
> > \
> > #define do_default(dest, value)
> > \
> > do {
> > \
> > dest = value;
> > \
> > - origin = "(internal default)";
> > \
> > + origin = "(setting: multipath internal)";
> > \
> > } while(0)
> >
> > #define mp_set_mpe(var)
> > \
> > -do_set(var, mp->mpe, mp->var, "(LUN setting)")
> > +do_set(var, mp->mpe, mp->var, "(setting: multipath.conf multipaths
> > section)")
> > #define mp_set_hwe(var)
> > \
> > -do_set(var, mp->hwe, mp->var, "(controller setting)")
> > +do_set(var, mp->hwe, mp->var, "(setting: array configuration)")
> > #define mp_set_ovr(var)
> > \
> > -do_set(var, conf->overrides, mp->var, "(overrides setting)")
> > +do_set(var, conf->overrides, mp->var, "(setting: multipath.conf
> > overrides section)")
> > #define mp_set_conf(var)
> > \
> > -do_set(var, conf, mp->var, "(config file default)")
> > +do_set(var, conf, mp->var, "(setting: multipath.conf
> > defaults/devices section)")
> > #define mp_set_default(var, value)
> > \
> > do_default(mp->var, value)
> >
> > #define pp_set_mpe(var)
> > \
> > -do_set(var, mpe, pp->var, "(LUN setting)")
> > +do_set(var, mpe, pp->var, "(setting: multipath.conf multipaths
> > section)")
> > #define pp_set_hwe(var)
> > \
> > -do_set(var, pp->hwe, pp->var, "(controller setting)")
> > +do_set(var, pp->hwe, pp->var, "(setting: array configuration)")
> > #define pp_set_conf(var)
> > \
> > -do_set(var, conf, pp->var, "(config file default)")
> > +do_set(var, conf, pp->var, "(setting: multipath.conf
> > defaults/devices section)")
> > #define pp_set_ovr(var)
> > \
> > -do_set(var, conf->overrides, pp->var, "(overrides setting)")
> > +do_set(var, conf->overrides, pp->var, "(setting: multipath.conf
> > overrides section)")
> > #define pp_set_default(var, value)
> > \
> > do_default(pp->var, value)
> >
> > @@ -77,9 +77,9 @@ do {
> > \
> > } while(0)
> >
> > #define set_attr_mpe(var, shift)
> > \
> > -do_attr_set(var, mp->mpe, shift, "(LUN setting)")
> > +do_attr_set(var, mp->mpe, shift, "(setting: multipath.conf
> > multipaths section)")
> > #define set_attr_conf(var, shift)
> > \
> > -do_attr_set(var, conf, shift, "(config file default)")
> > +do_attr_set(var, conf, shift, "(setting: multipath.conf
> > defaults/devices section)")
> >
> > int select_mode(struct config *conf, struct multipath *mp)
> > {
> > @@ -214,13 +214,13 @@ want_user_friendly_names(struct config *conf,
> > struct multipath * mp)
> > int user_friendly_names;
> >
> > do_set(user_friendly_names, mp->mpe, user_friendly_names,
> > - "(LUN setting)");
> > + "(setting: multipath.conf multipaths section)");
> > do_set(user_friendly_names, conf->overrides,
> > user_friendly_names,
> > - "(overrides setting)");
> > + "(setting: multipath.conf overrides section)");
> > do_set(user_friendly_names, mp->hwe, user_friendly_names,
> > - "(controller setting)");
> > + "(setting: array configuration)");
> > do_set(user_friendly_names, conf, user_friendly_names,
> > - "(config file setting)");
> > + "(setting: multipath.conf defaults/devices
> > section)");
> > do_default(user_friendly_names,
> > DEFAULT_USER_FRIENDLY_NAMES);
> > out:
> > condlog(3, "%s: user_friendly_names = %s %s", mp->wwid,
> > @@ -235,7 +235,7 @@ int select_alias(struct config *conf, struct
> > multipath * mp)
> >
> > if (mp->mpe && mp->mpe->alias) {
> > mp->alias = STRDUP(mp->mpe->alias);
> > - origin = "(LUN setting)";
> > + origin = "(setting: multipath.conf multipaths
> > section)";
> > goto out;
> > }
> >
> > @@ -312,24 +312,24 @@ int select_checker(struct config *conf, struct
> > path *pp)
> > char *origin, *checker_name;
> > struct checker * c = &pp->checker;
> >
> > - do_set(checker_name, conf->overrides, checker_name,
> > "(overrides setting)");
> > - do_set(checker_name, pp->hwe, checker_name, "(controller
> > setting)");
> > - do_set(checker_name, conf, checker_name, "(config file
> > setting)");
> > + do_set(checker_name, conf->overrides, checker_name,
> > "(setting: multipath.conf overrides section)");
> > + do_set(checker_name, pp->hwe, checker_name, "(setting: array
> > configuration)");
> > + do_set(checker_name, conf, checker_name, "(setting:
> > multipath.conf defaults/devices section)");
> > do_default(checker_name, DEFAULT_CHECKER);
> > out:
> > checker_get(conf->multipath_dir, c, checker_name);
> > condlog(3, "%s: path_checker = %s %s", pp->dev, c->name,
> > origin);
> > if (conf->checker_timeout) {
> > c->timeout = conf->checker_timeout;
> > - condlog(3, "%s: checker timeout = %u s (config file
> > default)",
> > + condlog(3, "%s: checker timeout = %u s (setting:
> > multipath.conf defaults/devices section)",
> > pp->dev, c->timeout);
> > }
> > else if (sysfs_get_timeout(pp, &c->timeout) > 0)
> > - condlog(3, "%s: checker timeout = %u ms (sysfs
> > setting)",
> > + condlog(3, "%s: checker timeout = %u ms (setting:
> > kernel sysfs)",
> > pp->dev, c->timeout);
> > else {
> > c->timeout = DEF_TIMEOUT;
> > - condlog(3, "%s: checker timeout = %u ms (internal
> > default)",
> > + condlog(3, "%s: checker timeout = %u ms (setting:
> > multipath internal)",
> > pp->dev, c->timeout);
> > }
> > return 0;
> > @@ -397,17 +397,17 @@ int select_prio(struct config *conf, struct
> > path *pp)
> > if (pp->detect_prio == DETECT_PRIO_ON) {
> > detect_prio(conf, pp);
> > if (prio_selected(p)) {
> > - origin = "(detected setting)";
> > + origin = "(setting: array autodetected)";
> > goto out;
> > }
> > }
> > mpe = find_mpe(conf->mptable, pp->wwid);
> > - set_prio(conf->multipath_dir, mpe, "(LUN setting)");
> > - set_prio(conf->multipath_dir, conf->overrides, "(overrides
> > setting)");
> > - set_prio(conf->multipath_dir, pp->hwe, "controller
> > setting)");
> > - set_prio(conf->multipath_dir, conf, "(config file
> > default)");
> > + set_prio(conf->multipath_dir, mpe, "(setting: multipath.conf
> > multipaths section)");
> > + set_prio(conf->multipath_dir, conf->overrides, "(setting:
> > multipath.conf overrides section)");
> > + set_prio(conf->multipath_dir, pp->hwe, "(setting: array
> > configuration)");
> > + set_prio(conf->multipath_dir, conf, "(setting:
> > multipath.conf defaults/devices section)");
> > prio_get(conf->multipath_dir, p, DEFAULT_PRIO,
> > DEFAULT_PRIO_ARGS);
> > - origin = "(internal default)";
> > + origin = "(setting: multipath internal)";
> > out:
> > /*
> > * fetch tpgs mode for alua, if its not already obtained
> > @@ -448,7 +448,7 @@ out:
> > condlog(3, "%s: no_path_retry = %s (inherited
> > setting)",
> > mp->alias, buff);
> > else
> > - condlog(3, "%s: no_path_retry = undef (internal
> > default)",
> > + condlog(3, "%s: no_path_retry = undef (setting:
> > multipath internal)",
> > mp->alias);
> > return 0;
> > }
> > @@ -458,10 +458,10 @@ select_minio_rq (struct config *conf, struct
> > multipath * mp)
> > {
> > char *origin;
> >
> > - do_set(minio_rq, mp->mpe, mp->minio, "(LUN setting)");
> > - do_set(minio_rq, conf->overrides, mp->minio, "(overrides
> > setting)");
> > - do_set(minio_rq, mp->hwe, mp->minio, "(controller
> > setting)");
> > - do_set(minio_rq, conf, mp->minio, "(config file setting)");
> > + do_set(minio_rq, mp->mpe, mp->minio, "(setting:
> > multipath.conf multipaths section)");
> > + do_set(minio_rq, conf->overrides, mp->minio, "(setting:
> > multipath.conf overrides section)");
> > + do_set(minio_rq, mp->hwe, mp->minio, "(setting: array
> > configuration)");
> > + do_set(minio_rq, conf, mp->minio, "(setting: multipath.conf
> > defaults/devices section)");
> > do_default(mp->minio, DEFAULT_MINIO_RQ);
> > out:
> > condlog(3, "%s: minio = %i %s", mp->alias, mp->minio,
> > origin);
>
> --
> Dr. Martin Wilck <mwilck@suse.com>, Tel. +49 (0)911 74053 2107
> SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton
> HRB 21284 (AG Nürnberg)
>
> --
> dm-devel mailing list
> dm-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel
[Attachment #5 (text/html)]
<div dir="ltr">I also like the more structured wording you propose, and I can see how \
greping "setting:" can help.<div><br></div><div>I'll wait a little \
longer to receive ack (not receive nack?) from distributors, as they face the annoyed \
users whose scripts break when we change our log strings \
format.</div><div><br></div><div>Thanks.</div></div><div class="gmail_extra"><br><div \
class="gmail_quote">On Fri, Nov 25, 2016 at 10:00 AM, Martin Wilck <span \
dir="ltr"><<a href="mailto:mwilck@suse.com" \
target="_blank">mwilck@suse.com</a>></span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">I am not sure I see the merit of these changes. If it's \
really<br> necessary to change the wording of the log messages which people have<br>
got used to, the new ones should be really more self-explanatory than<br>
the old ones.<br>
<br>
The common "setting: " formatting is nice but IMHO this alone \
doesn't<br> really justify overthrowing old habits.<br>
<span class=""><br>
On Thu, 2016-11-24 at 23:44 +0100, Xose Vazquez Perez wrote:<br>
> sysfs setting -> setting: kernel sysfs<br>
><br>
> detected setting -> setting: array autodetected<br>
><br>
> controller setting -> setting: array configuration<br>
<br>
</span>Is "array" really more understandable to users than \
"controller"?<br> These settings come from hwentries, so in reality \
they're either part<br> of the built-in hwtable or of the multipath.conf \
"devices" section, or<br> am I wrong here?<br>
<span class=""><br>
> internal default -> setting: multipath internal<br>
><br>
> overrides setting -> setting: multipath.conf overrides \
section<br> ><br>
> LUN setting -> setting: multipath.conf multipaths \
section<br> ><br>
> config file default<br>
> config file setting -> setting: multipath.conf defaults/devices<br>
> section<br>
<br>
</span>Have you double-checked if this is correct for all options? It would be<br>
helpful for users to be able to distinguish which config file section<br>
the option was taken from (defaults/devices/multipaths/<wbr>overrides). The<br>
same argument applies to the hardware enries - being able to<br>
distinguish built in hardware defaults from config file "devices"<br>
settings would be really helpful, but this patch doesn't provide that<br>
functionality.<br>
<br>
Regards<br>
Martin<br>
<div><div class="h5"><br>
<br>
<br>
<br>
><br>
><br>
> Cc: Benjamin Marzinski <<a \
href="mailto:bmarzins@redhat.com">bmarzins@redhat.com</a>><br> > Cc: Hannes \
Reinecke <<a href="mailto:hare@suse.de">hare@suse.de</a>><br> > Cc: \
Christophe Varoqui <<a \
href="mailto:christophe.varoqui@opensvc.com">christophe.varoqui@opensvc.<wbr>com</a>><br>
> Cc: device-mapper development <<a \
href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a>><br> > Signed-off-by: \
Xose Vazquez Perez <<a \
href="mailto:xose.vazquez@gmail.com">xose.vazquez@gmail.com</a>><br> > ---<br>
> libmultipath/propsel.c | 66 +++++++++++++++++++++++++-----<wbr>----------<br>
> ----------<br>
> 1 file changed, 33 insertions(+), 33 deletions(-)<br>
><br>
> diff --git a/libmultipath/propsel.c b/libmultipath/propsel.c<br>
> index d1db7c3..c0bc616 100644<br>
> --- a/libmultipath/propsel.c<br>
> +++ b/libmultipath/propsel.c<br>
> @@ -41,28 +41,28 @@ do {<br>
> \<br>
> #define do_default(dest, value)<br>
> \<br>
> do {<br>
> \<br>
> dest = value;<br>
> \<br>
> - origin = "(internal default)";<br>
> \<br>
> + origin = "(setting: multipath internal)";<br>
> \<br>
> } while(0)<br>
> <br>
> #define mp_set_mpe(var)<br>
> \<br>
> -do_set(var, mp->mpe, mp->var, "(LUN setting)")<br>
> +do_set(var, mp->mpe, mp->var, "(setting: multipath.conf \
multipaths<br> > section)")<br>
> #define mp_set_hwe(var)<br>
> \<br>
> -do_set(var, mp->hwe, mp->var, "(controller setting)")<br>
> +do_set(var, mp->hwe, mp->var, "(setting: array \
configuration)")<br> > #define mp_set_ovr(var)<br>
> \<br>
> -do_set(var, conf->overrides, mp->var, "(overrides \
setting)")<br> > +do_set(var, conf->overrides, mp->var, "(setting: \
multipath.conf<br> > overrides section)")<br>
> #define mp_set_conf(var)<br>
> \<br>
> -do_set(var, conf, mp->var, "(config file default)")<br>
> +do_set(var, conf, mp->var, "(setting: multipath.conf<br>
> defaults/devices section)")<br>
> #define mp_set_default(var, value)<br>
> \<br>
> do_default(mp->var, value)<br>
> <br>
> #define pp_set_mpe(var)<br>
> \<br>
> -do_set(var, mpe, pp->var, "(LUN setting)")<br>
> +do_set(var, mpe, pp->var, "(setting: multipath.conf multipaths<br>
> section)")<br>
> #define pp_set_hwe(var)<br>
> \<br>
> -do_set(var, pp->hwe, pp->var, "(controller setting)")<br>
> +do_set(var, pp->hwe, pp->var, "(setting: array \
configuration)")<br> > #define pp_set_conf(var)<br>
> \<br>
> -do_set(var, conf, pp->var, "(config file default)")<br>
> +do_set(var, conf, pp->var, "(setting: multipath.conf<br>
> defaults/devices section)")<br>
> #define pp_set_ovr(var)<br>
> \<br>
> -do_set(var, conf->overrides, pp->var, "(overrides \
setting)")<br> > +do_set(var, conf->overrides, pp->var, "(setting: \
multipath.conf<br> > overrides section)")<br>
> #define pp_set_default(var, value)<br>
> \<br>
> do_default(pp->var, value)<br>
> <br>
> @@ -77,9 +77,9 @@ do {<br>
> \<br>
> } while(0)<br>
> <br>
> #define set_attr_mpe(var, shift)<br>
> \<br>
> -do_attr_set(var, mp->mpe, shift, "(LUN setting)")<br>
> +do_attr_set(var, mp->mpe, shift, "(setting: multipath.conf<br>
> multipaths section)")<br>
> #define set_attr_conf(var, shift)<br>
> \<br>
> -do_attr_set(var, conf, shift, "(config file default)")<br>
> +do_attr_set(var, conf, shift, "(setting: multipath.conf<br>
> defaults/devices section)")<br>
> <br>
> int select_mode(struct config *conf, struct multipath *mp)<br>
> {<br>
> @@ -214,13 +214,13 @@ want_user_friendly_names(<wbr>struct config *conf,<br>
> struct multipath * mp)<br>
> int user_friendly_names;<br>
> <br>
> do_set(user_friendly_names, mp->mpe, user_friendly_names,<br>
> - "(LUN setting)");<br>
> + "(setting: multipath.conf multipaths \
section)");<br> > do_set(user_friendly_names, \
conf->overrides,<br> > user_friendly_names,<br>
> - "(overrides setting)");<br>
> + "(setting: multipath.conf overrides \
section)");<br> > do_set(user_friendly_names, mp->hwe, \
user_friendly_names,<br> > - "(controller \
setting)");<br> > + "(setting: array \
configuration)");<br> > do_set(user_friendly_names, conf, \
user_friendly_names,<br> > - "(config file \
setting)");<br> > + "(setting: multipath.conf \
defaults/devices<br> > section)");<br>
> do_default(user_friendly_<wbr>names,<br>
> DEFAULT_USER_FRIENDLY_NAMES);<br>
> out:<br>
> condlog(3, "%s: user_friendly_names = %s %s", \
mp->wwid,<br> > @@ -235,7 +235,7 @@ int select_alias(struct config *conf, \
struct<br> > multipath * mp)<br>
> <br>
> if (mp->mpe && mp->mpe->alias) {<br>
> mp->alias = STRDUP(mp->mpe->alias);<br>
> - origin = "(LUN setting)";<br>
> + origin = "(setting: multipath.conf multipaths<br>
> section)";<br>
> goto out;<br>
> }<br>
> <br>
> @@ -312,24 +312,24 @@ int select_checker(struct config *conf, struct<br>
> path *pp)<br>
> char *origin, *checker_name;<br>
> struct checker * c = &pp->checker;<br>
> <br>
> - do_set(checker_name, conf->overrides, checker_name,<br>
> "(overrides setting)");<br>
> - do_set(checker_name, pp->hwe, checker_name, "(controller<br>
> setting)");<br>
> - do_set(checker_name, conf, checker_name, "(config file<br>
> setting)");<br>
> + do_set(checker_name, conf->overrides, checker_name,<br>
> "(setting: multipath.conf overrides section)");<br>
> + do_set(checker_name, pp->hwe, checker_name, "(setting: \
array<br> > configuration)");<br>
> + do_set(checker_name, conf, checker_name, "(setting:<br>
> multipath.conf defaults/devices section)");<br>
> do_default(checker_name, DEFAULT_CHECKER);<br>
> out:<br>
> checker_get(conf->multipath_<wbr>dir, c, checker_name);<br>
> condlog(3, "%s: path_checker = %s %s", pp->dev, \
c->name,<br> > origin);<br>
> if (conf->checker_timeout) {<br>
> c->timeout = conf->checker_timeout;<br>
> - condlog(3, "%s: checker timeout = %u s (config \
file<br> > default)",<br>
> + condlog(3, "%s: checker timeout = %u s (setting:<br>
> multipath.conf defaults/devices section)",<br>
> pp->dev, c->timeout);<br>
> }<br>
> else if (sysfs_get_timeout(pp, &c->timeout) > 0)<br>
> - condlog(3, "%s: checker timeout = %u ms (sysfs<br>
> setting)",<br>
> + condlog(3, "%s: checker timeout = %u ms (setting:<br>
> kernel sysfs)",<br>
> pp->dev, c->timeout);<br>
> else {<br>
> c->timeout = DEF_TIMEOUT;<br>
> - condlog(3, "%s: checker timeout = %u ms (internal<br>
> default)",<br>
> + condlog(3, "%s: checker timeout = %u ms (setting:<br>
> multipath internal)",<br>
> pp->dev, c->timeout);<br>
> }<br>
> return 0;<br>
> @@ -397,17 +397,17 @@ int select_prio(struct config *conf, struct<br>
> path *pp)<br>
> if (pp->detect_prio == DETECT_PRIO_ON) {<br>
> detect_prio(conf, pp);<br>
> if (prio_selected(p)) {<br>
> - origin = "(detected setting)";<br>
> + origin = "(setting: array \
autodetected)";<br> > goto out;<br>
> }<br>
> }<br>
> mpe = find_mpe(conf->mptable, pp->wwid);<br>
> - set_prio(conf->multipath_dir, mpe, "(LUN setting)");<br>
> - set_prio(conf->multipath_dir, conf->overrides, \
"(overrides<br> > setting)");<br>
> - set_prio(conf->multipath_dir, pp->hwe, "controller<br>
> setting)");<br>
> - set_prio(conf->multipath_dir, conf, "(config file<br>
> default)");<br>
> + set_prio(conf->multipath_dir, mpe, "(setting: \
multipath.conf<br> > multipaths section)");<br>
> + set_prio(conf->multipath_dir, conf->overrides, \
"(setting:<br> > multipath.conf overrides section)");<br>
> + set_prio(conf->multipath_dir, pp->hwe, "(setting: array<br>
> configuration)");<br>
> + set_prio(conf->multipath_dir, conf, "(setting:<br>
> multipath.conf defaults/devices section)");<br>
> prio_get(conf->multipath_dir, p, DEFAULT_PRIO,<br>
> DEFAULT_PRIO_ARGS);<br>
> - origin = "(internal default)";<br>
> + origin = "(setting: multipath internal)";<br>
> out:<br>
> /*<br>
> * fetch tpgs mode for alua, if its not already obtained<br>
> @@ -448,7 +448,7 @@ out:<br>
> condlog(3, "%s: no_path_retry = %s (inherited<br>
> setting)",<br>
> mp->alias, buff);<br>
> else<br>
> - condlog(3, "%s: no_path_retry = undef (internal<br>
> default)",<br>
> + condlog(3, "%s: no_path_retry = undef (setting:<br>
> multipath internal)",<br>
> mp->alias);<br>
> return 0;<br>
> }<br>
> @@ -458,10 +458,10 @@ select_minio_rq (struct config *conf, struct<br>
> multipath * mp)<br>
> {<br>
> char *origin;<br>
> <br>
> - do_set(minio_rq, mp->mpe, mp->minio, "(LUN \
setting)");<br> > - do_set(minio_rq, conf->overrides, mp->minio, \
"(overrides<br> > setting)");<br>
> - do_set(minio_rq, mp->hwe, mp->minio, "(controller<br>
> setting)");<br>
> - do_set(minio_rq, conf, mp->minio, "(config file \
setting)");<br> > + do_set(minio_rq, mp->mpe, mp->minio, \
"(setting:<br> > multipath.conf multipaths section)");<br>
> + do_set(minio_rq, conf->overrides, mp->minio, "(setting:<br>
> multipath.conf overrides section)");<br>
> + do_set(minio_rq, mp->hwe, mp->minio, "(setting: array<br>
> configuration)");<br>
> + do_set(minio_rq, conf, mp->minio, "(setting: multipath.conf<br>
> defaults/devices section)");<br>
> do_default(mp->minio, DEFAULT_MINIO_RQ);<br>
> out:<br>
> condlog(3, "%s: minio = %i %s", mp->alias, \
mp->minio,<br> > origin);<br>
<br>
--<br>
</div></div>Dr. Martin Wilck <<a \
href="mailto:mwilck@suse.com">mwilck@suse.com</a>>, Tel. <a \
href="tel:%2B49%20%280%29911%2074053%202107" value="+49911740532107">+49 (0)911 74053 \
2107</a><br> SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham \
Norton<br> HRB 21284 (AG Nürnberg)<br>
<br>
--<br>
dm-devel mailing list<br>
<a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/dm-devel" rel="noreferrer" \
target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/dm-devel</a></blockquote></div><br></div>
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic