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

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


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

On Fri, Nov 30, 2007 at 07:38:12PM +0100, Hans de Goede wrote:
> Matthew Dharm wrote:
> > On Fri, Nov 30, 2007 at 10:35:56AM -0500, Kyle McMartin wrote:
> >> On Fri, Nov 30, 2007 at 12:33:28AM -0800, Matthew Dharm wrote:
> >>> Someone remind me again why we bother masking out the fake CD-ROM in =
these
> >>> devices?
> >>>
> >> It's confusing to users, and completely useless -- it's added by the
> >> manufacturer because Another OS can only autorun from a cdrom, which
> >> they need to do to be able to install drivers.
> >=20
> > Beyond being confusing to users, does it cause any *problems*?
> >=20
> > I'm hesitant to add code which doesn't actually *fix* anything.
> >=20
>=20
> Well it fixes the very annoying fact that when you plug one of these in a=
nd are=20
> using a hal enabled desktop first a filemanager window with your files po=
ps up=20
> and then when you want to navigate to the correct dir a filemanger window=
 with=20
> autorun.inf and other stuff pops up.

Then shouldn't you talk to the people who write the code to make the window
pop up?  That's a desktop environment issue, not a driver issue.

> But I believe that just adding prove only a single lun flag entry for the=
se is=20
> wrong. I would like to advocate a special U3 flag and a mod param / sysfs=
=20
> attribute for the usb-massstorage driver which allows one to configure th=
e=20
> behaviour. Some people have come up with all kind of ingenious uses for t=
he=20
> cdrom emulation.

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.

> Still I believe that some kernel support is needed to avoid the _very_ an=
noying=20
> popping up of the second useless lun, but this should be a special U3 fla=
g so=20
> that it can be configured. Also note that U3 is not sandisk only, there i=
s an=20
> U3 consortium and others may very well come with U3 disks in the future t=
oo.

Definately not.  If the poping up of a window is the problem, then fix the
window; don't fix the driver.

Matt

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

E:  You run this ship with Windows?!  YOU IDIOT!
L:  Give me a break, it came bundled with the computer!
					-- ESR and Lan Solaris
User Friendly, 12/8/1998

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

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

iD8DBQFHUFyNHL9iwnUZqnkRAkUIAKCaYR3iiFozrS95yoTZhLL25RXBdQCePIjm
awCoaiznJ+hjKw+dABngFmw=
=8Bu2
-----END PGP SIGNATURE-----

--FwyhczKCDPOVeYh6--


--===============0284636189==
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
--===============0284636189==
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

--===============0284636189==--