public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Jan Scholz <scholz@fias.uni-frankfurt.de>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Paul Mackerras <paulus@samba.org>, Ingo Molnar <mingo@elte.hu>,
	linux-kernel@vger.kernel.org, Adrian Bunk <bunk@kernel.org>,
	pm list <linux-pm@lists.linux-foundation.org>
Subject: Re: [regression, bisected] adb trackpad disappears after suspend to ram
Date: Fri, 29 May 2009 20:10:09 +0200	[thread overview]
Message-ID: <200905292010.10615.rjw@sisk.pl> (raw)
In-Reply-To: <87tz34n97p.fsf@scholz.fias.uni-frankfurt.de>

On Friday 29 May 2009, Jan Scholz wrote:
> Benjamin Herrenschmidt <benh@kernel.crashing.org> writes:
> 
> >> We are too late in the cycle to revert this commit and it really is needed to
> >> fix a more serious issue.  Nevertheless knowing that it caused the problem to
> >> appear on your system is also important.
> >> 
> >> Ben, do you have an idea what may be going on here?  Does __disable_irq() mask
> >> the interrupt on this platform?
> >
> > I suppose so :-) I'll have to check what's going on, it's not
> > immediately clear to me.
> >
> > Cheers,
> > Ben.
> >
> >> 
> >> I'd like to see a boot log, preferably containing a suspend-resume in which
> >> the problem was reproduced.
> 
> Here is a log from booting, through several suspend cycles, until the
> trackpad disappeared. The first few suspends were done while X was
> running but from the console. As you can tell from lines like
>    May 28 23:51:26 [kernel] adb devices: [2]: 2 c4 [3]: 3 1 [7]: 7 1f
> the trackpad was still present. The last suspend was done from within X,
> here the trackpad did not make it.
>    May 28 23:58:09 [kernel] adb devices: [2]: 2 c4 [7]: 7 1f
> However, there have been cases, although not in this log, were the
> trackpad has been alive even when suspending from within X.

This means the problem is probably timing-related.

Can you please try to comment out suspend_device_irqs() and
resume_device_irqs()  in drivers/base/power/main.c and see if that changes
anything?  It's not entirely safe (well, that's why the calls are there after
all), but hopefully the box won't hang during this test.

Best,
Rafael

  reply	other threads:[~2009-05-29 18:10 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-25 15:44 [regression] adb trackpad disappears after suspend to ram Jan Scholz
2009-05-28  7:59 ` [regression, bisected] " Jan Scholz
2009-05-28 16:58   ` Rafael J. Wysocki
2009-05-28 22:23     ` Benjamin Herrenschmidt
2009-05-28 22:39       ` Jan Scholz
2009-05-29 18:10         ` Rafael J. Wysocki [this message]
2009-06-01 13:44           ` Jan Scholz
2009-06-01 16:36             ` Rafael J. Wysocki
2009-06-01 22:34               ` Jan Scholz
2009-06-01 22:42                 ` Rafael J. Wysocki
2009-06-01 22:52                   ` Benjamin Herrenschmidt
     [not found]                 ` <200906031202.28916.rjw@sisk.pl>
2009-06-03 12:18                   ` Jan Scholz
     [not found]                     ` <200906032200.55563.rjw@sisk.pl>
     [not found]                       ` <200906032220.19915.rjw@sisk.pl>
2009-06-03 22:17                         ` [linux-pm] " Jan Scholz
2009-09-23  3:32                       ` Benjamin Herrenschmidt
2009-09-23 13:38                         ` Rafael J. Wysocki
2009-09-23 21:28                           ` Benjamin Herrenschmidt
2009-10-07 17:10                             ` Jan Scholz
2009-10-07 20:18                               ` Rafael J. Wysocki
2009-09-23 23:12                           ` Jan Scholz
2009-06-02  4:49               ` Benjamin Herrenschmidt

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=200905292010.10615.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=benh@kernel.crashing.org \
    --cc=bunk@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=mingo@elte.hu \
    --cc=paulus@samba.org \
    --cc=scholz@fias.uni-frankfurt.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