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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks.
Patch pushed.

On Mon, Oct 3, 2016 at 1:54 PM, Hannes Reinecke <hare@xxxxxxx> wrote:
On 10/03/2016 01:02 PM, Christophe Varoqui wrote:
> 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@xxxxxxxxx <mailto:xose.vazquez@xxxxxxxxx>> wrote:
>
>     It doesn't make any sense to have, by default, GROUP_BY_SERIAL in
>     real hardware.
>
>     Cc: Hannes Reinecke <hare@xxxxxxx <mailto:hare@xxxxxxx>>
>     Cc: Christophe Varoqui <christophe.varoqui@opensvc.com
>     <mailto:christophe.varoqui@opensvc.com>>
>     Cc: device-mapper development <dm-devel@xxxxxxxxxx
>     <mailto:dm-devel@xxxxxxxxxx>>
>     Signed-off-by: Xose Vazquez Perez <xose.vazquez@xxxxxxxxx
>     <mailto:xose.vazquez@xxxxxxxxx>>
>     ---
>      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,
>             },
This is the IBM FAStT 200, which is IIRC an RDAC machine. So multibus
will be wrong here, and it should be using the RDAC entry instead.

>             {
>     @@ -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,
>             },
>             {
That's the old IBM ESS 800; this is a fully symmetric active-active
array. So yes, multibus is correct here.

Cheers,

Hannes
--
Dr. Hannes Reinecke                   zSeries & Storage
hare@xxxxxxx                          +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux