[dm-devel] [PATCH] scsi_devinfo: remove synchronous ALUA for NETAPP/RDAC devices

[Xose Vazquez Perez]

On 11/09/2016 08:38 AM, George, Martin wrote:

Drop this one.

> As discussed with Hannes, this is not required for ONTAP's LUN C-Mode
> as well. So the ALUA blacklisting may be removed here as well, and
> not just for NetApp RDAC.

OK, done in v2.

Thank you.

> -----Original Message-----
> From: Hannes Reinecke [mailto:hare at suse.de] 
> Sent: Wednesday, November 9, 2016 12:20 PM
> To: Xose Vazquez Perez <xose.vazquez at gmail.com>
> Cc: George, Martin <Martin.George at netapp.com>; Stankey, Robert <Robert.Stankey at netapp.com>; Schremmer, Steven <Steve.Schremmer at netapp.com>; Stewart, Sean <Sean.Stewart at netapp.com>; Christophe Varoqui <christophe.varoqui at opensvc.com>; James E . J . Bottomley <jejb at linux.vnet.ibm.com>; Martin K . Petersen <martin.petersen at oracle.com>; SCSI ML <linux-scsi at vger.kernel.org>; device-mapper development <dm-devel at redhat.com>
> Subject: Re: [PATCH] scsi_devinfo: remove synchronous ALUA for NETAPP/RDAC devices
> 
> On 11/08/2016 08:32 PM, Xose Vazquez Perez wrote:
>> NetApp did confirm this is only required for ONTAP(LUN C-Mode).
>>
>> Cc: Martin George <Martin.George at netapp.com>
>> Cc: Robert Stankey <Robert.Stankey at netapp.com>
>> Cc: Steven Schremmer <Steven.Schremmer at netapp.com>
>> Cc: Sean Stewart <Sean.Stewart at netapp.com>
>> Cc: Hannes Reinecke <hare at suse.de>
>> Cc: Christophe Varoqui <christophe.varoqui at opensvc.com>
>> Cc: James E.J. Bottomley <jejb at linux.vnet.ibm.com>
>> Cc: Martin K. Petersen <martin.petersen at oracle.com>
>> Cc: SCSI ML <linux-scsi at vger.kernel.org>
>> Cc: device-mapper development <dm-devel at redhat.com>
>> Signed-off-by: Xose Vazquez Perez <xose.vazquez at gmail.com>
>> ---
>>  drivers/scsi/scsi_devinfo.c | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/drivers/scsi/scsi_devinfo.c b/drivers/scsi/scsi_devinfo.c 
>> index 2464569..1fb7964 100644
>> --- a/drivers/scsi/scsi_devinfo.c
>> +++ b/drivers/scsi/scsi_devinfo.c
>> @@ -221,7 +221,6 @@ static struct {
>>  	{"NEC", "PD-1 ODX654P", NULL, BLIST_FORCELUN | BLIST_SINGLELUN},
>>  	{"NEC", "iStorage", NULL, BLIST_REPORTLUN2},
>>  	{"NETAPP", "LUN C-Mode", NULL, BLIST_SYNC_ALUA},
>> -	{"NETAPP", "INF-01-00", NULL, BLIST_SYNC_ALUA},
>>  	{"NRC", "MBR-7", NULL, BLIST_FORCELUN | BLIST_SINGLELUN},
>>  	{"NRC", "MBR-7.4", NULL, BLIST_FORCELUN | BLIST_SINGLELUN},
>>  	{"PIONEER", "CD-ROM DRM-600", NULL, BLIST_FORCELUN | 
>> BLIST_SINGLELUN},
>>
> 
> Reviewed-by: Hannes Reinecke <hare at suse.com>
> 
> Cheers,
> 
> Hannes
>