From: "Rafael J. Wysocki" <rjw-KKrjLPT3xs0@public.gmane.org>
To: Alan Stern <stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org>
Cc: Eric Sesterhenn <snakebyte-Mmb7MZpHnFY@public.gmane.org>,
linux-pm-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
Kernel Testers
<kernel-testers-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
vandrove-hnqZr3NxcozrBKCeMvbIDA@public.gmane.org
Subject: Re: [linux-pm] Suspend regression, hang after matroxfb
Date: Sat, 31 Jan 2009 00:40:59 +0100 [thread overview]
Message-ID: <200901310041.00468.rjw@sisk.pl> (raw)
In-Reply-To: <Pine.LNX.4.44L0.0901301818400.2785-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
On Saturday 31 January 2009, Alan Stern wrote:
> On Fri, 30 Jan 2009, Rafael J. Wysocki wrote:
>
> > Suspend to RAM is reported to break on some machines as a result of
> > attempting to put one of driverless PCI devices into a low power
> > state. Avoid that by not attepmting to power manage driverless
> > devices during suspend.
>
> Just out of curiosity, what would happen if one of these troublesome
> devices _did_ have a driver? Would suspend still be broken?
That depends on what the driver would do to the device in its suspend callback.
> Would you then blame the driver instead of the PCI core?
Probably both. I'd try to find the root cause and presumably blacklist
the device this way or another.
> Or would the driver be smart enough to avoid putting the device in a
> low-power state?
I think that putting devices into low power states should better be done by
the core. In this particular case, we can add some quirks to the core so
that it can handle those devices correctly. I don't think adding such quirks
to drivers would be a good idea.
> But doesn't that defeat the purpose of suspending in the first place?
If the entire system goes into a sleep state, the platform will probably remove
power from the device anyway, so it doesn't really matter too much. It is
going to matter for run-time power saving, though.
Thanks,
Rafael
next prev parent reply other threads:[~2009-01-30 23:40 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20090129125855.GA4597@alice>
2009-01-29 22:12 ` [linux-pm] Suspend regression, hang after matroxfb Rafael J. Wysocki
[not found] ` <200901292312.44884.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 12:46 ` Eric Sesterhenn
2009-01-30 21:15 ` Rafael J. Wysocki
[not found] ` <200901302215.28057.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 22:03 ` Rafael J. Wysocki
[not found] ` <200901302303.23143.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 22:09 ` Eric Sesterhenn
2009-01-30 22:06 ` Eric Sesterhenn
2009-01-30 22:27 ` Rafael J. Wysocki
[not found] ` <200901302327.50887.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 22:31 ` Eric Sesterhenn
2009-01-30 22:59 ` Rafael J. Wysocki
[not found] ` <200901302359.18309.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 23:21 ` Alan Stern
[not found] ` <Pine.LNX.4.44L0.0901301818400.2785-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2009-01-30 23:40 ` Rafael J. Wysocki [this message]
2009-01-30 23:36 ` Eric Sesterhenn
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=200901310041.00468.rjw@sisk.pl \
--to=rjw-kkrjlpt3xs0@public.gmane.org \
--cc=kernel-testers-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-pm-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
--cc=snakebyte-Mmb7MZpHnFY@public.gmane.org \
--cc=stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org \
--cc=vandrove-hnqZr3NxcozrBKCeMvbIDA@public.gmane.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;
as well as URLs for NNTP newsgroup(s).