From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: David Brownell <david-b@pacbell.net>
Cc: linux-pm@lists.osdl.org
Subject: Re: RFC -- updated Documentation/power/devices.txt
Date: Mon, 24 Jul 2006 11:46:21 +0200 [thread overview]
Message-ID: <200607241146.21709.rjw@sisk.pl> (raw)
In-Reply-To: <200607232022.24854.david-b@pacbell.net>
On Monday 24 July 2006 05:22, David Brownell wrote:
> On Sunday 23 July 2006 3:45 pm, Rafael J. Wysocki wrote:
> > On Sunday 23 July 2006 15:03, Alan Stern wrote:
> > > On Sun, 23 Jul 2006, Rafael J. Wysocki wrote:
> > > ... In other words, devices get restored to the state
> > > they were in before suspend.
> >
> > Yes.
> >
> > > I don't see anything wrong with that. It does raise some more questions.
> > >
> > > Who should be responsible for remembering the device's state
> > > from before the system suspend: the PM core or the driver?
> > > (Note that the driver has a better idea of what that state
> > > actually was.)
> >
> > It seems to be easier to leave it to the driver.
>
> Just the way any true runtime pm operation would be, yes?
I think so.
> > However, I think the core
> > needs to have at least some control over it. Otherwise, for example,
> > the detection of misbehaving drivers would be quite difficult, IMHO.
>
> Remember that the definition of runtime PM under discussion is:
>
> > > > 2 While the system is running, independently of other power management
> > > > activity. Upstream drivers will normally not know (or care) if the
> > > > device is in some low power state when issuing requests; the driver
> > > > will auto-resume anything that's needed when it gets a request.
>
> That excludes the core having any such "control", or the ability to "detect"
> such misbehavior. (And why would it be misbehavior, when the difference is
> irrelevant to upstream drivers issuing requests to that device?) No matter
> what state the device resumes into, the only architectural requirement is
> that it respond as if it were fully "on". Because runtime low power states
> are exactly equivalent to "fully operational" except they use less power.
>
> > > Upon resuming from a system suspend, is it better to tell the
> > > driver to "go directly to the saved state" or to turn the
> > > device back fully ON and then change to the saved state (if it
> > > wasn't fully ON before the system suspend)?
> >
> > I think the rule should be "go directly to the saved state", if possible.
>
> Agreed, though as I described above it's not something that can be
> detected outside of that driver without actually measuring the power
> drain that driver imposes on the system.
And that would be difficult if not impossible.
So, it looks like the core will only need to tell the driver to "resume"
with the implicit "go directly to the saved state" meaning and the
driver will actually decide what to do.
Greetings,
Rafael
next prev parent reply other threads:[~2006-07-24 9:46 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-10 22:25 RFC -- updated Documentation/power/devices.txt David Brownell
2006-07-11 5:56 ` Andrew Morton
2006-07-11 16:38 ` David Brownell
2006-07-11 21:57 ` David Brownell
2006-07-12 12:25 ` Pavel Machek
2006-07-12 14:04 ` Alan Stern
2006-07-12 15:45 ` David Brownell
2006-07-12 16:03 ` Alan Stern
2006-07-23 1:37 ` David Brownell
2006-07-23 3:59 ` Alan Stern
2006-07-23 10:50 ` Rafael J. Wysocki
2006-07-23 13:03 ` Alan Stern
2006-07-23 22:45 ` Rafael J. Wysocki
2006-07-24 3:22 ` David Brownell
2006-07-24 9:46 ` Rafael J. Wysocki [this message]
2006-07-24 14:51 ` Alan Stern
2006-07-24 15:15 ` David Brownell
2006-07-24 15:42 ` Alan Stern
2006-07-24 17:11 ` David Brownell
2006-07-24 20:44 ` Alan Stern
2006-07-24 21:19 ` David Brownell
2006-07-25 15:42 ` Alan Stern
2006-08-10 23:38 ` [patch 2.6.18-rc] " David Brownell
2006-07-23 16:22 ` RFC -- " David Brownell
2006-07-11 14:40 ` Pavel Machek
2006-07-11 21:28 ` Pavel Machek
-- strict thread matches above, loose matches on Subject: below --
2006-07-11 7:56 Woodruff, Richard
2006-07-11 16:51 ` 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=200607241146.21709.rjw@sisk.pl \
--to=rjw@sisk.pl \
--cc=david-b@pacbell.net \
--cc=linux-pm@lists.osdl.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