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

[Martin George <marting@xxxxxxxxxx>]

On 12/8/2016 1:05 AM, Stewart, Sean wrote:
> On 11/11/16, 10:49 AM, "Xose Vazquez Perez" <xose.vazquez@xxxxxxxxx> wrote:
>
>      NetApp did confirm this is not required.
>      
>      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 | 2 --
>       1 file changed, 2 deletions(-)
>      
>      diff --git a/drivers/scsi/scsi_devinfo.c b/drivers/scsi/scsi_devinfo.c
>      index 2464569..28fea83 100644
>      --- a/drivers/scsi/scsi_devinfo.c
>      +++ b/drivers/scsi/scsi_devinfo.c
>      @@ -220,8 +220,6 @@ static struct {
>       	{"NAKAMICH", "MJ-5.16S", NULL, BLIST_FORCELUN | BLIST_SINGLELUN},
>       	{"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},
>      --
>      2.10.2
>      
> Reviewed-by: Sean Stewart <sean.stewart@xxxxxxxxxx>

Reviewed-by: Martin George <marting@xxxxxxxxxx>

Thanks,
-Martin
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html