Re: [PATCH 0/5][RFC] Use per cpuidle_device states

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 03, 2012 at 01:05:11PM +0200, Peter De Schrijver wrote:
> On Fri, Aug 31, 2012 at 11:19:15PM +0200, Daniel Lezcano wrote:
> > On 07/27/2012 01:28 PM, Peter De Schrijver wrote:
> > > On Wed, Jul 25, 2012 at 12:45:57PM +0200, Daniel Lezcano wrote:
> > >> Since commit 46bcfad7a819bd17ac4e831b04405152d59784ab,
> > >>         cpuidle: Single/Global registration of idle states
> > >>
> > >> we have a single registration for the cpuidle states which makes
> > >> sense. But now two new architectures are coming: tegra3 and big.LITTLE.
> > >>
> > >> These architectures have different cpus with different caracteristics
> > >> for power saving. High load => powerfull processors, idle => small processors.
> > >>
> > >> That implies different cpu latencies.
> > >>
> > >> This patchset present a simple way to keep the current behavior as introduced
> > >> by Deepthi without breaking the drivers and add the possibility to specify a
> > >> per cpu states.
> > >>
> > > Looks good! I will do some tests with this next week.
> > 
> > Hi Peter,
> > 
> > shall I consider adding your acked-by ?
> > 
> 
> You can. I don't see any problem with the code as such (besides the race).
> 
> > Did you have time to test the patchset ?
> > 
> 
> I haven't had time no :( I was on holiday most of august and had some other
> things to take care of last week :(. Working on it now.
> 

I did some preliminary testing, besides the issues I posted earlier. It looks
all good here.

Cheers,

Peter.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux