diff mbox series

multipath-tools: add NVME/Huawei-XSG1 to hwtable

Message ID 20210325230050.6732-1-xose.vazquez@gmail.com (mailing list archive)
State Not Applicable, archived
Delegated to: christophe varoqui
Headers show
Series multipath-tools: add NVME/Huawei-XSG1 to hwtable | expand

Commit Message

Xose Vazquez Perez March 25, 2021, 11 p.m. UTC
Info from: https://download.huawei.com/edownload/e/download.do?actionFlag=download&nid=EDOC1100154490&partNo=6001&mid=SUPE_DOC&_t=1612885603000

Cc: Zou Ming <zouming.zouming@huawei.com>
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>
---
 libmultipath/hwtable.c | 9 +++++++++
 1 file changed, 9 insertions(+)

Comments

Martin Wilck March 26, 2021, 4:32 p.m. UTC | #1
On Fri, 2021-03-26 at 00:00 +0100, Xose Vazquez Perez wrote:
> Info from: 
> https://download.huawei.com/edownload/e/download.do?actionFlag=download&nid=EDOC1100154490&partNo=6001&mid=SUPE_DOC&_t=1612885603000
> 
> Cc: Zou Ming <zouming.zouming@huawei.com>
> 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>
> ---
>  libmultipath/hwtable.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index 58fa7387..7923abba 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -1084,6 +1084,15 @@ static struct hwentry default_hw[] = {
>                 .pgpolicy      = GROUP_BY_PRIO,
>                 .prio_name     = PRIO_ALUA,
>         },
> +       {
> +               /* OceanStor NVMe */
> +               .vendor        = "NVME",
> +               .product       = "Huawei-XSG1",
> +               .pgpolicy      = MULTIBUS,
> +               .checker_name  = DIRECTIO,
> +               .retain_hwhandler = RETAIN_HWHANDLER_OFF,

This is a rather pointless setting, in particular if hardware_handler
is not set, and even more so for NVMe. Even for SCSI, no recent kernel
will allow changing the hwhandler.

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 58fa7387..7923abba 100644
--- a/libmultipath/hwtable.c
+++ b/libmultipath/hwtable.c
@@ -1084,6 +1084,15 @@  static struct hwentry default_hw[] = {
 		.pgpolicy      = GROUP_BY_PRIO,
 		.prio_name     = PRIO_ALUA,
 	},
+	{
+		/* OceanStor NVMe */
+		.vendor        = "NVME",
+		.product       = "Huawei-XSG1",
+		.pgpolicy      = MULTIBUS,
+		.checker_name  = DIRECTIO,
+		.retain_hwhandler = RETAIN_HWHANDLER_OFF,
+		.no_path_retry = 12,
+	},
 	/*
 	 * Kove
 	 */