Re: [usb-storage] [PATCH] Another SanDisk USB stick with a cdrom scsi lun

--===============2026803021==
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="xGGVyNQdqA79rdfn"
Content-Disposition: inline


--xGGVyNQdqA79rdfn
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, Nov 30, 2007 at 05:14:42PM -0800, Greg KH wrote:
> On Fri, Nov 30, 2007 at 07:56:09PM -0500, Ben Collins wrote:
> >=20
> > On Fri, 2007-11-30 at 12:46 -0800, Matthew Dharm wrote:
> > > (Adding Ben Collins, who put another U3 patch into the system.)
> > >=20
> > > 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:
> > > > >=20
> > > > >> This is overly complex, and doesn't really solve anything.  The =
fake=20
> > > > >> CD-ROM
> > > > >> is there.  Who knows what's on it?  If the desktop wants to igno=
re it,
> > > > >> that's fine... but as a transport-oriented driver, usb-storage s=
hould
> > > > >> cleanly transport the commands as much as possible.
> > > > >=20
> > > > > 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 do=
n't know
> > > > > if anyone has figured out how to change the CD-ROM image from Lin=
ux, but it
> > > > > is certainly a possibility.
> > > > >=20
> > > >=20
> > > > I understand, well then someone needs to file an RFE for this again=
st hal and=20
> > > > we all need to help to get a list of ids of u3 deviced in hal.
> > >=20
> > > 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?
> >=20
> > 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.
>=20
> 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.
>=20
> If anyone has the git id for the current U3 patch, I'll be glad to
> revert it.

I don't have a git id, but it was from May of this year.

It looks like only one entry, which is commented with Ben's e-mail address,
needs to be removed.

Matt

--=20
Matthew Dharm                              Home: mdharm-usb@one-eyed-alien.=
net=20
Maintainer, Linux USB Mass Storage Driver

God, root, what is difference?
					-- Pitr
User Friendly, 11/11/1999

--xGGVyNQdqA79rdfn
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)

iD8DBQFHULtZHL9iwnUZqnkRAtxCAJ0QXWUK4LGpbiYV8DB9dG8lap3sSQCgqyCD
5Mpf6PWkAJCdI3mvafk8Vfo=
=caLB
-----END PGP SIGNATURE-----

--xGGVyNQdqA79rdfn--


--===============2026803021==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

-------------------------------------------------------------------------
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
--===============2026803021==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-users

--===============2026803021==--