From: Andi Kleen <ak@suse.de>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: linux-kernel@vger.kernel.org, len.brown@intel.com
Subject: Re: [RFC] maximum latency tracking infrastructure
Date: 24 Aug 2006 23:50:27 +0200 [thread overview]
Message-ID: <p73r6z5erbw.fsf@verdi.suse.de> (raw)
In-Reply-To: <200608241429.45791.jbarnes@virtuousgeek.org>
Jesse Barnes <jbarnes@virtuousgeek.org> writes:
> On Thursday, August 24, 2006 2:20 pm, Arjan van de Ven wrote:
> > Jesse Barnes wrote:
> > > On Thursday, August 24, 2006 10:41 am, Arjan van de Ven wrote:
> > >> The reason for adding this infrastructure is that power management
> > >> in the idle loop needs to make a tradeoff between latency and power
> > >> savings (deeper power save modes have a longer latency to running
> > >> code again).
> > >
> > > What if a processor was already in a sleep state when a call to
> > > set_acceptable_latency() latency occurs?
> >
> > there's nothing sane that can be done in that case; any wake up
> > already will cause the unwanted latency! A premature wakeup is only
> > making it happen *now*, but now is as inconvenient a time as any...
> > (in fact it may be a worst case time scenario, say, an audio
> > interrupt...)
>
> Depends on what's going on. What if you have a two socket machine, and
> one CPU is in C3 when the latency setting occurs?
I didn't think there were currently any multi socket machines with C3
support? The best you get is dual core.
> Shouldn't you wake it
> up and prevent it from going that deep again? But you're right, you
> won't necessarily improve anything...
Generally there are so many events that wake up CPUs that the case is pretty
academic -- all CPUs will eventually wake up in a reasonable time
(before your driver initialization finished likely) and then follow
the new latency settings.
Maybe at some point if all the idle breaking events in Linux have been
fixed up it might be a problem, but I think that's a long time off.
-Andi
next prev parent reply other threads:[~2006-08-24 21:50 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-24 17:41 Arjan van de Ven
2006-08-24 19:18 ` Len Brown
2006-08-24 21:08 ` Jesse Barnes
2006-08-24 21:20 ` Arjan van de Ven
2006-08-24 21:29 ` Jesse Barnes
2006-08-24 21:50 ` Andi Kleen [this message]
2006-08-25 4:54 ` Nick Piggin
2006-08-25 7:58 ` Arjan van de Ven
2006-08-25 8:26 ` Nick Piggin
2006-08-25 8:30 ` Arjan van de Ven
2006-08-24 21:52 ` Daniel Walker
2006-08-24 21:57 ` Arjan van de Ven
2006-08-24 22:16 ` Daniel Walker
2006-08-24 22:24 ` Matthew Garrett
2006-08-25 8:19 ` Arjan van de Ven
2006-08-24 22:52 ` Matt Mackall
2006-08-25 7:56 ` Arjan van de Ven
2006-08-25 14:54 ` Matt Mackall
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=p73r6z5erbw.fsf@verdi.suse.de \
--to=ak@suse.de \
--cc=jbarnes@virtuousgeek.org \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome