Re: [stable] [PATCH] sbp2: fix panic when ejecting an ipod

From: Stefan Richter
Date: Sat Dec 10 2005 - 21:15:25 EST


Greg KH wrote:
On Sat, Dec 10, 2005 at 12:24:59PM +0100, Stefan Richter wrote:

sbp2: fix panic when ejecting an ipod

Sbp2 did not catch some bogus transfer directions in requests from upper
layers. Problem became apparent when iPods were to be ejected:
http://marc.theaimsgroup.com/?l=linux1394-devel&m=113399994920181
http://marc.theaimsgroup.com/?l=linux1394-user&m=112152701817435
Debugging and original variant of the patch by Andrew de Quincey.

Signed-off-by: Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx>
Cc: Andrew de Quincey <adq@xxxxxxxxxxxxx>


Is this in linus's tree yet?

My idea was that it goes in parallel to Linus and to -stable, hence the selection of recipients of my posting.

I will submit two related cleanup patches for sbp2 to linux1394-devel on Sunday morning. They will remove obsolete code and reformat code for readability. IMO they will *not* be suitable for Linus' tree before the next subsystem merge window.

Do the 1394 maintainers accept it as a valid fix?

Ben Collins and I are *sbp2* maintainers. I consider it a valid fix (but see below.) Jody McIntyre and Ben are *1394* maintainers. Jody posted a NAK a few hours ago:
|| NAK. James has a patch to fix this in the SCSI layer, which is his
|| preference.

I agree with Jody and the SCSI people that Jens' and James' patches are the actual fixes. What I want to accomplish is twofold:
- Don't let tiny mistakes lead to catastrophic failure (panic) if it
can be avoided without additional code.
- Get the panic fixed in -stable in one way or another ASAP.

James, I assume Jens' and your patch will be in Linus' tree soon. Therefore and because my pending sbp2 cleanups will land in Linus' tree eventually, this sbp2 patch here is not vital for the current kernel. But do you consider to submit the SCSI fixes or a derivative to -stable too? If not, I recommend my patch to be included in -stable.

Jody, I very much respect and appreciate your opinion. Please continue to step in the way when I'm doing goofy things. :-)
--
Stefan Richter
-=====-=-=-= ==-- -=-==
http://arcgraph.de/sr/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/