public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [Patch] scsi_dh_rdac.c: add Dell product information into rdac device handler
@ 2008-10-02 15:11 Yanqing_Liu
  2008-10-02 15:31 ` James Bottomley
  0 siblings, 1 reply; 4+ messages in thread
From: Yanqing_Liu @ 2008-10-02 15:11 UTC (permalink / raw)
  To: dm-devel, linux-scsi; +Cc: James.Bottomley, Shyam_Iyer


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

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch] scsi_dh_rdac.c: add Dell product information into rdac device handler
  2008-10-02 15:11 [Patch] scsi_dh_rdac.c: add Dell product information into rdac device handler Yanqing_Liu
@ 2008-10-02 15:31 ` James Bottomley
  2008-10-02 17:18   ` [Patch] scsi_dh_rdac.c: add Dell product information into rdacdevice handler Yanqing_Liu
  0 siblings, 1 reply; 4+ messages in thread
From: James Bottomley @ 2008-10-02 15:31 UTC (permalink / raw)
  To: Yanqing_Liu; +Cc: dm-devel, linux-scsi, sekharan, Shyam_Iyer

On Thu, 2008-10-02 at 10:11 -0500, Yanqing_Liu@Dell.com 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@dell.com>

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



^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [Patch] scsi_dh_rdac.c: add Dell product information into rdacdevice handler
  2008-10-02 15:31 ` James Bottomley
@ 2008-10-02 17:18   ` Yanqing_Liu
  2008-10-03  0:24     ` Chandra Seetharaman
  0 siblings, 1 reply; 4+ messages in thread
From: Yanqing_Liu @ 2008-10-02 17:18 UTC (permalink / raw)
  To: James.Bottomley, dm-devel, linux-scsi; +Cc: sekharan, Shyam_Iyer, Matt_Domsch

[-- Attachment #1: Type: text/plain, Size: 1604 bytes --]

Thanks James!

Attached is the patch.

Yanqing
---
 

-----Original Message-----
From: James Bottomley [mailto:James.Bottomley@HansenPartnership.com] 
Sent: Thursday, October 02, 2008 10:31 AM
To: Liu, Yanqing
Cc: dm-devel@redhat.com; linux-scsi@vger.kernel.org;
sekharan@us.ibm.com; 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@Dell.com 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@dell.com>

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



[-- Attachment #2: rdacPatch --]
[-- Type: application/octet-stream, Size: 525 bytes --]

The following patch is to add Dell Powervault storage arrays into device list for rdac device handler.

Signed-off-by: Yanqing Liu <yanqing_liu@dell.com>

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [Patch] scsi_dh_rdac.c: add Dell product information into rdacdevice handler
  2008-10-02 17:18   ` [Patch] scsi_dh_rdac.c: add Dell product information into rdacdevice handler Yanqing_Liu
@ 2008-10-03  0:24     ` Chandra Seetharaman
  0 siblings, 0 replies; 4+ messages in thread
From: Chandra Seetharaman @ 2008-10-03  0:24 UTC (permalink / raw)
  To: Yanqing_Liu
  Cc: James.Bottomley, dm-devel, linux-scsi, Shyam_Iyer, Matt_Domsch

Acked-by: Chandra Seetharaman <sekharan@us.ibm.com>

On Thu, 2008-10-02 at 12:18 -0500, Yanqing_Liu@Dell.com wrote:
> Thanks James!
> 
> Attached is the patch.
> 
> Yanqing
> ---
>  
> 
> -----Original Message-----
> From: James Bottomley [mailto:James.Bottomley@HansenPartnership.com] 
> Sent: Thursday, October 02, 2008 10:31 AM
> To: Liu, Yanqing
> Cc: dm-devel@redhat.com; linux-scsi@vger.kernel.org;
> sekharan@us.ibm.com; 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@Dell.com 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@dell.com>
> 
> 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
> 
> 


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-10-03  0:25 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-02 15:11 [Patch] scsi_dh_rdac.c: add Dell product information into rdac device handler Yanqing_Liu
2008-10-02 15:31 ` James Bottomley
2008-10-02 17:18   ` [Patch] scsi_dh_rdac.c: add Dell product information into rdacdevice handler Yanqing_Liu
2008-10-03  0:24     ` Chandra Seetharaman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox