From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Theodore Tso <tytso@mit.edu>
Cc: Pavel Machek <pavel@suse.cz>, David Miller <davem@davemloft.net>,
alan@lxorguk.ukuu.org.uk, mchan@broadcom.com,
dwmw2@infradead.org, bastian@waldi.eu.org,
linux-kernel@vger.kernel.org
Subject: Re: request_firmware vs. resume (was Re: [PATCH] bnx2 - use request_firmware())
Date: Thu, 10 Jul 2008 00:33:10 +0200 [thread overview]
Message-ID: <200807100033.11120.rjw@sisk.pl> (raw)
In-Reply-To: <20080709222347.GB10976@mit.edu>
On Thursday, 10 of July 2008, Theodore Tso wrote:
> On Wed, Jul 09, 2008 at 11:58:31PM +0200, Rafael J. Wysocki wrote:
> > > Translation: so much for saving "non-swappable kernel memory".
> > > Unless, of course we add a pre-suspend hook. (Which doesn't exist
> > > yet, AFAICT from a quick perusal of Documentation/power/devices.txt.)
> >
> > No, it doesn't.
> >
> > You will be able to register a suspend notifier instead, but this requires
> > one fix which is in the works.
>
> Will a suspend notifier be able to block the suspend until userspace
> gets back to the device driver calling request_firmware()?
I think so.
To be exact, I'm going to put the disabling of usermode helpers after the
invocation of the suspend notifiers and that will wait for all of the already
running helpers to finish.
Thanks,
Rafael
next prev parent reply other threads:[~2008-07-09 22:31 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20080704225415.GA557@wavehammer.waldi.eu.org>
2008-07-05 9:44 ` [PATCH] bnx2 - use request_firmware() David Woodhouse
2008-07-07 4:21 ` Michael Chan
2008-07-07 7:53 ` Bastian Blank
2008-07-07 9:03 ` David Woodhouse
2008-07-07 18:56 ` Michael Chan
2008-07-07 21:38 ` David Miller
2008-07-07 21:19 ` Alan Cox
2008-07-07 22:05 ` David Miller
2008-07-08 6:39 ` Alan Cox
2008-07-08 8:58 ` David Miller
2008-07-09 20:25 ` request_firmware vs. resume (was Re: [PATCH] bnx2 - use request_firmware()) Pavel Machek
2008-07-09 21:13 ` Rafael J. Wysocki
2008-07-09 21:20 ` Theodore Tso
2008-07-09 21:58 ` Rafael J. Wysocki
2008-07-09 22:23 ` Theodore Tso
2008-07-09 22:33 ` Rafael J. Wysocki [this message]
2008-07-07 22:08 ` [PATCH] bnx2 - use request_firmware() Jeff Garzik
2008-07-07 23:01 ` David Miller
2008-07-08 6:41 ` Alan Cox
2008-07-08 9:00 ` David Miller
2008-07-08 3:30 ` david
2008-07-08 6:49 ` Alan Cox
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=200807100033.11120.rjw@sisk.pl \
--to=rjw@sisk.pl \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=bastian@waldi.eu.org \
--cc=davem@davemloft.net \
--cc=dwmw2@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mchan@broadcom.com \
--cc=pavel@suse.cz \
--cc=tytso@mit.edu \
/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