[dm-devel] [PATCH] multipath-tools: add DellEMC/PowerStore to hardware table

[Martin Wilck]

On Thu, 2021-03-04 at 00:28 +0100, Xose Vazquez Perez wrote:
> Info from: 
> https://www.delltechnologies.com/en-us/collaterals/unauth/technical-guides-support-information/products/storage-2/docu5128.pdf
> 
> 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>
> ---
>  libmultipath/hwtable.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index 921aadc5..f5d26b04 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -388,6 +388,17 @@ static struct hwentry default_hw[] = {
>                 .product       = "^EMC PowerMax_",
>                 .pgpolicy      = MULTIBUS,
>         },
> +       {
> +               /* PowerStore */
> +               .vendor        = "DellEMC",
> +               .product       = "PowerStore",
> +               .pgpolicy      = GROUP_BY_PRIO,
> +               .prio_name     = PRIO_ALUA,
> +               .hwhandler     = "1 alua",
> +               .pgfailback    = -FAILBACK_IMMEDIATE,
> +               .no_path_retry = 30,

The linked Dell document says "no_path_retry 3" on page 225. That's
admittedly quite low, but why use "30" instead? Have you been in
contact with Dell?

Martin

> +               .fast_io_fail  = 15,
> +       },
>         /*
>          * Fujitsu
>          */