Re: [PATCH] Perverting cciss

2007-04-08 Thread Stephen Cameron
--- Christoph Hellwig <[EMAIL PROTECTED]> wrote: > Also is there a reason you don't want to pass down the various > other ioctls scsi_cmd_ioctl can handle to it? Didn't think about it, SG_IO is the one people were clamoring for. SG_GET_VERSION_NUM: easily doable. SCSI_IOCTL_GET_IDLUN: Hmm,

Re: [PATCH] Perverting cciss

2007-04-08 Thread Christoph Hellwig
On Fri, Apr 06, 2007 at 10:07:52AM -0700, Stephen Cameron wrote: > How about something like this: > (Since my mailer is sure to wreck the patch, it can be found intact here: > http://cciss.cvs.sourceforge.net/*checkout*/cciss/patches/kernel.org-2.6/cciss_sg_io_block_pc.patch?revision=1.1 Looks

Re: [PATCH] Perverting cciss

2007-04-08 Thread Christoph Hellwig
On Fri, Apr 06, 2007 at 10:07:52AM -0700, Stephen Cameron wrote: How about something like this: (Since my mailer is sure to wreck the patch, it can be found intact here: http://cciss.cvs.sourceforge.net/*checkout*/cciss/patches/kernel.org-2.6/cciss_sg_io_block_pc.patch?revision=1.1 Looks good

Re: [PATCH] Perverting cciss

2007-04-08 Thread Stephen Cameron
--- Christoph Hellwig [EMAIL PROTECTED] wrote: Also is there a reason you don't want to pass down the various other ioctls scsi_cmd_ioctl can handle to it? Didn't think about it, SG_IO is the one people were clamoring for. SG_GET_VERSION_NUM: easily doable. SCSI_IOCTL_GET_IDLUN: Hmm, cciss

Re: [PATCH] Perverting cciss

2007-04-06 Thread Stephen Cameron
Hannes Reinecke wrote: > this patch adds the SG_IO ioctl to the cciss driver. > As the driver is capable of sending SCSI CDBs to the controller there is > no reason why we shouldn't exploit it. > This way we get to use all the nice sg_utils for the cciss driver. > And a persistent device name for

Re: [PATCH] Perverting cciss

2007-04-06 Thread Stephen Cameron
Hannes Reinecke wrote: this patch adds the SG_IO ioctl to the cciss driver. As the driver is capable of sending SCSI CDBs to the controller there is no reason why we shouldn't exploit it. This way we get to use all the nice sg_utils for the cciss driver. And a persistent device name for

RE: [PATCH] Perverting cciss

2007-04-05 Thread Miller, Mike (OS Dev)
James wrote: > > On Thu, 2007-04-05 at 11:03 +0100, Christoph Hellwig wrote: > > On Thu, Apr 05, 2007 at 11:58:06AM +0200, Hannes Reinecke wrote: > > > Hi All, > > > > > > this patch adds the SG_IO ioctl to the cciss driver. > > > As the driver is capable of sending SCSI CDBs to the controller

Re: [PATCH] Perverting cciss

2007-04-05 Thread James Bottomley
On Thu, 2007-04-05 at 11:03 +0100, Christoph Hellwig wrote: > On Thu, Apr 05, 2007 at 11:58:06AM +0200, Hannes Reinecke wrote: > > Hi All, > > > > this patch adds the SG_IO ioctl to the cciss driver. > > As the driver is capable of sending SCSI CDBs to the controller there is > > no reason why we

Re: [PATCH] Perverting cciss

2007-04-05 Thread Christoph Hellwig
On Thu, Apr 05, 2007 at 11:58:06AM +0200, Hannes Reinecke wrote: > Hi All, > > this patch adds the SG_IO ioctl to the cciss driver. > As the driver is capable of sending SCSI CDBs to the controller there is > no reason why we shouldn't exploit it. > This way we get to use all the nice sg_utils

Re: [PATCH] Perverting cciss

2007-04-05 Thread Christoph Hellwig
On Thu, Apr 05, 2007 at 11:58:06AM +0200, Hannes Reinecke wrote: Hi All, this patch adds the SG_IO ioctl to the cciss driver. As the driver is capable of sending SCSI CDBs to the controller there is no reason why we shouldn't exploit it. This way we get to use all the nice sg_utils for the

Re: [PATCH] Perverting cciss

2007-04-05 Thread James Bottomley
On Thu, 2007-04-05 at 11:03 +0100, Christoph Hellwig wrote: On Thu, Apr 05, 2007 at 11:58:06AM +0200, Hannes Reinecke wrote: Hi All, this patch adds the SG_IO ioctl to the cciss driver. As the driver is capable of sending SCSI CDBs to the controller there is no reason why we shouldn't

RE: [PATCH] Perverting cciss

2007-04-05 Thread Miller, Mike (OS Dev)
James wrote: On Thu, 2007-04-05 at 11:03 +0100, Christoph Hellwig wrote: On Thu, Apr 05, 2007 at 11:58:06AM +0200, Hannes Reinecke wrote: Hi All, this patch adds the SG_IO ioctl to the cciss driver. As the driver is capable of sending SCSI CDBs to the controller there is no