Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-03-14 Thread Kevin Hilman
Colin Cross writes: > On Tue, Mar 13, 2012 at 5:28 PM, Colin Cross wrote: >> On Tue, Mar 13, 2012 at 4:52 PM, Kevin Hilman wrote: [...] >>> >>> Checking the ready_count seemed like an easy way to do this, but did you >>> have any other mechanisms in mind for CPUs to communicate that they've >

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-02-02 Thread Colin Cross
On Wed, Feb 1, 2012 at 10:07 AM, Lorenzo Pieralisi wrote: > On Wed, Feb 01, 2012 at 05:30:15PM +, Colin Cross wrote: >> On Wed, Feb 1, 2012 at 6:59 AM, Lorenzo Pieralisi >> wrote: >> > On Wed, Feb 01, 2012 at 12:13:26PM +, Vincent Guittot wrote: >> > >> > [...] >> > >> >> >> In your patch

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-02-01 Thread Lorenzo Pieralisi
On Wed, Feb 01, 2012 at 05:30:15PM +, Colin Cross wrote: > On Wed, Feb 1, 2012 at 6:59 AM, Lorenzo Pieralisi > wrote: > > On Wed, Feb 01, 2012 at 12:13:26PM +, Vincent Guittot wrote: > > > > [...] > > > >> >> In your patch, you put in safe state (WFI for most of platform) the > >> >> cpus

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-02-01 Thread Colin Cross
On Wed, Feb 1, 2012 at 6:59 AM, Lorenzo Pieralisi wrote: > On Wed, Feb 01, 2012 at 12:13:26PM +, Vincent Guittot wrote: > > [...] > >> >> In your patch, you put in safe state (WFI for most of platform) the >> >> cpus that become idle and these cpus are woken up each time a new cpu >> >> of the

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-02-01 Thread Lorenzo Pieralisi
On Wed, Feb 01, 2012 at 12:13:26PM +, Vincent Guittot wrote: [...] > >> In your patch, you put in safe state (WFI for most of platform) the > >> cpus that become idle and these cpus are woken up each time a new cpu > >> of the cluster becomes idle. Then, the cluster state is chosen and the >

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-02-01 Thread Vincent Guittot
Hi Colin, Sorry for this late reply On 27 January 2012 18:32, Colin Cross wrote: > On Fri, Jan 27, 2012 at 12:54 AM, Vincent Guittot > wrote: >> On 20 January 2012 21:40, Colin Cross wrote: >>> On Fri, Jan 20, 2012 at 12:46 AM, Daniel Lezcano >>> wrote: Hi Colin, this patchset

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-01-27 Thread Colin Cross
On Fri, Jan 27, 2012 at 12:54 AM, Vincent Guittot wrote: > On 20 January 2012 21:40, Colin Cross wrote: >> On Fri, Jan 20, 2012 at 12:46 AM, Daniel Lezcano >> wrote: >>> Hi Colin, >>> >>> this patchset could be interesting to resolve in a generic way the cpu >>> dependencies. >>> What is the sta

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2012-01-27 Thread Vincent Guittot
On 20 January 2012 21:40, Colin Cross wrote: > On Fri, Jan 20, 2012 at 12:46 AM, Daniel Lezcano > wrote: >> Hi Colin, >> >> this patchset could be interesting to resolve in a generic way the cpu >> dependencies. >> What is the status of this patchset ? > > I can't do much with it right now, becau

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2011-12-22 Thread Colin Cross
On Thu, Dec 22, 2011 at 12:53 AM, Arjan van de Ven wrote: > On 12/22/2011 9:35 AM, Shilimkar, Santosh wrote: > >> Indeed. The SOCs, Arch's which does support low power >> state independently and doesn't need any co-ordination between CPU's >> will continue to work same way as before with this seri

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2011-12-22 Thread Shilimkar, Santosh
On Thu, Dec 22, 2011 at 2:23 PM, Arjan van de Ven wrote: > On 12/22/2011 9:35 AM, Shilimkar, Santosh wrote: > >> Indeed. The SOCs, Arch's which does support low power >> state independently and doesn't need any co-ordination between CPU's >> will continue to work same way as before with this serie

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2011-12-22 Thread Arjan van de Ven
On 12/22/2011 9:35 AM, Shilimkar, Santosh wrote: > Indeed. The SOCs, Arch's which does support low power > state independently and doesn't need any co-ordination between CPU's > will continue to work same way as before with this series. btw I think you misunderstand; I don't object to a need for

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2011-12-22 Thread Shilimkar, Santosh
On Thu, Dec 22, 2011 at 1:12 AM, Colin Cross wrote: > On Wed, Dec 21, 2011 at 11:36 AM, Arjan van de Ven > wrote: .. or it enters WFI, and a physical device sends it an interrupt, at which point it exits. >>> >>> None of the cpus will return to the idle loop until all cpus have >>>

Re: [linux-pm] [PATCH 0/3] coupled cpuidle state support

2011-12-21 Thread Colin Cross
On Wed, Dec 21, 2011 at 11:36 AM, Arjan van de Ven wrote: >>> >>> .. or it enters WFI, and a physical device sends it an interrupt, >>> at which point it exits. >> >> None of the cpus will return to the idle loop until all cpus have >> decremented the ready counter back to 0, so they can't wrap ar