lkml.org 
[lkml]   [2005]   [Dec]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [stable] [PATCH] sbp2: fix panic when ejecting an ipod
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@s5r6.in-berlin.de>
>>Cc: Andrew de Quincey <adq@lidskialf.net>
>
>
> 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@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-12-11 02:07    [from the cache]
©2003-2011 Jasper Spaans