RE: [Patch] scsi_dh_rdac.c: add Dell product information into rdacdevice handler

[Chandra Seetharaman <sekharan@xxxxxxxxxx>]

Acked-by: Chandra Seetharaman <sekharan@xxxxxxxxxx>

On Thu, 2008-10-02 at 12:18 -0500, Yanqing_Liu@xxxxxxxx wrote:
> Thanks James!
> 
> Attached is the patch.
> 
> Yanqing
> ---
>  
> 
> -----Original Message-----
> From: James Bottomley [mailto:James.Bottomley@xxxxxxxxxxxxxxxxxxxxx] 
> Sent: Thursday, October 02, 2008 10:31 AM
> To: Liu, Yanqing
> Cc: dm-devel@xxxxxxxxxx; linux-scsi@xxxxxxxxxxxxxxx;
> sekharan@xxxxxxxxxx; Iyer, Shyam
> Subject: Re: [Patch] scsi_dh_rdac.c: add Dell product information into
> rdacdevice handler
> 
> On Thu, 2008-10-02 at 10:11 -0500, Yanqing_Liu@xxxxxxxx wrote:
> > Hi all,
> > 
> > Re-send this patch in the proper format. Thanks for James's email 
> > about this.
> > ---
> > 
> > The following patch is to add Dell Powervault storage arrays into 
> > device list of rdac device handler.
> > 
> > 
> > Signed-off-by: Yanqing Liu <yanqing_liu@xxxxxxxx>
> 
> This is all much better, thanks!
> 
> > --- linux-2.6.27-rc7/drivers/scsi/device_handler/scsi_dh_rdac.c.orig
> > 2008-09-29 02:31:26.000000000 -0400
> > +++ linux-2.6.27-rc7/drivers/scsi/device_handler/scsi_dh_rdac.c
> > 2008-09-30 02:06:26.000000000 -0400
> > @@ -590,6 +590,8 @@
> > {"STK", "OPENstorage D280"},
> > {"SUN", "CSM200_R"},
> > {"SUN", "LCSM100_F"},
> > + {"DELL", "MD3000"},
> > + {"DELL", "MD3000i"},
> > {NULL, NULL},
> > };
> 
> This, unfortunately, is still  unapplyable ... looks like exchange has
> eaten the initial TABs.  No-one seems to have managed to get outlook to
> behave properly as a patch submission tool, so if you can't use an
> alternative email tool (see Documentation/email-clients.txt) then just
> attach the patch (until someone works out how to get outlook to behave,
> it's the one case where only mime email works).
> 
> 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