Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-15 Thread Rafael J. Wysocki
On Wednesday, August 08, 2012, Colin Cross wrote: > On Tue, Aug 7, 2012 at 3:15 PM, Rafael J. Wysocki wrote: > > On Wednesday, July 25, 2012, Colin Cross wrote: > >> The cpu hotplug notifier gets called in both atomic and non-atomic > >> contexts, it is not always safe to lock a mutex. Filter

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-15 Thread Rafael J. Wysocki
On Wednesday, August 08, 2012, Colin Cross wrote: On Tue, Aug 7, 2012 at 3:15 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, July 25, 2012, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-07 Thread Colin Cross
On Tue, Aug 7, 2012 at 3:15 PM, Rafael J. Wysocki wrote: > On Wednesday, July 25, 2012, Colin Cross wrote: >> The cpu hotplug notifier gets called in both atomic and non-atomic >> contexts, it is not always safe to lock a mutex. Filter out all events >> except the six necessary ones, which are

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-07 Thread Rafael J. Wysocki
On Wednesday, July 25, 2012, Colin Cross wrote: > The cpu hotplug notifier gets called in both atomic and non-atomic > contexts, it is not always safe to lock a mutex. Filter out all events > except the six necessary ones, which are all sleepable, before taking > the mutex. > > Signed-off-by:

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-07 Thread Rafael J. Wysocki
On Wednesday, July 25, 2012, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones, which are all sleepable, before taking the mutex. Signed-off-by: Colin

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-07 Thread Colin Cross
On Tue, Aug 7, 2012 at 3:15 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, July 25, 2012, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones, which

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-01 Thread Srivatsa S. Bhat
On 07/31/2012 11:57 PM, Colin Cross wrote: > On Tue, Jul 31, 2012 at 8:43 AM, Srivatsa S. Bhat > wrote: >> On 07/26/2012 02:50 AM, Colin Cross wrote: >>> The cpu hotplug notifier gets called in both atomic and non-atomic >>> contexts, it is not always safe to lock a mutex. Filter out all events

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-08-01 Thread Srivatsa S. Bhat
On 07/31/2012 11:57 PM, Colin Cross wrote: On Tue, Jul 31, 2012 at 8:43 AM, Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com wrote: On 07/26/2012 02:50 AM, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex.

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-31 Thread Colin Cross
On Tue, Jul 31, 2012 at 8:43 AM, Srivatsa S. Bhat wrote: > On 07/26/2012 02:50 AM, Colin Cross wrote: >> The cpu hotplug notifier gets called in both atomic and non-atomic >> contexts, it is not always safe to lock a mutex. Filter out all events >> except the six necessary ones, which are all

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-31 Thread Srivatsa S. Bhat
On 07/26/2012 02:50 AM, Colin Cross wrote: > The cpu hotplug notifier gets called in both atomic and non-atomic > contexts, it is not always safe to lock a mutex. Filter out all events > except the six necessary ones, which are all sleepable, before taking > the mutex. > > Signed-off-by: Colin

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-31 Thread Srivatsa S. Bhat
On 07/26/2012 02:50 AM, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones, which are all sleepable, before taking the mutex. Signed-off-by: Colin Cross

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-31 Thread Colin Cross
On Tue, Jul 31, 2012 at 8:43 AM, Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com wrote: On 07/26/2012 02:50 AM, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Shilimkar, Santosh
On Thu, Jul 26, 2012 at 9:25 AM, Shilimkar, Santosh wrote: > On Wed, Jul 25, 2012 at 11:20 PM, Colin Cross wrote: >> The cpu hotplug notifier gets called in both atomic and non-atomic >> contexts, it is not always safe to lock a mutex. Filter out all events >> except the six necessary ones,

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Rafael J. Wysocki
On Thursday, July 26, 2012, Rafael J. Wysocki wrote: > On Thursday, July 26, 2012, Colin Cross wrote: > > On Thu, Jul 26, 2012 at 12:55 PM, Rafael J. Wysocki wrote: > > > On Wednesday, July 25, 2012, Colin Cross wrote: > > >> The cpu hotplug notifier gets called in both atomic and non-atomic > >

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Rafael J. Wysocki
On Thursday, July 26, 2012, Colin Cross wrote: > On Thu, Jul 26, 2012 at 12:55 PM, Rafael J. Wysocki wrote: > > On Wednesday, July 25, 2012, Colin Cross wrote: > >> The cpu hotplug notifier gets called in both atomic and non-atomic > >> contexts, it is not always safe to lock a mutex. Filter out

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Colin Cross
On Thu, Jul 26, 2012 at 12:55 PM, Rafael J. Wysocki wrote: > On Wednesday, July 25, 2012, Colin Cross wrote: >> The cpu hotplug notifier gets called in both atomic and non-atomic >> contexts, it is not always safe to lock a mutex. Filter out all events >> except the six necessary ones, which are

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Rafael J. Wysocki
On Wednesday, July 25, 2012, Colin Cross wrote: > The cpu hotplug notifier gets called in both atomic and non-atomic > contexts, it is not always safe to lock a mutex. Filter out all events > except the six necessary ones, which are all sleepable, before taking > the mutex. I wonder what mutual

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Shilimkar, Santosh
On Wed, Jul 25, 2012 at 11:20 PM, Colin Cross wrote: > The cpu hotplug notifier gets called in both atomic and non-atomic > contexts, it is not always safe to lock a mutex. Filter out all events > except the six necessary ones, which are all sleepable, before taking > the mutex. > >

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Shilimkar, Santosh
On Wed, Jul 25, 2012 at 11:20 PM, Colin Cross ccr...@android.com wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones, which are all sleepable, before taking the mutex.

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Rafael J. Wysocki
On Wednesday, July 25, 2012, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones, which are all sleepable, before taking the mutex. I wonder what mutual

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Colin Cross
On Thu, Jul 26, 2012 at 12:55 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, July 25, 2012, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones,

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Rafael J. Wysocki
On Thursday, July 26, 2012, Colin Cross wrote: On Thu, Jul 26, 2012 at 12:55 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, July 25, 2012, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Rafael J. Wysocki
On Thursday, July 26, 2012, Rafael J. Wysocki wrote: On Thursday, July 26, 2012, Colin Cross wrote: On Thu, Jul 26, 2012 at 12:55 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, July 25, 2012, Colin Cross wrote: The cpu hotplug notifier gets called in both atomic and non-atomic

Re: [PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-26 Thread Shilimkar, Santosh
On Thu, Jul 26, 2012 at 9:25 AM, Shilimkar, Santosh santosh.shilim...@ti.com wrote: On Wed, Jul 25, 2012 at 11:20 PM, Colin Cross ccr...@android.com wrote: The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events

[PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-25 Thread Colin Cross
The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones, which are all sleepable, before taking the mutex. Signed-off-by: Colin Cross --- drivers/cpuidle/coupled.c | 12

[PATCH] cpuidle: coupled: fix sleeping while atomic in cpu notifier

2012-07-25 Thread Colin Cross
The cpu hotplug notifier gets called in both atomic and non-atomic contexts, it is not always safe to lock a mutex. Filter out all events except the six necessary ones, which are all sleepable, before taking the mutex. Signed-off-by: Colin Cross ccr...@android.com --- drivers/cpuidle/coupled.c