mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjan@infradead.org>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org, lenb@kernel.org, mingo@elte.hu,
	yanmin_zhang@linux.intel.com, jens.axboe@oracle.com,
	ink@jurassic.park.msu.ru, Corrado Zoccolo <czoccolo@gmail.com>
Subject: Re: [PATCH v2] cpuidle: Fix the menu governor to boost IO performance
Date: Wed, 16 Sep 2009 06:35:20 +0200	[thread overview]
Message-ID: <20090916063520.675f64c6@infradead.org> (raw)
In-Reply-To: <20090915162306.66c5ce9d.akpm@linux-foundation.org>

On Tue, 15 Sep 2009 16:23:06 -0700
Andrew Morton <akpm@linux-foundation.org> wrote:

> On Tue, 15 Sep 2009 05:42:59 +0200
> Arjan van de Ven <arjan@infradead.org> wrote:
> 
> > Fix the menu idle governor which balances power savings, energy
> > efficiency and performance impact.
> 
> This patch clashes a bit with
> cpuidle-menu-governor-reduce-latency-on-exit.patch (which was sent to
> the ACPI maintainers a month ago and ignored).
> 
> <im-fed-up-with-this-crap-ill-just-merge-it-and-if-it-breaks-dont-blame-me>
> 
> 
> I restaged cpuidle-menu-governor-reduce-latency-on-exit.patch so it
> goes after
> cpuidle-fix-the-menu-governor-to-boost-io-performance.patch. perhaps
> you could take a look at Corrado's change?

ok this patch is an interesting approach, I'll need to check if it's
valid to do this in the cpuidle framework (need to check if "what was
my last residency" stays valid until all the way deep into the next
idle, I kinda fear it won't).

As for the amount of gain.. yeah we're talking about maybe 100 cycles
on a .. say 200 usec latency. But it's a neat optimization if it can
work.


-- 
Arjan van de Ven 	Intel Open Source Technology Centre
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org

  reply	other threads:[~2009-09-16  4:35 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-15  3:42 Arjan van de Ven
2009-09-15  3:54 ` Andrew Morton
2009-09-15  4:15   ` Arjan van de Ven
2009-09-15  4:18     ` Arjan van de Ven
2009-09-15  5:03 ` Rik van Riel
2009-09-15 23:23 ` Andrew Morton
2009-09-16  4:35   ` Arjan van de Ven [this message]
2009-09-17 13:39   ` Arjan van de Ven
2009-11-04  9:39 ` Corrado Zoccolo
2009-11-08 20:40   ` Arjan van de Ven
2009-11-08 21:59     ` Corrado Zoccolo
2009-11-08 22:30       ` Arjan van de Ven
2009-11-09 13:29         ` Corrado Zoccolo
2009-11-08 22:39       ` Arjan van de Ven

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=20090916063520.675f64c6@infradead.org \
    --to=arjan@infradead.org \
    --cc=akpm@linux-foundation.org \
    --cc=czoccolo@gmail.com \
    --cc=ink@jurassic.park.msu.ru \
    --cc=jens.axboe@oracle.com \
    --cc=lenb@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=yanmin_zhang@linux.intel.com \
    /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®