Re: Problem with usb-storage using HP 8200 external CD-ROM burner

[Matthew Dharm]

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

On Mon, Aug 06, 2001 at 03:49:46PM -0400, Brent Baccala wrote:
> Matthew Dharm wrote:
> > Of course, I'm interested in knowing how the command_abort function can=
 be
> > made safe -- I think there are already patches in the 2.4.8 kernel which
> > should fix the cause of this function getting called.
> >=20
> > Any ideas on how to fix this issue?
>=20
> Well, what comes to mind immediately is two things.
>=20
> First, does scsiglue.c's abort_command really need to handshake with the
> code in usb.c?  If not, just get rid of the down and its matching up.

Unfortunately, it does.  The SCSI layer seems to believe that once we've
returned from the abort_command() routine, the driver is in a position to
accept a new command.  Thus, some level of handshaking is necessary.

> Second, this code (in scsi_error.c):
>=20
>       774         spin_lock_irqsave(&io_request_lock, flags);
>       775         rtn =3D SCpnt->host->hostt->eh_abort_handler(SCpnt);
>       776         spin_unlock_irqrestore(&io_request_lock, flags);
>=20
> seems like a real shotgun approach.  Get rid of the spinlock stuff, and
> make sure that the abort handlers lock io_request_lock themselves if
> they need it.  Of course, this would require changes to all the scsi
> drivers.

Hrm... perhaps I could just unlock that spinlock and then re-lock it before
returning.  Anyone have a clue if this would work?

Matt

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

Would you mind not using our Web server? We're trying to have a game of=20
Quake here.
					-- Greg
User Friendly, 5/11/1998

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

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE7b13az64nssGU+ykRAhi9AKDDR86RyGgJEiq+ZDOiSV8XoN9rkQCg/RVP
omYkrZfojY8DtJSWQlhMR10=
=BGo7
-----END PGP SIGNATURE-----

--jousvV0MzM2p6OtC--