diff mbox series

multipath-tools: make EMC/Invista config work with alua and multibus

Message ID 20210928173121.18081-1-xose.vazquez@gmail.com (mailing list archive)
State Not Applicable, archived
Delegated to: christophe varoqui
Headers show
Series multipath-tools: make EMC/Invista config work with alua and multibus | expand

Commit Message

Xose Vazquez Perez Sept. 28, 2021, 5:31 p.m. UTC
Optimal Path Management (OPM) was introduced with VPLEX 5.5 to improve VPLEX
performance. OPM uses the ALUA mechanism to spread the I/O load across VPLEX directors
while gaining cache locality, pag #187:
https://www.delltechnologies.com/en-us/collaterals/unauth/technical-guides-support-information/products/storage-2/docu5128.pdf

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
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>
---
 libmultipath/hwtable.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Martin Wilck Sept. 28, 2021, 7:25 p.m. UTC | #1
On Tue, 2021-09-28 at 19:31 +0200, Xose Vazquez Perez wrote:
> Optimal Path Management (OPM) was introduced with VPLEX 5.5 to improve
> VPLEX
> performance. OPM uses the ALUA mechanism to spread the I/O load across
> VPLEX directors
> while gaining cache locality, pag #187:
> https://www.delltechnologies.com/en-us/collaterals/unauth/technical-guides-support-information/products/storage-2/docu5128.pdf
> 
> Cc: Martin Wilck <mwilck@suse.com>
> Cc: Benjamin Marzinski <bmarzins@redhat.com>
> 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>

Reviewed-by: Martin Wilck <mwilck@suse.com>

