On 06/20/2016 08:58 AM, Christophe Varoqui wrote:
> I tend to agree with Xose this is unfair to set user_friendly_name in
> the hwtable entries.
> Does anybody have a vote against my merging this patch ?
> 
> On Fri, Jun 17, 2016 at 4:52 PM, Xose Vazquez Perez
> <[email protected] <mailto:[email protected]>> wrote:
> 
>     This is up to sysadmins, and it is already off by default.
> 
>     Cc: Christophe Varoqui <[email protected]
>     <mailto:[email protected]>>
>     Cc: device-mapper development <[email protected]
>     <mailto:[email protected]>>
>     Signed-off-by: Xose Vazquez Perez <[email protected]
>     <mailto:[email protected]>>
>     ---
>      libmultipath/hwtable.c | 2 --
>      1 file changed, 2 deletions(-)
> 
>     diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
>     index 598b9b8..bef8d31 100644
>     --- a/libmultipath/hwtable.c
>     +++ b/libmultipath/hwtable.c
>     @@ -794,7 +794,6 @@ static struct hwentry default_hw[] = {
>                     .prio_name     = PRIO_ONTAP,
>                     .prio_args     = NULL,
>                     .retain_hwhandler = RETAIN_HWHANDLER_ON,
>     -               .user_friendly_names = USER_FRIENDLY_NAMES_OFF,
>                     .detect_prio   = DETECT_PRIO_ON,
>             },
>             /*
>     @@ -1133,7 +1132,6 @@ static struct hwentry default_hw[] = {
>                     .pgfailback    = -FAILBACK_IMMEDIATE,
>                     .checker_name  = TUR,
>                     .fast_io_fail  = 10,
>     -               .user_friendly_names = USER_FRIENDLY_NAMES_OFF,
>                     .prio_name     = DEFAULT_PRIO,
>                     .no_path_retry = 0,

No, indeed, that's fine.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke                Teamlead Storage & Networking
[email protected]                                   +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)

--
dm-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to