[linux-usb] USB storage race still there

[Pavel Machek]

Hi!

USB storage race is still there: in Bulk_transport, you are doing

@@ -991,6 +976,9 @@
        US_DEBUGP("Bulk command S 0x%x T 0x%x L %d F %d CL %d\n",
                  bcb.Signature, bcb.Tag, bcb.DataTransferLength,
                  bcb.Flags, bcb.Length);
+/* BUG BUG BUG!
+   We are calling schedule, but we are holding spinlock_irqsave!
+*/
        result = usb_bulk_msg(us->pusb_dev, pipe, &bcb,
                              US_BULK_CB_WRAP_LEN, &partial, HZ*5);
        US_DEBUGP("Bulk command transfer result=%d\n", result);

which just can't be right.

                                                                Pavel

-- 
I'm [EMAIL PROTECTED] "In my country we have almost anarchy and I don't care."
Panos Katsaloulis describing me w.r.t. patents me at [EMAIL PROTECTED]

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]