linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: John Stultz <john.stultz@linaro.org>
To: Peter Zijlstra <peterz@infradead.org>
Cc: lkml <linux-kernel@vger.kernel.org>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	arve@android.com, markgross@thegnar.org,
	Alan Stern <stern@rowland.harvard.edu>,
	amit.kucheria@linaro.org, farrowg@sg.ibm.com,
	"Dmitry Fink (Palm GBU)" <Dmitry.Fink@palm.com>,
	linux-pm@lists.linux-foundation.org, khilman@ti.com,
	Magnus Damm <damm@opensource.se>,
	mjg@redhat.com, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH 0/6] [RFC] Proposal for optimistic suspend idea.
Date: Wed, 28 Sep 2011 20:45:25 -0700	[thread overview]
Message-ID: <1317267925.3112.810.camel@work-vm> (raw)
In-Reply-To: <1317200359.5781.47.camel@twins>

On Wed, 2011-09-28 at 10:59 +0200, Peter Zijlstra wrote:
> On Tue, 2011-09-27 at 15:56 -0700, John Stultz wrote:
> > 
> > But I also want to separate my specific solution from the problem at
> > large. I do think that there are issues that my proposal and wakelocks
> > address that the hand-wavy "just do it in userspace" rebuttals don't
> > deal with (again specifically: wakeup event consumption in userland
> > before the next suspend). 
> 
> You know, once you drop the whole suspend notion that race goes away.
> 
> Esp. on the mobile hardware there really isn't anything different
> between a deep idle state and suspend.

Well, except timer noise and device irq noise.

> So just make the thing idle and your suspend race goes away.

Maybe hardware vendors will surprise me, but I don't think the power
difference between suspend and idle will get close enough in a
reasonable amount of time on server hardware.

Even then, I doubt you'll see standard distros that get minutes of
un-interrupted deep-idle. 

How long until you realistically expect to leave your laptop overnight
off of AC without suspending it (and not have it be on fumes in the
morning)? 

> There's still things like the cgroup-freezer if you really want to force
> stuff down, but really your core system should be sane and not actually
> do anything unless asked.

I think the cgroup-freezer is closer to the lines I'm thinking of, but
with the potential to do "importance" inheritance so interactions
between tasks in different groups (be they cgroups or sched classes) can
work normally.

thanks
-john



  reply	other threads:[~2011-09-29  3:45 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-26 19:13 [PATCH 0/6] [RFC] Proposal for optimistic suspend idea John Stultz
2011-09-26 19:13 ` [PATCH 1/6] [RFC] suspend: Block suspend when wakeups are in-progress John Stultz
2011-09-26 19:13 ` [PATCH 2/6] [RFC] sched: Add support for SCHED_STAYAWAKE flag John Stultz
2011-09-26 19:13 ` [PATCH 3/6] [RFC] rtc: rtc-cmos: Add pm_stay_awake/pm_relax calls around IRQ John Stultz
2011-10-01 21:31   ` NeilBrown
2011-09-26 19:13 ` [PATCH 4/6] [RFC] rtc: interface: Add pm_stay_awake/pm_relax chaining rtc workqueue processing John Stultz
2011-09-26 19:13 ` [PATCH 5/6] [RFC] alarmtimer: Add pm_stay_awake /pm_relax calls John Stultz
2011-09-26 19:13 ` [PATCH 6/6] [RFC] alarmtimer: Deboost on nanosleep John Stultz
2011-09-26 20:16 ` [PATCH 0/6] [RFC] Proposal for optimistic suspend idea Peter Zijlstra
2011-09-26 22:27   ` John Stultz
2011-09-27 10:37     ` Peter Zijlstra
2011-09-27 22:56       ` John Stultz
2011-09-28  7:51         ` Peter Zijlstra
2011-09-28  7:57           ` Richard Cochran
2011-09-28  8:02             ` Peter Zijlstra
2011-09-28  8:19         ` Peter Zijlstra
2011-09-29  3:07           ` John Stultz
2011-09-28  8:19         ` Peter Zijlstra
2011-09-29  3:27           ` John Stultz
2011-09-28  8:40         ` Peter Zijlstra
2011-09-28  8:59         ` Peter Zijlstra
2011-09-29  3:45           ` John Stultz [this message]
2011-09-28  9:16         ` Peter Zijlstra
2011-09-28 10:45           ` Borislav Petkov
2011-09-28 21:02         ` Rafael J. Wysocki
2011-09-28  0:09     ` Thomas Gleixner
2011-09-28  1:19       ` John Stultz
2011-09-28  8:18         ` Thomas Gleixner

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=1317267925.3112.810.camel@work-vm \
    --to=john.stultz@linaro.org \
    --cc=Dmitry.Fink@palm.com \
    --cc=amit.kucheria@linaro.org \
    --cc=arve@android.com \
    --cc=damm@opensource.se \
    --cc=farrowg@sg.ibm.com \
    --cc=khilman@ti.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=markgross@thegnar.org \
    --cc=mjg@redhat.com \
    --cc=peterz@infradead.org \
    --cc=rjw@sisk.pl \
    --cc=stern@rowland.harvard.edu \
    --cc=tglx@linutronix.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;
as well as URLs for NNTP newsgroup(s).