Oh, one more thing I forgot... this test was on 2.4.0-test6 with a patch to
the latest usb-storage code from the CVS archive. Anyone who wants to
look at that code can get it from
cvs -d:pserver:cvs.linux-usb.sourceforge.net:/cvsroot/linux-usb co storage
The in_interrupt() test in the command_abort() function is not in the stock
2.4.0-test6 kernel -- it's only in the CVS archive.
Matt
On Sun, Aug 13, 2000 at 01:07:22PM -0700, Matthew Dharm wrote:
> I've confirmed the presence of (what I consider to be) an urgent 2.4.0 bug
> in the scsi error handling code. For a long time I thought this was a bug
> in the usb-storage code, but it's now clearly a SCSI layer bug.
>
> The following comes from 2.4.0-test6.
>
> This is what appears on the console:
> -----------------------------------
> usb-storage: command_abort() called from an interrupt!!! BAD!!! ...
> usb-storage: bus_reset() requested but not implemented
> usb-storage: bus_reset() requested but not implemented
> usb-storage: host_reset() requested but not implemented
>
> Note that the first line comes from the usb-storage command abort function,
> and is a test to see if(in_interrupt()) -- this is the primary indication
> of the bug. This function is supposed to be called from a thread context
> only, and is not IRQ context safe. But, it's not supposed to have to be
> IRQ safe.
>
> Matt Dharm
>
> --
> Matthew Dharm Home: [EMAIL PROTECTED]
> Senior Engineer, QCP Inc. Work: [EMAIL PROTECTED]
>
> Department of Justice agent. I have come to purify the flock.
> -- DOJ agent
> User Friendly, 5/22/1998
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to [EMAIL PROTECTED]
--
Matthew Dharm Home: [EMAIL PROTECTED]
Senior Engineer, QCP Inc. Work: [EMAIL PROTECTED]
Somebody call an exorcist!
-- Dust Puppy
User Friendly, 5/16/1998
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]