All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jordan Crouse" <jordan.crouse@amd.com>
To: linux-pm@lists.osdl.org
Subject: Re: Toward runtime power management in Linux
Date: Mon, 1 Aug 2005 09:10:23 -0600	[thread overview]
Message-ID: <20050801151023.GA12097@cosmic.amd.com> (raw)
In-Reply-To: <Pine.LNX.4.44L0.0508010935440.4961-100000@iolanthe.rowland.org>

[-- Attachment #1: Type: text/plain, Size: 1536 bytes --]

> > 	This could potentially make performance-conscious apps "hiccup"
> > once every second as this thread goes walking the list looking for
> > candidates to shut off.  Try to avoid this; if nothing is happening, nothing
> > should be running.
> 
> I don't understand this comment at all.  Lots of things happen 
> periodically in the kernel: threads wake up, timers go off...  Are you 
> suggesting that, for example, the page-flush thread shouldn't wake up 
> from time to time either?

While I don't agree that it will be a horrible drain on performance, I do
see a large potential for abuse with a big kernel thread.  Things like the
page-flush thread are well known and (hopefully) optimized entities -
the RTPM thread will have to depend on hundreds of driver writers to be kind
to not suck time and resources from the system.  About the time that somebody
puts a large udelay into their AC97 driver to turn off the DAC, then I'm sure
we will question our motives in this regard.

That said, I think I tend to favor the big kernel thread, or at least timeout
threads on a bus level.  The single entity handling the idle math timeout
would facilitate future issues such as priority in handling idle timeouts 
(do we address certain buses/devices before others, for example), plus it
would help centralize the functionality, and make it easier to control with
any future power management policy concepts.

Jordan

-- 
Jordan Crouse
Senior Linux Engineer
AMD - Personal Connectivity Solutions Group
<www.amd.com/embeddedprocessors>


[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  reply	other threads:[~2005-08-01 15:10 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-31  2:36 Toward runtime power management in Linux Alan Stern
2005-08-01  2:10 ` Leo L. Schwab
2005-08-01 11:44   ` Amit Kucheria
2005-08-01 14:16     ` Alan Stern
     [not found]     ` <20050802024415.C3518DB57B@adsl-69-107-32-110.dsl.pltn13.pacbell.net>
2005-08-04  8:06       ` Tony Lindgren
2005-08-04 16:02         ` david-b
2005-08-14 19:53         ` Pavel Machek
2005-08-01 14:07   ` Alan Stern
2005-08-01 15:10     ` Jordan Crouse [this message]
2005-08-01 15:23       ` Alan Stern
2005-08-04 17:24     ` Igor Stoppa
     [not found] ` <Pine.LNX.4.50.0508011712220.2764-100000@monsoon.he.net>
2005-08-02 17:45   ` Geoff Levand
2005-08-25  3:12 ` Benjamin Herrenschmidt
2005-08-25 15:27   ` Alan Stern
2005-08-25 21:42     ` Benjamin Herrenschmidt
2005-08-26  2:25       ` Alan Stern
     [not found] <Pine.LNX.4.50.0508012316380.2764-100000@monsoon.he.net>
2005-08-02 14:35 ` Alan Stern
  -- strict thread matches above, loose matches on Subject: below --
2005-08-25 13:59 Brown, Len

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=20050801151023.GA12097@cosmic.amd.com \
    --to=jordan.crouse@amd.com \
    --cc=linux-pm@lists.osdl.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.