RE: [PATCH] Perverting cciss

[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 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 free.
> > 
> > Instead of adding yet another implementation of SG_IO 
> please implement 
> > support for REQ_TYPE_BLOCK_PC requests and add all the nice block 
> > layer passthrough ioctls to it.
> 
> Actually, I happen to know that HP is in the process of 
> implementing this correctly (via REQ_TYPE_BLOCK_PC).  I can't 
> reveal the details but it has something to do with a well 
> known Linux High Availability company needing SG_IO for 
> sg_persist to work ...
> 
> James

NAK. Please do not add this patch to cciss. We are working SG_IO
internally and will post the patch soon.

mikem


> 
> 
> 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/