diff mbox

[v2] scsi_devinfo: remove synchronous ALUA for NETAPP devices

Message ID 20161111164949.2322-1-xose.vazquez@gmail.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Xose Vazquez Perez Nov. 11, 2016, 4:49 p.m. UTC 
NetApp did confirm this is not required.

Cc: Martin George <Martin.George@netapp.com>
Cc: Robert Stankey <Robert.Stankey@netapp.com>
Cc: Steven Schremmer <Steven.Schremmer@netapp.com>
Cc: Sean Stewart <Sean.Stewart@netapp.com>
Cc: Hannes Reinecke <hare@suse.de>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: James E.J. Bottomley <jejb@linux.vnet.ibm.com>
Cc: Martin K. Petersen <martin.petersen@oracle.com>
Cc: SCSI ML <linux-scsi@vger.kernel.org>
Cc: device-mapper development <dm-devel@redhat.com>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 drivers/scsi/scsi_devinfo.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Stewart, Sean Dec. 7, 2016, 7:35 p.m. UTC | #1 
On 11/11/16, 10:49 AM, "Xose Vazquez Perez" <xose.vazquez@gmail.com> wrote:

    NetApp did confirm this is not required.
    
    Cc: Martin George <Martin.George@netapp.com>
    Cc: Robert Stankey <Robert.Stankey@netapp.com>
    Cc: Steven Schremmer <Steven.Schremmer@netapp.com>
    Cc: Sean Stewart <Sean.Stewart@netapp.com>
    Cc: Hannes Reinecke <hare@suse.de>
    Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
    Cc: James E.J. Bottomley <jejb@linux.vnet.ibm.com>
    Cc: Martin K. Petersen <martin.petersen@oracle.com>
    Cc: SCSI ML <linux-scsi@vger.kernel.org>
    Cc: device-mapper development <dm-devel@redhat.com>
    Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>

    ---
     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@netapp.com>


Thanks,
Sean
Martin K. Petersen Dec. 7, 2016, 11:14 p.m. UTC | #2 
>>>>> "Xose" == Xose Vazquez Perez <xose.vazquez@gmail.com> writes:

Xose> NetApp did confirm this is not required.

Applied to 4.10/scsi-queue.
Martin George Dec. 8, 2016, 7:51 p.m. UTC | #3 
On 12/8/2016 1:05 AM, Stewart, Sean wrote:
> On 11/11/16, 10:49 AM, "Xose Vazquez Perez" <xose.vazquez@gmail.com> wrote:
>
>      NetApp did confirm this is not required.
>      
>      Cc: Martin George <Martin.George@netapp.com>
>      Cc: Robert Stankey <Robert.Stankey@netapp.com>
>      Cc: Steven Schremmer <Steven.Schremmer@netapp.com>
>      Cc: Sean Stewart <Sean.Stewart@netapp.com>
>      Cc: Hannes Reinecke <hare@suse.de>
>      Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
>      Cc: James E.J. Bottomley <jejb@linux.vnet.ibm.com>
>      Cc: Martin K. Petersen <martin.petersen@oracle.com>
>      Cc: SCSI ML <linux-scsi@vger.kernel.org>
>      Cc: device-mapper development <dm-devel@redhat.com>
>      Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
>      ---
>       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@netapp.com>
>

Reviewed-by: Martin George <marting@netapp.com>

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

Patch

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