From: Magnus Damm <magnus.damm@gmail.com>
To: Arjan van de Ven <arjan@infradead.org>
Cc: Alan Stern <stern@rowland.harvard.edu>,
"Rafael J. Wysocki" <rjw@sisk.pl>, Greg KH <gregkh@suse.de>,
LKML <linux-kernel@vger.kernel.org>,
ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
Linux-pm mailing list <linux-pm@lists.linux-foundation.org>,
Ingo Molnar <mingo@elte.hu>
Subject: Re: [linux-pm] [patch update 2 fix] PM: Introduce core framework for run-time PM of I/O devices
Date: Mon, 22 Jun 2009 16:27:29 +0900 [thread overview]
Message-ID: <aec7e5c30906220027v63c88e8aj32950a8d84558cce@mail.gmail.com> (raw)
In-Reply-To: <20090621234324.06fda564@infradead.org>
On Mon, Jun 22, 2009 at 3:43 PM, Arjan van de Ven<arjan@infradead.org> wrote:
> On Mon, 22 Jun 2009 15:20:43 +0900
> Magnus Damm <magnus.damm@gmail.com> wrote:
>
>> On Sat, Jun 20, 2009 at 11:30 PM, Alan
>> Stern<stern@rowland.harvard.edu> wrote:
>> > Some more thoughts...
>> >
>> > Magnus, you might have some insights here. It occurred to me that
>> > some devices can switch power levels very quickly, and the drivers
>> > might therefore want the runtime suspend and resume methods to be
>> > called as soon as possible, even in interrupt context.
>>
>> I'd like to call pm_request_suspend() from interrupt context. I don't
>
> there are some really strong reasons to at least be able to call the
> resume function from an interrupt handler.... shared interrupts are one
> of them.
I suppose you mean that you need to resume the hardware device before
you can check if it has a pending interrupt source? If so then you
also mean that suspended hardware devices may generate interrupts, no?
My plan for SuperH SoC is that runtime suspend always stops the clock,
but register save and power off may happen. Regardless, interrupts are
not generated from suspended hardware devices.
In the rare case of shared interrupts on SuperH we can just skip over
the platform devices that have been runtime suspended since they will
not have generated any interrupt.
But yes, if there is hardware than can generate interrupts from
suspended state then we need to resume from interrupt context. This
probably means that the entire bus hierarchy must resume from
interrupt context as well.
/ magnus
next prev parent reply other threads:[~2009-06-22 7:33 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-13 22:23 [PATCH] " Rafael J. Wysocki
2009-06-14 9:41 ` Magnus Damm
2009-06-14 10:29 ` Rafael J. Wysocki
2009-06-14 9:58 ` [linux-pm] " Rafael J. Wysocki
2009-06-14 22:57 ` [patch update] " Rafael J. Wysocki
2009-06-14 23:18 ` Arjan van de Ven
2009-06-15 20:02 ` Rafael J. Wysocki
2009-06-15 21:08 ` Alan Stern
2009-06-15 23:21 ` Rafael J. Wysocki
2009-06-16 14:30 ` Alan Stern
2009-06-16 21:30 ` [patch update 2] " Rafael J. Wysocki
2009-06-16 22:33 ` [patch update 2 fix] " Rafael J. Wysocki
2009-06-17 20:08 ` Alan Stern
2009-06-17 23:07 ` Rafael J. Wysocki
2009-06-18 18:17 ` Alan Stern
2009-06-19 0:38 ` Rafael J. Wysocki
2009-06-19 16:25 ` Alan Stern
2009-06-19 22:42 ` Rafael J. Wysocki
2009-06-20 2:34 ` Alan Stern
2009-06-20 14:30 ` [linux-pm] " Alan Stern
2009-06-20 23:48 ` Rafael J. Wysocki
2009-06-21 2:30 ` Alan Stern
2009-06-21 11:32 ` Rafael J. Wysocki
2009-06-22 14:16 ` Alan Stern
2009-06-22 15:27 ` Rafael J. Wysocki
2009-06-22 15:39 ` Alan Stern
2009-06-22 15:53 ` Rafael J. Wysocki
2009-06-22 6:20 ` Magnus Damm
2009-06-22 6:43 ` Arjan van de Ven
2009-06-22 7:27 ` Magnus Damm [this message]
2009-06-22 13:49 ` Arjan van de Ven
2009-06-22 15:39 ` Rafael J. Wysocki
2009-06-22 15:33 ` Rafael J. Wysocki
2009-06-22 8:15 ` Oliver Neukum
2009-06-20 23:38 ` [patch update 3] " Rafael J. Wysocki
2009-06-21 2:23 ` Alan Stern
2009-06-21 12:46 ` Rafael J. Wysocki
2009-06-22 15:01 ` Alan Stern
2009-06-22 15:49 ` Rafael J. Wysocki
2009-06-22 16:28 ` Alan Stern
2009-06-22 23:02 ` Rafael J. Wysocki
2009-06-23 17:02 ` Alan Stern
2009-06-23 17:45 ` Rafael J. Wysocki
2009-06-23 18:26 ` Alan Stern
2009-06-24 0:17 ` Rafael J. Wysocki
2009-06-24 14:51 ` Alan Stern
2009-06-24 19:14 ` Rafael J. Wysocki
2009-06-24 20:19 ` Alan Stern
2009-06-24 21:23 ` Rafael J. Wysocki
2009-06-24 15:04 ` [patch update] " Pavel Machek
2009-06-27 21:52 ` Rafael J. Wysocki
2009-07-06 8:28 ` Pavel Machek
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=aec7e5c30906220027v63c88e8aj32950a8d84558cce@mail.gmail.com \
--to=magnus.damm@gmail.com \
--cc=arjan@infradead.org \
--cc=gregkh@suse.de \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=mingo@elte.hu \
--cc=rjw@sisk.pl \
--cc=stern@rowland.harvard.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
all inboxes | Powered by JetHome®