From: Knut Petersen <Knut_Petersen@t-online.de>
To: gregkh@suse.de
Cc: axboe@kernel.dk, kay.sievers@vrfy.org,
"David S. Miller" <davem@davemloft.net>,
linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [BUG 2.6.38.8] Bisected to commit 5fb46ae7b8c51b05a12c6a66108e8d398c20ee09
Date: Mon, 04 Jul 2011 15:06:40 +0200 [thread overview]
Message-ID: <4E11BAE0.5020208@t-online.de> (raw)
Hi everybody!
An openSuse 11.2 system reliably refused to boot properly with kernel 2.6.38.8
although the prior 2.6.38.y kernels worked fine. After a few seconds booting
stopped. Several <ctrl-c> did help, but a lot of services were not started properly,
only the root partition was available. Nothing unusual could be found in the
kernel messages.
I bisected the problem to commit 5fb46ae7b8c51b05a12c6a66108e8d398c20ee09.
That gave me the idea to disconnect the Plextor PX-755A dvd drive and to
try 2.6.38.8 again. It booted without problem.
As I do use the dvd drive only once or twice a year on that system I checked the
hardware - it seems that the drive has a mechanical or optical problem.
No kernel can repair a broken dvd drive, but I strongly believe that at least
error handling should be restored to something more reasonable.
cu,
Knut
knut@linux:~/git/linux-2.6.38.y> git bisect bad
5fb46ae7b8c51b05a12c6a66108e8d398c20ee09 is the first bad commit
commit 5fb46ae7b8c51b05a12c6a66108e8d398c20ee09
Author: Tejun Heo <tj@kernel.org>
Date: Wed Mar 9 19:54:27 2011 +0100
ide: Convert to bdops->check_events()
commit 5b03a1b140e13a28ff6be1526892a9dc538ddef6 upstream.
Convert ->media_changed() to the new ->check_events() method. The
conversion is mostly mechanical. The only notable change is that
cdrom now doesn't generate any event if @slot_nr isn't CDSL_CURRENT.
It used to return -EINVAL which would be treated as media changed. As
media changer isn't supported anyway, this doesn't make any
difference.
This makes ide emit the standard disk events and allows kernel event
polling. Currently, only MEDIA_CHANGE event is implemented. Adding
support for EJECT_REQUEST shouldn't be difficult; however, given that
ide driver is already deprecated, it probably is best to leave it
alone.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Jens Axboe <axboe@kernel.dk>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: linux-ide@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
next reply other threads:[~2011-07-04 13:25 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-04 13:06 Knut Petersen [this message]
2011-07-07 0:33 ` [BUG 2.6.38.8] Bisected to commit 5fb46ae7b8c51b05a12c6a66108e8d398c20ee09 Greg KH
2011-07-07 4:23 ` Knut Petersen
2011-07-07 4:43 ` Gene Heskett
2011-07-07 13:33 ` 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=4E11BAE0.5020208@t-online.de \
--to=knut_petersen@t-online.de \
--cc=axboe@kernel.dk \
--cc=davem@davemloft.net \
--cc=gregkh@suse.de \
--cc=kay.sievers@vrfy.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.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