All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjan@linux.intel.com>
To: svaidy@linux.vnet.ibm.com
Cc: Andi Kleen <ak@linux.intel.com>,
	Trinabh Gupta <trinabh@linux.vnet.ibm.com>,
	Venkatesh Pallipadi <venki@google.com>,
	peterz@infradead.org, lenb@kernel.org, suresh.b.siddha@intel.com,
	benh@kernel.crashing.org, linux-kernel@vger.kernel.org
Subject: Re: [RFC V1] cpuidle: add idle routine registration and cleanup pm_idle pointer
Date: Wed, 20 Oct 2010 12:44:19 -0700	[thread overview]
Message-ID: <4CBF4693.1070802@linux.intel.com> (raw)
In-Reply-To: <20101020194045.GB706@dirshya.in.ibm.com>

  On 10/20/2010 12:40 PM, Vaid
>> you ALWAYS have at least 2 idle handling states. The platform idle
>> one and the generic busy waiting one.
>> the later is needed for "I want absolutely 0 latency" cases.
> Some special overrides like idle=poll should handle this case even if
> cpuidle and related registration mechanism is compiled out. The point
> is that we need some flexibility even if the full framework is not
> included.

this is not idle=poll

this is an (privileged) app or driver, at runtime, requesting a 0 usec 
max latency for a short or long period of time.


>>> Making current cpuidle as default in kernel
>> not "in the kernel" but "for x86".
>> You're solving an x86 problem here, right?
>> (the pm_idle is an x86 only problem. other architectures should be
>> able to keep doing what they are doing)
>> For x86, lets solve it by going to cpuidle period... and if Andi can
>> find some bloat in cpuidle, lets see if the fat can be trimmed.
> Ok, you are suggesting that for x86 lets move cpuidle in kernel
> always, while it can be an optional module for other archs as it
> stands today.  We can slim down the cpuidle from current 7K or atleast
> split some parts like governors as modules if needed.

governors as modules is a total pain. modules don't solve the problem. 
really. it's still code you need.
we have two governors today, menu and ladder
menu is best on anything that is tickless
ladder is useless on any tickless kernel, and likely not better than 
menu on non-tickless.
that's it.
It will be good to have other archs also follow the same cpuidle
> framework and call it from their kernel/idle routines so that we need
> not have a hierarchy of idle routines there.

yes. but we don't have to force that to happen at the exact same time.


  reply	other threads:[~2010-10-20 19:44 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-19 18:36 [RFC V1] cpuidle: add idle routine registration and cleanup pm_idle pointer Trinabh Gupta
2010-10-19 18:38 ` Arjan van de Ven
2010-10-19 18:49   ` Venkatesh Pallipadi
2010-10-19 19:01     ` Trinabh Gupta
2010-10-20 15:12       ` Trinabh Gupta
2010-10-20 15:18         ` Arjan van de Ven
2010-10-20 15:34           ` Andi Kleen
2010-10-20 16:03             ` Arjan van de Ven
2010-10-20 19:19               ` Vaidyanathan Srinivasan
2010-10-20 19:25                 ` Arjan van de Ven
2010-10-20 19:28                   ` Peter Zijlstra
2010-10-20 19:29                     ` Arjan van de Ven
2010-10-20 19:40                   ` Vaidyanathan Srinivasan
2010-10-20 19:44                     ` Arjan van de Ven [this message]
2010-10-20 19:47                       ` Venkatesh Pallipadi
2010-10-20 20:03                         ` Vaidyanathan Srinivasan
2010-10-20 20:47                         ` Arjan van de Ven
2010-10-20 21:19                           ` Venkatesh Pallipadi
2010-10-20 20:55               ` Dipankar Sarma
2010-10-20 15:57           ` Trinabh Gupta

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=4CBF4693.1070802@linux.intel.com \
    --to=arjan@linux.intel.com \
    --cc=ak@linux.intel.com \
    --cc=benh@kernel.crashing.org \
    --cc=lenb@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=peterz@infradead.org \
    --cc=suresh.b.siddha@intel.com \
    --cc=svaidy@linux.vnet.ibm.com \
    --cc=trinabh@linux.vnet.ibm.com \
    --cc=venki@google.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 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.