[dm-devel] [PATCH v3] multipath-tools: add more info for NetApp RDAC arrays

[Schremmer, Steven]

> From: Xose Vazquez Perez <xose.vazquez at gmail.com>
> Sent: Friday, September 9, 2022 9:14 AM
> Subject: [PATCH v3] multipath-tools: add more info for NetApp RDAC arrays
> 
> To use ALUA mode:
> "Select operating system:" should be changed to "Linux DM-MP (Kernel 3.10 or
> later)".
> 
> To use RDAC mode:
> "Select operating system:" should be changed to "Linux DM-MP (Kernel 3.9 or
> earlier)".
> 
> Info from:
>  https://library.netapp.com/ecmdocs/ECMLP2439710/html/GUID-49E94674-
> 947F-4921-A1D1-CE9B42DCBA39.html
>  https://library.netapp.com/ecmdocs/ECMLP2439710/html/GUID-CA17AE34-
> F5B2-4FF4-8FD4-4A7AA4238976.html
>  https://library.netapp.com/ecmdocs/ECMLP2439710/html/GUID-3DA657A9-
> FF4A-4237-90EC-BA8863B7431E.html
> 
> Add it to prio and hardware_handler.
> 
> Cc: NetApp RDAC team <ng-eseries-upstream-maintainers at netapp.com>
> Cc: Martin Wilck <mwilck at suse.com>
> Cc: Benjamin Marzinski <bmarzins at redhat.com>
> Cc: Christophe Varoqui <christophe.varoqui at opensvc.com>
> Cc: DM-DEVEL ML <dm-devel at redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez at gmail.com>
> ---
> v2: delete "Automatic Load Balancing option" obsolete info
> v3: a part of the body wa trimmed by git
> ---
>  README.md                  | 2 +-
>  multipath/multipath.conf.5 | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/README.md b/README.md
> index b05b1332..45234dc6 100644
> --- a/README.md
> +++ b/README.md
> @@ -165,7 +165,7 @@ To enable ALUA, the following options should be
> changed:
>     "LUN Affinity" and "ALUA" should be changed to "Enable", "Redundancy Type"
>     must be "Active-Active".
> 
> -- LSI/Engenio/NetApp RDAC class, as NetApp SANtricity E/EF Series and OEM
> arrays:
> +- LSI/Engenio/NetApp RDAC class, as NetApp SANtricity E/EF Series and
> rebranded arrays:
>     "Select operating system:" should be changed to "Linux DM-MP (Kernel 3.10
> or later)".
> 
>  - NetApp ONTAP:
> diff --git a/multipath/multipath.conf.5 b/multipath/multipath.conf.5
> index acdd1ae6..6018fa54 100644
> --- a/multipath/multipath.conf.5
> +++ b/multipath/multipath.conf.5
> @@ -320,7 +320,7 @@ Generate the path priority for NetApp ONTAP class, and
> rebranded arrays.
>  .I rdac
>  (Hardware-dependent)
>  Generate the path priority for LSI/Engenio/NetApp RDAC class as NetApp
> SANtricity
> -E/EF Series, and rebranded arrays.
> +E/EF Series and rebranded arrays, with "Linux DM-MP (Kernel 3.9 or earlier)"
> option.
>  .TP
>  .I hp_sw
>  (Hardware-dependent)
> @@ -1568,7 +1568,7 @@ with Failover Mode 1 (Passive Not Ready(PNR)).
>  .I 1 rdac
>  (Hardware-dependent)
>  Hardware handler for LSI/Engenio/NetApp RDAC class as NetApp SANtricity
> E/EF
> -Series, and rebranded arrays.
> +Series and rebranded arrays, with "Linux DM-MP (Kernel 3.9 or earlier)" option.
>  .TP
>  .I 1 hp_sw
>  (Hardware-dependent)
> --
> 2.37.3

These changes look okay. Just keep in mind that multipath on the Linux host
operates in the proper mode (alua v rdac) based on the E/EF Series behaviors,
without putting any special settings in multipath.conf.

Reviewed-by: Steve Schremmer <steve.schremmer at netapp.com>

Thanks,
Steve