LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Julian Blake Kongslie <jblake@omgwallhack.org>
Cc: LKML <linux-kernel@vger.kernel.org>, Pavel Machek <pavel@ucw.cz>
Subject: Re: Hang on suspend
Date: Sun, 24 Feb 2008 23:49:29 +0100	[thread overview]
Message-ID: <200802242349.29396.rjw@sisk.pl> (raw)
In-Reply-To: <1203893049.3983.6.camel@dukephillips.omgwallhack.org>

On Sunday, 24 of February 2008, Julian Blake Kongslie wrote:
> On Sun, 2008-02-24 at 23:35 +0100, Rafael J. Wysocki wrote:
> > [Please don't drop the CC list from replies.]
> 
> Yeah, sorry, I accidentally whacked the wrong reply button.
> 
> > On Sunday, 24 of February 2008, Julian Blake Kongslie wrote:
> > > On Sun, 2008-02-24 at 12:23 +0100, Rafael J. Wysocki wrote:
> > > > Can you please check what's at the address platform_suspend_late+0x19?
> > > 
> > > 0xc0253ca9 is in platform_suspend_late (drivers/base/platform.c:579).
> > > 574	{
> > > 575		struct platform_driver *drv = to_platform_driver(dev->driver);
> > > 576		struct platform_device *pdev;
> > > 577		int ret = 0;
> > > 578	
> > > 579		pdev = container_of(dev, struct platform_device, dev);
> > > 580		if (dev->driver && drv->suspend_late)
> > > 581			ret = drv->suspend_late(pdev, mesg);
> > > 582	
> > > 583		return ret;
> > 
> > That's strange, it looks like the container_of() accessing beyond the
> > structure.
> > 
> > Is this 100% reproducible?
> 
> Yes, it is. I've had this hang on suspend since at least some time in
> early 2.6.24-rc, although because I have to transcribe the oops manually
> I haven't had enough incentive to get it reported until now.
> Unfortunately, I just don't need to suspend very often, so it may have
> been introduced some time before then.
> 
> If you don't have any other ideas,

No, I don't.

> I can try to bisect and find the change that introduced it,

That would be great.

> but it'll probably take me a day or two; I have other things I need to use my
> computer for, so it's a bit difficult to do the constant-reboot-and-test
> cycle. 

Not a problem at all.

Thanks,
Rafael

  reply	other threads:[~2008-02-24 22:50 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-21  0:32 Julian Blake Kongslie
2008-02-23 20:43 ` Rafael J. Wysocki
2008-02-24  4:30   ` Julian Blake Kongslie
2008-02-24 11:23     ` Rafael J. Wysocki
2008-02-24 21:20       ` Julian Blake Kongslie
2008-02-24 22:35         ` Rafael J. Wysocki
2008-02-24 22:44           ` Julian Blake Kongslie
2008-02-24 22:49             ` Rafael J. Wysocki [this message]
2008-03-13  8:13             ` Pavel Machek
2008-03-14 17:43               ` Julian Blake Kongslie

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=200802242349.29396.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=jblake@omgwallhack.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@ucw.cz \
    --subject='Re: Hang on suspend' \
    /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

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