<div dir="ltr">I'm not comfortable with this patch that induces a less predictable behaviour.<div>We have that retain hardware handler switch that users can activate to achieve the purpose.</div><div><br></div><div>Hannes, Ben, would you support that patch ?</div><div><br></div><div>Best Regards,</div><div>Christophe</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 6, 2016 at 8:50 PM, Xose Vazquez Perez <span dir="ltr"><<a href="mailto:xose.vazquez@gmail.com" target="_blank">xose.vazquez@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Autodetected.<br>
<br>
Cc: Hannes Reinecke <<a href="mailto:hare@suse.de">hare@suse.de</a>><br>
Cc: Christophe Varoqui <<a href="mailto:christophe.varoqui@opensvc.com">christophe.varoqui@opensvc.<wbr>com</a>><br>
Cc: device-mapper development <<a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a>><br>
Signed-off-by: Xose Vazquez Perez <<a href="mailto:xose.vazquez@gmail.com">xose.vazquez@gmail.com</a>><br>
---<br>
 libmultipath/hwtable.c | 7 -------<br>
 1 file changed, 7 deletions(-)<br>
<br>
diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c<br>
index 367aa0a..1fc2f24 100644<br>
--- a/libmultipath/hwtable.c<br>
+++ b/libmultipath/hwtable.c<br>
@@ -46,7 +46,6 @@ static struct hwentry default_hw[] = {<br>
                .product       = "VV",<br>
                .pgpolicy      = GROUP_BY_PRIO,<br>
                .pgfailback    = -FAILBACK_IMMEDIATE,<br>
-               .hwhandler     = "1 alua",<br>
                .prio_name     = PRIO_ALUA,<br>
                .no_path_retry = 18,<br>
        },<br>
@@ -124,7 +123,6 @@ static struct hwentry default_hw[] = {<br>
                /* SAN Virtualization Services Platform */<br>
                .vendor        = "HP",<br>
                .product       = "HSVX700",<br>
-               .hwhandler     = "1 alua",<br>
                .pgpolicy      = GROUP_BY_PRIO,<br>
                .pgfailback    = -FAILBACK_IMMEDIATE,<br>
                .no_path_retry = 12,<br>
@@ -504,7 +502,6 @@ static struct hwentry default_hw[] = {<br>
                .vendor        = "IBM",<br>
                .product       = "^IPR",<br>
                .no_path_retry = NO_PATH_RETRY_QUEUE,<br>
-               .hwhandler     = "1 alua",<br>
                .pgpolicy      = GROUP_BY_PRIO,<br>
                .pgfailback    = -FAILBACK_IMMEDIATE,<br>
                .prio_name     = PRIO_ALUA,<br>
@@ -546,7 +543,6 @@ static struct hwentry default_hw[] = {<br>
        {<br>
                .vendor        = "AIX",<br>
                .product       = "NVDISK",<br>
-               .hwhandler     = "1 alua",<br>
                .pgpolicy      = GROUP_BY_PRIO,<br>
                .pgfailback    = -FAILBACK_IMMEDIATE,<br>
                .no_path_retry = (300 / DEFAULT_CHECKINT),<br>
@@ -654,7 +650,6 @@ static struct hwentry default_hw[] = {<br>
                /* M-Series */<br>
                .vendor        = "NEC",<br>
                .product       = "DISK ARRAY",<br>
-               .hwhandler     = "1 alua",<br>
                .pgpolicy      = GROUP_BY_PRIO,<br>
                .pgfailback    = -FAILBACK_IMMEDIATE,<br>
                .prio_name     = PRIO_ALUA,<br>
@@ -780,7 +775,6 @@ static struct hwentry default_hw[] = {<br>
        {<br>
                .vendor        = "(Intel|INTEL)",<br>
                .product       = "Multi-Flex",<br>
-               .hwhandler     = "1 alua",<br>
                .pgpolicy      = GROUP_BY_PRIO,<br>
                .pgfailback    = -FAILBACK_IMMEDIATE,<br>
                .no_path_retry = NO_PATH_RETRY_QUEUE,<br>
@@ -792,7 +786,6 @@ static struct hwentry default_hw[] = {<br>
        {<br>
                .vendor        = "(LIO-ORG|SUSE)",<br>
                .product       = "RBD",<br>
-               .hwhandler     = "1 alua",<br>
                .pgpolicy      = GROUP_BY_PRIO,<br>
                .pgfailback    = -FAILBACK_IMMEDIATE,<br>
                .no_path_retry = 12,<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.10.1<br>
<br>
</font></span></blockquote></div><br></div>