All of lore.kernel.org
 help / color / mirror / Atom feed
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

  parent reply	other threads:[~2009-01-30 23:40 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-29 12:58 Suspend regression, hang after matroxfb Eric Sesterhenn
2009-01-29 22:12 ` [linux-pm] " Rafael J. Wysocki
2009-01-30 12:46   ` Eric Sesterhenn
     [not found]   ` <200901292312.44884.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 12:46     ` [linux-pm] " Eric Sesterhenn
2009-01-30 21:15       ` Rafael J. Wysocki
2009-01-30 21:15       ` [linux-pm] " Rafael J. Wysocki
2009-01-30 22:03         ` Rafael J. Wysocki
     [not found]         ` <200901302215.28057.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 22:03           ` [linux-pm] " Rafael J. Wysocki
     [not found]             ` <200901302303.23143.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 22:09               ` Eric Sesterhenn
2009-01-30 22:09             ` Eric Sesterhenn
2009-01-30 22:06           ` [linux-pm] " Eric Sesterhenn
2009-01-30 22:27             ` Rafael J. Wysocki
2009-01-30 22:31               ` Eric Sesterhenn
     [not found]               ` <200901302327.50887.rjw-KKrjLPT3xs0@public.gmane.org>
2009-01-30 22:31                 ` [linux-pm] " Eric Sesterhenn
2009-01-30 22:59                   ` Rafael J. Wysocki
2009-01-30 22:59                   ` [linux-pm] " 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:40                         ` Rafael J. Wysocki
2009-01-30 23:36                       ` [linux-pm] " Eric Sesterhenn
2009-01-30 23:21                     ` Alan Stern
2009-01-30 23:36                     ` Eric Sesterhenn
2009-01-30 22:27             ` Rafael J. Wysocki
2009-01-30 22:06         ` Eric Sesterhenn
2009-01-29 22:12 ` Rafael J. Wysocki

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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.