public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Nigel Cunningham <ncunningham@linuxmail.org>
To: Dave Jones <davej@redhat.com>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: peculiar suspend/resume bug.
Date: Wed, 16 Aug 2006 11:05:44 +1000	[thread overview]
Message-ID: <1155690344.3193.25.camel@nigel.suspend2.net> (raw)
In-Reply-To: <20060816003728.GA3605@redhat.com>

Hi Dave.

On Tue, 2006-08-15 at 20:37 -0400, Dave Jones wrote:
> On Wed, Aug 16, 2006 at 10:19:59AM +1000, Nigel Cunningham wrote:
>  > Hi Dave.
>  > 
>  > On Tue, 2006-08-15 at 18:10 -0400, Dave Jones wrote:
>  > > Here's a fun one.
>  > > - Get a dual core cpufreq aware laptop (Like say, a core-duo)
>  > > - Add a cpufreq monitor to gnome-panel. Configure it
>  > >   to watch the 2nd core.
>  > > - Suspend.
>  > > - Resume.
>  > > 
>  > > Watch the cpufreq monitor die horribly.
>  > > 
>  > > I believe this is because we take down the 2nd core at suspend
>  > > time with cpu hotplug, and for some reason we're scheduling
>  > > userspace before we bring that second core back up.
>  > > 
>  > > Anyone have any clues why this is happening?
>  > 
>  > If you hotunplug and replug the cpu using the sysfs interface, rather
>  > than suspending and resuming, does the same thing happen?
> 
> cpufreq-applet crashes as soon as the cpu goes offline.
> Now, the applet should be written to deal with this scenario more
> gracefully, but I'm questioning whether or not userspace should
> *see* the unplug/replug that suspend does at all.
> 
> IMO, when we shouldn't schedule userspace until the system is
> in the exact state it was before we suspended.

At the moment, the cpu hotplugging/unplugging is done outside of
freezing processes because once we've frozen processes we can't (afaik)
move ones that are tied to the cpu being unplugged to another processor,
and won't also be able to kill kernel threads that are tied to the
processor(s) being taken down.

Personally, I wouldn't mind being seeing this addressed as I see a few
other benefits to being able to hot[un]plug later, besides simplifying
life for the cpufreq-applet (although it shouldn't crash if a cpu is
offlined anyway).

Regards,

Nigel


  reply	other threads:[~2006-08-16  1:05 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-15 22:10 peculiar suspend/resume bug Dave Jones
2006-08-16  0:19 ` Nigel Cunningham
2006-08-16  0:37   ` Dave Jones
2006-08-16  1:05     ` Nigel Cunningham [this message]
2006-08-16  2:41     ` Matthew Garrett
2006-08-16  3:53       ` Dave Jones
2006-08-16  8:54         ` Rafael J. Wysocki
2006-08-17  1:44       ` Nigel Cunningham
2006-08-17  5:44         ` Rafael J. Wysocki
2006-08-17  5:55           ` Nigel Cunningham
2006-08-17  6:30             ` Rafael J. Wysocki
2006-08-16 22:06 ` Pavel Machek

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=1155690344.3193.25.camel@nigel.suspend2.net \
    --to=ncunningham@linuxmail.org \
    --cc=davej@redhat.com \
    --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