Re: [PATCH] scsi_devinfo: remove synchronous ALUA for NETAPP/RDAC devices

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

 



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@xxxxxxx] 
> Sent: Wednesday, November 9, 2016 12:20 PM
> To: Xose Vazquez Perez <xose.vazquez@xxxxxxxxx>
> Cc: George, Martin <Martin.George@xxxxxxxxxx>; Stankey, Robert <Robert.Stankey@xxxxxxxxxx>; Schremmer, Steven <Steve.Schremmer@xxxxxxxxxx>; Stewart, Sean <Sean.Stewart@xxxxxxxxxx>; Christophe Varoqui <christophe.varoqui@xxxxxxxxxxx>; James E . J . Bottomley <jejb@xxxxxxxxxxxxxxxxxx>; Martin K . Petersen <martin.petersen@xxxxxxxxxx>; SCSI ML <linux-scsi@xxxxxxxxxxxxxxx>; device-mapper development <dm-devel@xxxxxxxxxx>
> 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@xxxxxxxxxx>
>> Cc: Robert Stankey <Robert.Stankey@xxxxxxxxxx>
>> Cc: Steven Schremmer <Steven.Schremmer@xxxxxxxxxx>
>> Cc: Sean Stewart <Sean.Stewart@xxxxxxxxxx>
>> Cc: Hannes Reinecke <hare@xxxxxxx>
>> Cc: Christophe Varoqui <christophe.varoqui@xxxxxxxxxxx>
>> Cc: James E.J. Bottomley <jejb@xxxxxxxxxxxxxxxxxx>
>> Cc: Martin K. Petersen <martin.petersen@xxxxxxxxxx>
>> Cc: SCSI ML <linux-scsi@xxxxxxxxxxxxxxx>
>> Cc: device-mapper development <dm-devel@xxxxxxxxxx>
>> Signed-off-by: Xose Vazquez Perez <xose.vazquez@xxxxxxxxx>
>> ---
>>  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@xxxxxxxx>
> 
> Cheers,
> 
> Hannes
> 

--
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