On 07/10/2016 10:25 PM, Martin George wrote: > Remove the NetApp IBM Nseries code due to the following reasons: > > 1) The NetApp IBM Nseries product is obsolete now.
This really doesn't matter. > 2) Even earlier, the NetApp IBM Nseries product line never used a > VID/PID of "IBM" & "Nseries" respectively. Instead, it always > used the same VID/PID of "NETAPP" & "LUN" itself, similar to the > ONTAP product line. So this piece of code was redundant from the > beginning itself. OK, thank you for the notice. > Signed-off-by: Martin George <[email protected]> > --- > libmultipath/hwtable.c | 14 -------------- > multipath/multipath.conf.5 | 2 +- > 2 files changed, 1 insertion(+), 15 deletions(-) > > diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c > index a89ea87..140e802 100644 > --- a/libmultipath/hwtable.c > +++ b/libmultipath/hwtable.c > @@ -770,20 +770,6 @@ static struct hwentry default_hw[] = { > .retain_hwhandler = RETAIN_HWHANDLER_ON, > .detect_prio = DETECT_PRIO_ON, > }, > - { > - /* IBM NSeries */ > - .vendor = "IBM", > - .product = "Nseries.*", > - .features = "1 queue_if_no_path", > - .hwhandler = DEFAULT_HWHANDLER, > - .pgpolicy = GROUP_BY_PRIO, > - .pgfailback = -FAILBACK_IMMEDIATE, > - .rr_weight = RR_WEIGHT_NONE, > - .no_path_retry = NO_PATH_RETRY_UNDEF, > - .minio = 128, > - .prio_name = PRIO_ONTAP, > - .prio_args = NULL, > - }, > /* > * NEXENTA > * > diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5 > index 709ab3c..0b4d366 100644 > --- a/multipath/multipath.conf.5 > +++ b/multipath/multipath.conf.5 > @@ -224,7 +224,7 @@ accepts the optional prio_arg \fIexclusive_pref_bit\fR. > .TP > .I ontap > (Hardware-dependent) > -Generate the path priority for NetApp ONTAP class and OEM arrays as IBM > NSeries. > +Generate the path priority for NetApp ONTAP class arrays. > .TP > .I rdac > (Hardware-dependent) But I would like to preserve that text in the man page. -- dm-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/dm-devel
