login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2010
»
May
»
27
Re: [PATCH 1/8] PM: Opportunistic suspend support.
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From: Rafael J. Wysocki
Subject:
Re: [PATCH 1/8] PM: Opportunistic suspend support.
Date: Thursday, May 27, 2010 - 1:00 pm
On Thursday 27 May 2010, Dmitry Torokhov wrote:
quoted text
> On Wed, May 26, 2010 at 05:52:40PM -0700, Arve Hjønnevåg wrote: > > 2010/5/26 Alan Stern <stern@rowland.harvard.edu>: > > > On Wed, 26 May 2010, Arve Hjønnevåg wrote: > > > > > >> > I must be missing something. In Arve's patch 1/8, if the system is in > > >> > opportunistic suspend, and a wakeup event occurs but no suspend > > >> > blockers get enabled by the handler, what causes the system to go back > > >> > into suspend after the event is handled? Isn't that a loop of some > > >> > sort? > > >> > > > >> > > >> Yes it is a loop. I think what you are missing is that it only loops > > >> repeatedly if the driver that aborts suspend does not use a suspend > > >> blocker. > > > > > > You mean "the driver that handles the wakeup event". I was asking what > > > happened if suspend succeeded and then a wakeup occurred. But yes, if > > > a suspend blocker is used then its release causes another suspend > > > attempt, with no looping. > > > > > >> > And even if it isn't, so what? What's wrong with looping behavior? > > >> > > >> It is a significant power drain. > > > > > > Not in the situation I was discussing. > > > > > > > If you meant it spend most of the time suspended, then I agree. It > > only wastes power when a driver blocks suspend by returning an error > > from its suspend hook and we are forced to loop doing no useful work. > > > > If driver refuses to suspend that means there are events that need > processing. I fail to see why it would be called "looping doing no > useful work".
I guess Arve meant the case of events that didn't propagate to user space. Rafael
quoted text
> >
--
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
Re: [PATCH 1/8] PM: Opportunistic suspend support.
, Alan Stern
, (Wed May 26, 5:47 pm)
Re: [PATCH 1/8] PM: Opportunistic suspend support.
, Arve Hjønnevåg
, (Wed May 26, 5:52 pm)
Re: [PATCH 1/8] PM: Opportunistic suspend support.
, Dmitry Torokhov
, (Thu May 27, 11:13 am)
Re: [PATCH 1/8] PM: Opportunistic suspend support.
, Rafael J. Wysocki
, (Thu May 27, 1:00 pm)
Re: [PATCH 1/8] PM: Opportunistic suspend support.
, Arve Hjønnevåg
, (Thu May 27, 4:36 pm)
Re: [PATCH 1/8] PM: Opportunistic suspend support.
, Dmitry Torokhov
, (Thu May 27, 4:48 pm)
Re: [PATCH 1/8] PM: Opportunistic suspend support.
, Arve Hjønnevåg
, (Thu May 27, 4:52 pm)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Jesse Barnes
Re: [stable] [BUG][PATCH] cpqphp: fix kernel NULL pointer dereference
Greg KH
[003/136] p54usb: add Zcomax XG-705A usbid
Magnus Damm
[PATCH 03/07] ARM: Use shared GIC entry macros on Realview
Oliver Neukum
Re: [Bug #13682] The webcam stopped working when upgrading from 2.6.29 to 2.6.30
Martin Schwidefsky
Re: [PATCH] optimized ktime_get[_ts] for GENERIC_TIME=y
git
:
Junio C Hamano
Re: Some advanced index playing
Jeff King
Re: confusion over t