diff mbox series

[resend] ALUA support for PURE FlashArray

Message ID 20191228002451.9536-1-xose.vazquez@gmail.com (mailing list archive)
State Not Applicable, archived
Delegated to: Mike Snitzer
Headers show
Series [resend] ALUA support for PURE FlashArray | expand

Commit Message

Xose Vazquez Perez Dec. 28, 2019, 12:24 a.m. UTC
From: Brian Bunker <brian@purestorage.com>

Hello all,

It has been some time since we updated our PURE FlashArray
configuration. The Linux vendors that we had been seeing in the field
were using very old versions of multipath-tools, so we haven’t needed to
change anything for some time. With the release of RHEL8, some of our
earlier values have been lost by upstream changes.

In addition we have our Active Cluster feature which leverages ALUA
since our last patch. The ALUA confguration will work for all
FlashArrays with or without Active Cluster.

We are changing 3 things.

1. ALUA support
2. Fast fail timeout from the default of 5 seconds to 10 seconds (We
   need this for our FC NPIV port migration).
3. Maximum sector size of 4MB. Some Linux vendors don’t honor the block
   limits VPD page of INQUIRY).

Thanks,
Brian

Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: DM-DEVEL ML <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
Signed-off-by: Brian Bunker <brian@purestorage.com>
---
 libmultipath/hwtable.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Comments

Xose Vazquez Perez March 2, 2020, 2:10 p.m. UTC | #1
On 12/28/19 1:24 AM, Xose Vazquez Perez wrote:

Hi Christophe,

This one is missing.

Thank you.

> From: Brian Bunker <brian@purestorage.com>
> 
> Hello all,
> 
> It has been some time since we updated our PURE FlashArray
> configuration. The Linux vendors that we had been seeing in the field
> were using very old versions of multipath-tools, so we haven’t needed to
> change anything for some time. With the release of RHEL8, some of our
> earlier values have been lost by upstream changes.
> 
> In addition we have our Active Cluster feature which leverages ALUA
> since our last patch. The ALUA confguration will work for all
> FlashArrays with or without Active Cluster.
> 
> We are changing 3 things.
> 
> 1. ALUA support
> 2. Fast fail timeout from the default of 5 seconds to 10 seconds (We
>     need this for our FC NPIV port migration).
> 3. Maximum sector size of 4MB. Some Linux vendors don’t honor the block
>     limits VPD page of INQUIRY).
> 
> Thanks,
> Brian
> 
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: DM-DEVEL ML <dm-devel@redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> Signed-off-by: Brian Bunker <brian@purestorage.com>
> ---
>   libmultipath/hwtable.c | 7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index 16627ec..9bc59f1 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -1039,7 +1039,12 @@ static struct hwentry default_hw[] = {
>   		/* FlashArray */
>   		.vendor        = "PURE",
>   		.product       = "FlashArray",
> -		.pgpolicy      = MULTIBUS,
> +		.pgpolicy      = GROUP_BY_PRIO,
> +		.pgfailback    = -FAILBACK_IMMEDIATE,
> +		.hwhandler     = "1 alua",
> +		.prio_name     = PRIO_ALUA,
> +		.fast_io_fail  = 10,
> +		.max_sectors_kb = 4096,
>   	},
>   	/*
>   	 * Huawei
> 


--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
Christophe Varoqui March 2, 2020, 2:19 p.m. UTC | #2
Hi Xose, Brian,

I don't have the original patch in my mailbox, ... only Xose's
notifications.
Brian, please resend to: me, cc: list

Best,
Christophe.

On Mon, Mar 2, 2020 at 3:10 PM Xose Vazquez Perez <xose.vazquez@gmail.com>
wrote:

> On 12/28/19 1:24 AM, Xose Vazquez Perez wrote:
>
> Hi Christophe,
>
> This one is missing.
>
> Thank you.
>
> > From: Brian Bunker <brian@purestorage.com>
> >
> > Hello all,
> >
> > It has been some time since we updated our PURE FlashArray
> > configuration. The Linux vendors that we had been seeing in the field
> > were using very old versions of multipath-tools, so we haven’t needed to
> > change anything for some time. With the release of RHEL8, some of our
> > earlier values have been lost by upstream changes.
> >
> > In addition we have our Active Cluster feature which leverages ALUA
> > since our last patch. The ALUA confguration will work for all
> > FlashArrays with or without Active Cluster.
> >
> > We are changing 3 things.
> >
> > 1. ALUA support
> > 2. Fast fail timeout from the default of 5 seconds to 10 seconds (We
> >     need this for our FC NPIV port migration).
> > 3. Maximum sector size of 4MB. Some Linux vendors don’t honor the block
> >     limits VPD page of INQUIRY).
> >
> > Thanks,
> > Brian
> >
> > Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> > Cc: DM-DEVEL ML <dm-devel@redhat.com>
> > Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> > Signed-off-by: Brian Bunker <brian@purestorage.com>
> > ---
> >   libmultipath/hwtable.c | 7 ++++++-
> >   1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> > index 16627ec..9bc59f1 100644
> > --- a/libmultipath/hwtable.c
> > +++ b/libmultipath/hwtable.c
> > @@ -1039,7 +1039,12 @@ static struct hwentry default_hw[] = {
> >               /* FlashArray */
> >               .vendor        = "PURE",
> >               .product       = "FlashArray",
> > -             .pgpolicy      = MULTIBUS,
> > +             .pgpolicy      = GROUP_BY_PRIO,
> > +             .pgfailback    = -FAILBACK_IMMEDIATE,
> > +             .hwhandler     = "1 alua",
> > +             .prio_name     = PRIO_ALUA,
> > +             .fast_io_fail  = 10,
> > +             .max_sectors_kb = 4096,
> >       },
> >       /*
> >        * Huawei
> >
>
>
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
diff mbox series

Patch

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index 16627ec..9bc59f1 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -1039,7 +1039,12 @@  static struct hwentry default_hw[] = {
 		/* FlashArray */
 		.vendor        = "PURE",
 		.product       = "FlashArray",
-		.pgpolicy      = MULTIBUS,
+		.pgpolicy      = GROUP_BY_PRIO,
+		.pgfailback    = -FAILBACK_IMMEDIATE,
+		.hwhandler     = "1 alua",
+		.prio_name     = PRIO_ALUA,
+		.fast_io_fail  = 10,
+		.max_sectors_kb = 4096,
 	},
 	/*
 	 * Huawei