From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: Greg KH <greg@kroah.com>
Cc: stable@kernel.org, torvalds@osdl.org, scjody@modernduck.com,
linux1394-devel@lists.sourceforge.net, bcollins@debian.org,
adq@lidskialf.net, linux-kernel@vger.kernel.org,
James Bottomley <James.Bottomley@SteelEye.com>
Subject: Re: [stable] [PATCH] sbp2: fix panic when ejecting an ipod
Date: Sun, 11 Dec 2005 02:02:07 +0100 [thread overview]
Message-ID: <439B7A8F.6000209@s5r6.in-berlin.de> (raw)
In-Reply-To: <20051210232837.GE11094@kroah.com>
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/
next prev parent reply other threads:[~2005-12-11 1:04 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20051209171922.GW19441@conscoop.ottawa.on.ca>
2005-12-10 11:24 ` [PATCH] sbp2: fix panic when ejecting an ipod Stefan Richter
2005-12-10 23:28 ` [stable] " Greg KH
2005-12-11 1:02 ` Stefan Richter [this message]
2005-12-14 20:09 ` Stefan Richter
2005-12-14 20:38 ` Greg KH
2005-12-14 22:32 ` [PATCH 2.6.14.3 1/2] SCSI: fix transfer direction in sd (kernel panic when ejecting iPod) Stefan Richter
2005-12-14 22:34 ` [PATCH 2.6.14.3] SCSI: fix transfer direction in scsi_lib and st Stefan Richter
2005-12-14 22:43 ` Stefan Richter
2005-12-23 19:05 ` [stable] " Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=439B7A8F.6000209@s5r6.in-berlin.de \
--to=stefanr@s5r6.in-berlin.de \
--cc=James.Bottomley@SteelEye.com \
--cc=adq@lidskialf.net \
--cc=bcollins@debian.org \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux1394-devel@lists.sourceforge.net \
--cc=scjody@modernduck.com \
--cc=stable@kernel.org \
--cc=torvalds@osdl.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).