linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [BUG 2.6.38.8] Bisected to commit 5fb46ae7b8c51b05a12c6a66108e8d398c20ee09
@ 2011-07-04 13:06 Knut Petersen
  2011-07-07  0:33 ` Greg KH
  0 siblings, 1 reply; 5+ messages in thread
From: Knut Petersen @ 2011-07-04 13:06 UTC (permalink / raw)
  To: gregkh; +Cc: axboe, kay.sievers, David S. Miller, linux-ide, linux-kernel

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>


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-07-07 13:34 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-04 13:06 [BUG 2.6.38.8] Bisected to commit 5fb46ae7b8c51b05a12c6a66108e8d398c20ee09 Knut Petersen
2011-07-07  0:33 ` Greg KH
2011-07-07  4:23   ` Knut Petersen
2011-07-07  4:43     ` Gene Heskett
2011-07-07 13:33     ` Greg KH

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).