Re: [PATCH] scsi: replace wrong device handler name for CLARiiON arrays

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

 



On Sat, 2016-10-08 at 19:17 +0200, Hannes Reinecke wrote:
> On 10/07/2016 06:19 PM, Xose Vazquez Perez wrote:
> > At drivers/scsi/device_handler/scsi_dh_emc.c it was defined as:
> > #define CLARIION_NAME                   "emc"
> > 
> > Cc: Christoph Hellwig <hch@xxxxxx>
> > 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_dh.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/drivers/scsi/scsi_dh.c b/drivers/scsi/scsi_dh.c
> > index 54d446c..b8d3b97 100644
> > --- a/drivers/scsi/scsi_dh.c
> > +++ b/drivers/scsi/scsi_dh.c
> > @@ -36,9 +36,9 @@ struct scsi_dh_blist {
> >  };
> > 
> >  static const struct scsi_dh_blist scsi_dh_blist[] = {
> > -	{"DGC", "RAID",			"clariion" },
> > -	{"DGC", "DISK",			"clariion" },
> > -	{"DGC", "VRAID",		"clariion" },
> > +	{"DGC", "RAID",			"emc" },
> > +	{"DGC", "DISK",			"emc" },
> > +	{"DGC", "VRAID",		"emc" },
> > 
> >  	{"COMPAQ", "MSA1000 VOLUME",	"hp_sw" },
> >  	{"COMPAQ", "HSV110",		"hp_sw" },
> > 
> Whee. Of course.
> 
> Reviewed-by: Hannes Reinecke <hare@xxxxxxxx>

Actually, should we replace?  Aren't there older arrays still bearing
the clariion name, so we should have these entries in addition?

James


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



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux