mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Tino Keitel <tino.keitel@gmx.de>
To: Pavel Machek <pavel@ucw.cz>, linux-usb@vger.kernel.org
Cc: Soeren Sonnenburg <sonne@nn7.de>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel@vger.kernel.org, Len Brown <lenb@kernel.org>
Subject: Re: 2.6.25-rc6 hangs at resume after suspend to RAM on Mac mini Core Duo
Date: Tue, 1 Apr 2008 20:37:32 +0200	[thread overview]
Message-ID: <20080401183732.GA5043@dose.home.local> (raw)
In-Reply-To: <20080401105213.GB10554@elf.ucw.cz>

On Tue, Apr 01, 2008 at 12:52:13 +0200, Pavel Machek wrote:
> Hi!
> 
> > > > [...]
> > > > 
> > > > > Can you please try to boot with acpi_new_pts_ordering and retest?
> > > > 
> > > > I just tried current -git (a9edadbf790d72adf6ebed476cb5caf7743e7e4a),
> > > > without success. I still got the same hand at resume.
> > > 
> > > I am having the hang on resume problem too (macbook pro), cf.
> > > http://bugzilla.kernel.org/show_bug.cgi?id=10319, 2.6.24 was fine.
> > 
> > What about 2.6.25-rc5? Up to this, resume works for me. -rc6 is broken.
> 
> Any chance to bisect?

git-bisect revealed this:

$ git bisect good
e82cc1288fa57857c6af8c57f3d07096d4bcd9d9 is first bad commit
commit e82cc1288fa57857c6af8c57f3d07096d4bcd9d9
Author: David Brownell <david-b@pacbell.net>
Date:   Fri Mar 7 13:49:42 2008 -0800

    USB: fix ehci unlink regressions
    
    The recent EHCI driver update to split the IAA watchdog timer out from
    the other timers made several things work better, but not everything;
    and it created a couple new issues in bugzilla.  Ergo this patch:
    
      - Handle a should-be-rare SMP race between the watchdog firing
        and (very late) IAA interrupts;
    
      - Remove a shouldn't-have-been-added WARN_ON() test;
    
      - Guard against one observed OOPS;
    
      - If this watchdog fires during clean HC shutdown, it should act
        as a NOP instead of interfering with the shutdown sequence;
    
      - Guard against silicon errata hypothesized by some vendors:
          * IAA status latch broken, but IAAD cleared OK;
          * IAAD wasn't cleared when IAA status got reported;
    
    The WARN_ON is in bugzilla as 10168; the OOPS as 10078; these are
    both regressions.
    
    Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
    Tested-by: Gordon Farquharson <gordonfarquharson@gmail.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

:040000 040000 ff621bb62396fdf71c7b97877ddbb8542f57b871
251bc90076ea0b3a14d0b88dd250581fa815b5de M	drivers

For the linux-usb people: this commit breaks resume from suspend to RAM
on my Mac mini Core Duo. It worked with 2.6.24, 2.6.25-rc5, and was
broken since 2.6.25-rc6 (also with -rc7 and later). See
http://lkml.org/lkml/2008/3/20/23 for more information.

Regards,
Tino

  parent reply	other threads:[~2008-04-01 18:37 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-20  6:05 Tino Keitel
2008-03-20 10:10 ` Rafael J. Wysocki
2008-03-20 10:25 ` Andrew Morton
2008-03-20 10:54   ` Tino Keitel
2008-03-25 23:13     ` Tino Keitel
2008-03-28 23:03       ` Rafael J. Wysocki
2008-03-31 21:28         ` Tino Keitel
2008-03-31 21:50           ` Pavel Machek
2008-03-31 22:01             ` Tino Keitel
2008-03-31 22:17               ` Rafael J. Wysocki
2008-04-01  8:04                 ` Tino Keitel
2008-04-01 21:03                   ` Rafael J. Wysocki
2008-04-01  6:23           ` Soeren Sonnenburg
2008-04-01  6:27             ` Tino Keitel
2008-04-01  6:38               ` Soeren Sonnenburg
2008-04-01 21:05                 ` Rafael J. Wysocki
2008-04-02 17:33                   ` Soeren Sonnenburg
2008-04-02 17:56                     ` Rafael J. Wysocki
2008-04-02 18:42                       ` Soeren Sonnenburg
2008-04-01 10:52               ` Pavel Machek
2008-04-01 12:55                 ` Tino Keitel
2008-04-01 18:37                 ` Tino Keitel [this message]
2008-04-01 19:06                   ` Alan Stern
2008-04-01 20:25                     ` Tino Keitel
2008-04-01 21:02                       ` Alan Stern
2008-04-01 21:06                         ` Tino Keitel
2008-04-02 14:25                           ` Alan Stern
2008-04-02 19:26                           ` Alan Stern
2008-04-02 19:47                             ` Tino Keitel
2008-04-02 20:37                               ` Soeren Sonnenburg
2008-04-01 20:06                   ` David Brownell

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=20080401183732.GA5043@dose.home.local \
    --to=tino.keitel@gmx.de \
    --cc=akpm@linux-foundation.org \
    --cc=lenb@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=pavel@ucw.cz \
    --cc=rjw@sisk.pl \
    --cc=sonne@nn7.de \
    /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

Powered by JetHome