<div dir="ltr">Hi Xose,<div><br></div><div>Group by serial sure can make sense for hardware not capable of multibus.<br></div><div><br></div><div>Can you or Hannes (listed as maintainer for IBM) confirm the concerned IBM arrays are multibus capable ?</div><div><br></div><div>Best regards,</div><div>Christophe Varoqui</div><div>OpenSVC</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 18, 2016 at 7:58 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">It doesn't make any sense to have, by default, GROUP_BY_SERIAL in real hardware.<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 | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c<br>
index aa3ea9b..72b5e1d 100644<br>
--- a/libmultipath/hwtable.c<br>
+++ b/libmultipath/hwtable.c<br>
@@ -443,7 +443,7 @@ static struct hwentry default_hw[] = {<br>
                /* DS4200 / FAStT200 */<br>
                .vendor        = "IBM",<br>
                .product       = "^3542",<br>
-               .pgpolicy      = GROUP_BY_SERIAL,<br>
+               .pgpolicy      = MULTIBUS,<br>
                .pgfailback    = FAILBACK_UNDEF,<br>
        },<br>
        {<br>
@@ -451,7 +451,7 @@ static struct hwentry default_hw[] = {<br>
                .vendor        = "IBM",<br>
                .product       = "^2105",<br>
                .no_path_retry = NO_PATH_RETRY_QUEUE,<br>
-               .pgpolicy      = GROUP_BY_SERIAL,<br>
+               .pgpolicy      = MULTIBUS,<br>
                .pgfailback    = FAILBACK_UNDEF,<br>
        },<br>
        {<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.7.4<br>
<br>
</font></span></blockquote></div><br></div>