From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Oliver Neukum <oliver@neukum.org>
Cc: linux-pm@lists.linux-foundation.org,
Alan Stern <stern@rowland.harvard.edu>,
Partha Basak <p-basak2@ti.com>,
linux-omap@vger.kernel.org
Subject: Re: [linux-pm] [PATCH ver. 3] PM: add synchronous runtime interface for interrupt handlers
Date: Thu, 25 Nov 2010 21:03:29 +0100 [thread overview]
Message-ID: <201011252103.29401.rjw@sisk.pl> (raw)
In-Reply-To: <201011251958.49249.oliver@neukum.org>
On Thursday, November 25, 2010, Oliver Neukum wrote:
> Am Donnerstag, 25. November 2010, 16:52:39 schrieb Alan Stern:
> > When a device is declared irq-safe in this way, the PM core increments
> > the parent's usage count, so the parent will never be runtime
> > suspended. This prevents difficult situations in which an irq-safe
> > device can't resume because it is forced to wait for its non-irq-safe
> > parent.
>
> Shouldn't you walk further up the tree if the parent itself is irq-safe?
> It seems like a waste of power to not suspend a paren, that can be
> woken in irq.
Then we should walk up the tree if the parent's parent is irq-safe and so
on, which would make us spend uncertain amount of time in the ISR (or generally
with interrupts off). I think it's safer not to do that.
Thanks,
Rafael
next prev parent reply other threads:[~2010-11-25 20:04 UTC|newest]
Thread overview: 144+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-24 0:05 runtime_pm_get_sync() from ISR with IRQs disabled? Kevin Hilman
2010-09-24 15:13 ` [linux-pm] " Alan Stern
2010-09-24 18:54 ` Kevin Hilman
2010-09-24 20:04 ` Rafael J. Wysocki
2010-09-24 20:04 ` [linux-pm] " Rafael J. Wysocki
2010-09-27 13:57 ` Alan Stern
2010-09-27 20:00 ` Rafael J. Wysocki
2010-09-27 20:39 ` Alan Stern
2010-09-27 21:09 ` Rafael J. Wysocki
2010-09-28 14:55 ` Alan Stern
2010-09-28 18:19 ` Rafael J. Wysocki
2010-09-28 18:19 ` [linux-pm] " Rafael J. Wysocki
2010-09-30 18:25 ` [PATCH] PM: add synchronous runtime interface for interrupt handlers Alan Stern
2010-09-30 20:15 ` Rafael J. Wysocki
2010-09-30 21:42 ` Alan Stern
2010-09-30 22:41 ` Rafael J. Wysocki
2010-10-01 14:28 ` Alan Stern
2010-10-01 21:23 ` Rafael J. Wysocki
2010-10-02 14:12 ` Alan Stern
2010-10-02 22:06 ` Rafael J. Wysocki
2010-10-02 22:06 ` Rafael J. Wysocki
2010-10-03 15:52 ` Alan Stern
2010-10-03 20:33 ` Rafael J. Wysocki
2010-10-03 20:33 ` Rafael J. Wysocki
2010-10-03 15:52 ` Alan Stern
2010-10-08 23:24 ` PATCH: PM / Runtime: Remove idle notification after failing suspend (was: Re: [PATCH] PM: add synchronous ...) Rafael J. Wysocki
2010-10-08 23:24 ` PATCH: PM / Runtime: Remove idle notification after failing suspend (was: Re: [linux-pm] " Rafael J. Wysocki
2010-10-10 20:18 ` Alan Stern
2010-10-10 20:18 ` PATCH: PM / Runtime: Remove idle notification after failing suspend (was: " Alan Stern
2010-10-05 21:44 ` [PATCH] PM: add synchronous runtime interface for interrupt handlers Kevin Hilman
2010-10-05 21:44 ` Kevin Hilman
2010-10-06 15:58 ` Alan Stern
2010-10-06 19:33 ` Rafael J. Wysocki
2010-10-06 19:33 ` Rafael J. Wysocki
2010-10-06 19:35 ` Kevin Hilman
2010-10-06 20:28 ` Alan Stern
2010-10-06 20:28 ` Alan Stern
2010-10-06 21:47 ` Rafael J. Wysocki
2010-10-07 15:26 ` Alan Stern
2010-10-07 15:26 ` Alan Stern
2010-10-07 16:52 ` Kevin Hilman
2010-10-07 17:35 ` Alan Stern
2010-10-07 21:11 ` Rafael J. Wysocki
2010-10-07 23:15 ` Kevin Hilman
2010-10-07 23:37 ` Rafael J. Wysocki
2010-10-07 23:37 ` Rafael J. Wysocki
2010-10-07 23:55 ` Kevin Hilman
2010-10-07 23:55 ` Kevin Hilman
2010-10-08 16:22 ` Alan Stern
2010-10-08 16:22 ` Alan Stern
2010-10-08 21:04 ` Kevin Hilman
2010-10-08 21:04 ` Kevin Hilman
2010-10-08 19:57 ` Rafael J. Wysocki
2010-10-08 19:57 ` Rafael J. Wysocki
2010-10-07 23:15 ` Kevin Hilman
2010-10-08 16:18 ` Alan Stern
2010-10-08 19:53 ` Rafael J. Wysocki
2010-10-09 11:09 ` Rafael J. Wysocki
2010-10-09 11:09 ` [linux-pm] " Rafael J. Wysocki
2010-10-11 17:00 ` Alan Stern
2010-10-11 17:00 ` [linux-pm] " Alan Stern
2010-10-11 22:30 ` Rafael J. Wysocki
2010-10-11 22:30 ` Rafael J. Wysocki
2010-10-08 19:53 ` Rafael J. Wysocki
2010-10-08 16:18 ` Alan Stern
2010-10-07 21:11 ` Rafael J. Wysocki
2010-10-07 17:35 ` Alan Stern
2010-10-07 16:52 ` Kevin Hilman
2010-11-19 15:45 ` [PATCH ver. 2] " Alan Stern
2010-11-20 12:56 ` Rafael J. Wysocki
2010-11-20 12:56 ` Rafael J. Wysocki
2010-11-20 16:59 ` Alan Stern
2010-11-20 16:59 ` Alan Stern
2010-11-20 19:41 ` [linux-pm] " Alan Stern
2010-11-21 23:45 ` Rafael J. Wysocki
2010-11-21 23:45 ` [linux-pm] " Rafael J. Wysocki
2010-11-20 19:41 ` Alan Stern
2010-11-21 23:41 ` Rafael J. Wysocki
2010-11-21 23:41 ` Rafael J. Wysocki
2010-11-22 15:38 ` Alan Stern
2010-11-22 15:38 ` Alan Stern
2010-11-22 23:01 ` Rafael J. Wysocki
2010-11-23 3:19 ` Alan Stern
2010-11-23 3:19 ` Alan Stern
2010-11-23 22:51 ` Rafael J. Wysocki
2010-11-23 22:51 ` Rafael J. Wysocki
2010-11-24 0:11 ` Kevin Hilman
2010-11-24 0:11 ` Kevin Hilman
2010-11-24 16:43 ` Alan Stern
2010-11-24 16:43 ` Alan Stern
2010-11-24 18:03 ` Kevin Hilman
2010-11-24 18:03 ` Kevin Hilman
2010-11-24 14:56 ` Alan Stern
2010-11-24 20:33 ` Rafael J. Wysocki
2010-11-24 20:33 ` Rafael J. Wysocki
2010-11-25 15:52 ` [PATCH ver. 3] " Alan Stern
2010-11-25 15:52 ` Alan Stern
2010-11-25 18:58 ` Oliver Neukum
2010-11-25 18:58 ` [linux-pm] " Oliver Neukum
2010-11-25 20:03 ` Rafael J. Wysocki
2010-11-25 20:03 ` Rafael J. Wysocki [this message]
2010-11-26 22:23 ` Rafael J. Wysocki
2010-11-26 22:23 ` Rafael J. Wysocki
2010-11-24 14:56 ` [PATCH ver. 2] " Alan Stern
2010-11-22 23:01 ` Rafael J. Wysocki
2010-11-19 15:45 ` Alan Stern
2011-04-11 15:47 ` Sylwester Nawrocki
2011-04-11 16:08 ` Alan Stern
2011-04-11 17:20 ` Sylwester Nawrocki
2011-04-11 18:37 ` Alan Stern
2010-10-06 21:47 ` [PATCH] " Rafael J. Wysocki
2010-10-06 23:51 ` Kevin Hilman
2010-10-06 23:51 ` Kevin Hilman
2010-10-06 19:35 ` Kevin Hilman
2010-10-06 15:58 ` Alan Stern
2010-10-02 14:12 ` Alan Stern
2010-10-01 14:28 ` Alan Stern
2010-09-30 22:41 ` Rafael J. Wysocki
2010-09-30 21:42 ` Alan Stern
2010-09-30 21:42 ` Alan Stern
2010-09-30 20:15 ` Rafael J. Wysocki
2010-09-30 22:02 ` Rafael J. Wysocki
2010-10-01 14:12 ` Alan Stern
2010-10-01 14:12 ` Alan Stern
2010-10-01 21:14 ` Rafael J. Wysocki
2010-10-01 21:14 ` Rafael J. Wysocki
2010-10-01 22:37 ` [PATCH] PM / Runtime: Reduce code duplication in core helper functions (was: Re: [linux-pm] [PATCH] PM: add synchronous ...) Rafael J. Wysocki
2010-10-02 14:15 ` [PATCH] PM / Runtime: Reduce code duplication in core helper functions (was: " Alan Stern
2010-10-02 14:15 ` [PATCH] PM / Runtime: Reduce code duplication in core helper functions (was: Re: [linux-pm] " Alan Stern
2010-10-01 22:37 ` [PATCH] PM / Runtime: Reduce code duplication in core helper functions (was: " Rafael J. Wysocki
2010-09-30 18:25 ` [PATCH] PM: add synchronous runtime interface for interrupt handlers Alan Stern
2010-09-28 14:55 ` runtime_pm_get_sync() from ISR with IRQs disabled? Alan Stern
2010-09-27 21:09 ` Rafael J. Wysocki
2010-09-27 21:11 ` [linux-pm] " Kevin Hilman
2010-09-27 21:11 ` Kevin Hilman
2010-09-27 20:39 ` Alan Stern
2010-09-27 20:00 ` Rafael J. Wysocki
2010-09-27 13:57 ` Alan Stern
2010-09-24 20:27 ` Alan Stern
2010-09-24 20:27 ` [linux-pm] " Alan Stern
2010-09-24 21:52 ` Kevin Hilman
2010-09-24 21:52 ` [linux-pm] " Kevin Hilman
2010-09-24 18:54 ` Kevin Hilman
2010-09-24 15:13 ` Alan Stern
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=201011252103.29401.rjw@sisk.pl \
--to=rjw@sisk.pl \
--cc=linux-omap@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=oliver@neukum.org \
--cc=p-basak2@ti.com \
--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 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.