> ---
>  libmultipath/hwtable.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index 7095aaf1..4e8b52ff 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -350,8 +350,9 @@ static struct hwentry default_hw[] = {
>                 .vendor        = "EMC",
>                 .product       = "Invista",
>                 .bl_product    = "LUNZ",
> -               .pgpolicy      = MULTIBUS,
> +               .pgpolicy      = GROUP_BY_PRIO,
>                 .no_path_retry = 5,
> +               .pgfailback    = -FAILBACK_IMMEDIATE,
>         },
>         {
>                 /* XtremIO */



--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
berthiaume, wayne Sept. 30, 2021, 2:32 p.m. UTC | #2
Hi Xose.

	OPM is no longer supported in the Dell VPLEX product. If we at Dell had wished to change the default device stanzas for any of our products they would have been done when the product and/or feature is released. Please remove this patch as well. It is not needed.

Regards,
Wayne.

Wayne Berthiaume
Manager, E-Lab Engineering Platform Qualification Team 
Dell EMC | Server & Infrastructure Systems | Platform and System Engineering | Infrastructure Engineering & Operations | e2e Validation Lab (E-LAB) 
176 South Street
Hopkinton, MA 01748
+1 (512) 647-6157 (office)
wayne.berthiaume@dell.com 
"FOLLOW" E-Lab Engineering at https://inside.dell.com/community/active/po/platform-system-engineering/elab 
Confidentiality Notice: This email message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential or proprietary information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, immediately contact the sender by reply e-mail and destroy all copies of the original message.


Internal Use - Confidential

-----Original Message-----
From: dm-devel-bounces@redhat.com <dm-devel-bounces@redhat.com> On Behalf Of Xose Vazquez Perez
Sent: Tuesday, September 28, 2021 1:31 PM
Cc: Xose Vazquez Perez; Martin Wilck; DM-DEVEL ML
Subject: [dm-devel] [PATCH] multipath-tools: make EMC/Invista config work with alua and multibus


[EXTERNAL EMAIL] 

Optimal Path Management (OPM) was introduced with VPLEX 5.5 to improve VPLEX performance. OPM uses the ALUA mechanism to spread the I/O load across VPLEX directors while gaining cache locality, pag #187:
https://www.delltechnologies.com/en-us/collaterals/unauth/technical-guides-support-information/products/storage-2/docu5128.pdf

Cc: Martin Wilck <mwilck@suse.com>
Cc: Benjamin Marzinski <bmarzins@redhat.com>
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>
---
 libmultipath/hwtable.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c index 7095aaf1..4e8b52ff 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -350,8 +350,9 @@ static struct hwentry default_hw[] = {
 		.vendor        = "EMC",
 		.product       = "Invista",
 		.bl_product    = "LUNZ",
-		.pgpolicy      = MULTIBUS,
+		.pgpolicy      = GROUP_BY_PRIO,
 		.no_path_retry = 5,
+		.pgfailback    = -FAILBACK_IMMEDIATE,
 	},
 	{
 		/* XtremIO */
--
2.32.0

--
dm-devel mailing list
dm-devel@redhat.com
https://urldefense.com/v3/__https://listman.redhat.com/mailman/listinfo/dm-devel__;!!LpKI!3xx3cBf4_gTYjsFEdPLEd12Uz-4rxuFhn91YEI31Wyhqt-tu-rdX0PIivxsto8wZ0L8$ [listman[.]redhat[.]com]


--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
Martin Wilck Oct. 20, 2021, 6:44 p.m. UTC | #3
Wayne,

On Thu, 2021-09-30 at 14:32 +0000, berthiaume, wayne wrote:
> Hi Xose.
> 
>         OPM is no longer supported in the Dell VPLEX product. If we
> at Dell had wished to change the default device stanzas for any of
> our products they would have been done when the product and/or
> feature is released. Please remove this patch as well. It is not
> needed.

thanks for the feedback, and thanks for paying attention. 
The patch will be removed. The one for SYMMETRIX, likewise.


Regards
Martin


--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
berthiaume, wayne Oct. 22, 2021, 8:59 p.m. UTC | #4
Thanks Martin.

	I'm still researching this vast company for the product teams responsible Compellent and PowerVault to be sure they are okay with the changes. We should hold off on those as well until I can get confirmation.

Regards,
Wayne.

Wayne Berthiaume
Manager, E-Lab Engineering Platform Qualification Team 
Dell EMC | Server & Infrastructure Systems | Platform and System Engineering | Infrastructure Engineering & Operations | e2e Validation Lab (E-LAB) 
176 South Street
Hopkinton, MA 01748
+1 (512) 647-6157 (office)
wayne.berthiaume@dell.com 
"FOLLOW" E-Lab Engineering at https://inside.dell.com/community/active/po/platform-system-engineering/elab 
Confidentiality Notice: This email message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential or proprietary information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, immediately contact the sender by reply e-mail and destroy all copies of the original message.


Internal Use - Confidential

-----Original Message-----
From: Martin Wilck <mwilck@suse.com> 
Sent: Wednesday, October 20, 2021 2:45 PM
To: berthiaume, wayne; Xose Vazquez Perez
Cc: DM-DEVEL ML
Subject: Re: [dm-devel] [PATCH] multipath-tools: make EMC/Invista config work with alua and multibus


[EXTERNAL EMAIL] 

Wayne,

On Thu, 2021-09-30 at 14:32 +0000, berthiaume, wayne wrote:
> Hi Xose.
> 
>         OPM is no longer supported in the Dell VPLEX product. If we at 
> Dell had wished to change the default device stanzas for any of our 
> products they would have been done when the product and/or feature is 
> released. Please remove this patch as well. It is not needed.

thanks for the feedback, and thanks for paying attention. The patch will be removed. The one for SYMMETRIX, likewise.


Regards
Martin



--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
diff mbox series

Patch

diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
index 7095aaf1..4e8b52ff 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -350,8 +350,9 @@  static struct hwentry default_hw[] = {
 		.vendor        = "EMC",
 		.product       = "Invista",
 		.bl_product    = "LUNZ",
-		.pgpolicy      = MULTIBUS,
+		.pgpolicy      = GROUP_BY_PRIO,
 		.no_path_retry = 5,
+		.pgfailback    = -FAILBACK_IMMEDIATE,
 	},
 	{
 		/* XtremIO */