[dm-devel] [PATCH 2/4] multipath-tools: replace GROUP_BY_SERIAL with MULTIBUS for IBM 3542 and 2105

[Christophe Varoqui]

Hi Xose,

Group by serial sure can make sense for hardware not capable of multibus.

Can you or Hannes (listed as maintainer for IBM) confirm the concerned IBM
arrays are multibus capable ?

Best regards,
Christophe Varoqui
OpenSVC

On Thu, Aug 18, 2016 at 7:58 PM, Xose Vazquez Perez <xose.vazquez at gmail.com>
wrote:

> It doesn't make any sense to have, by default, GROUP_BY_SERIAL in real
> hardware.
>
> Cc: Hannes Reinecke <hare at suse.de>
> Cc: Christophe Varoqui <christophe.varoqui at opensvc.com>
> Cc: device-mapper development <dm-devel at redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez at gmail.com>
> ---
>  libmultipath/hwtable.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c
> index aa3ea9b..72b5e1d 100644
> --- a/libmultipath/hwtable.c
> +++ b/libmultipath/hwtable.c
> @@ -443,7 +443,7 @@ static struct hwentry default_hw[] = {
>                 /* DS4200 / FAStT200 */
>                 .vendor        = "IBM",
>                 .product       = "^3542",
> -               .pgpolicy      = GROUP_BY_SERIAL,
> +               .pgpolicy      = MULTIBUS,
>                 .pgfailback    = FAILBACK_UNDEF,
>         },
>         {
> @@ -451,7 +451,7 @@ static struct hwentry default_hw[] = {
>                 .vendor        = "IBM",
>                 .product       = "^2105",
>                 .no_path_retry = NO_PATH_RETRY_QUEUE,
> -               .pgpolicy      = GROUP_BY_SERIAL,
> +               .pgpolicy      = MULTIBUS,
>                 .pgfailback    = FAILBACK_UNDEF,
>         },
>         {
> --
> 2.7.4
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20161003/91ecd346/attachment.htm>