On Fri, 2007-11-30 at 17:14 -0800, Greg KH wrote:
> On Fri, Nov 30, 2007 at 07:56:09PM -0500, Ben Collins wrote:
> > 
> > On Fri, 2007-11-30 at 12:46 -0800, Matthew Dharm wrote:
> > > (Adding Ben Collins, who put another U3 patch into the system.)
> > > 
> > > On Fri, Nov 30, 2007 at 08:40:25PM +0100, Hans de Goede wrote:
> > > > David Brown wrote:
> > > > > On Fri, Nov 30, 2007 at 10:55:09AM -0800, Matthew Dharm wrote:
> > > > > 
> > > > >> This is overly complex, and doesn't really solve anything.  The fake 
> > > > >> CD-ROM
> > > > >> is there.  Who knows what's on it?  If the desktop wants to ignore 
> > > > >> it,
> > > > >> that's fine... but as a transport-oriented driver, usb-storage should
> > > > >> cleanly transport the commands as much as possible.
> > > > > 
> > > > > It is also possible to change the data that is on the fake CD-ROM, so 
> > > > > this
> > > > > could conceivably be used for something from Linux as well.  I don't 
> > > > > know
> > > > > if anyone has figured out how to change the CD-ROM image from Linux, 
> > > > > but it
> > > > > is certainly a possibility.
> > > > > 
> > > > 
> > > > I understand, well then someone needs to file an RFE for this against 
> > > > hal and 
> > > > we all need to help to get a list of ids of u3 deviced in hal.
> > > 
> > > We should also find out why there is another U3 patch in the kernel.  It
> > > came in with a patch that referred to some Ubuntu bugs.  Ben, can you
> > > clarify what bug that was, and what the original report was?
> > 
> > Not sure if that was an Ubuntu bug. Pretty sure that was a stick I had,
> > and added to the list to get rid of the cdrom lun.
> 
> I agree with Matt, this should not be done in the kernel.  If you don't
> like seeing the additional LUN, which is really present in the device,
> do something in HAL or your window manager.  That kind of "policy"
> should not be in the usb-storage driver.
> 
> If anyone has the git id for the current U3 patch, I'll be glad to
> revert it.

Makes sense to me. I was only following precedent from other devices
already being treated like this.

-- 
Ubuntu   : http://www.ubuntu.com/
Linux1394: http://wiki.linux1394.org/
SwissDisk: http://www.swissdisk.com/


-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
_______________________________________________
Linux-usb-users@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-users

Reply via email to