Re: [PATCH powerpc] fix a lockdep complaint in start_secondary

2012-05-18 Thread Deepthi Dharwar
On 05/18/2012 08:24 AM, Li Zhong wrote: On Thu, 2012-05-17 at 15:52 +0530, Deepthi Dharwar wrote: On 05/17/2012 09:58 AM, Benjamin Herrenschmidt wrote: On Thu, 2012-05-17 at 12:01 +0800, Li Zhong wrote: This patch tries to fix following lockdep complaints: .../...

Re: [PATCH powerpc] fix a lockdep complaint in start_secondary

2012-05-18 Thread Li Zhong
On Fri, 2012-05-18 at 16:54 +0530, Deepthi Dharwar wrote: On 05/18/2012 08:24 AM, Li Zhong wrote: On Thu, 2012-05-17 at 15:52 +0530, Deepthi Dharwar wrote: On 05/17/2012 09:58 AM, Benjamin Herrenschmidt wrote: On Thu, 2012-05-17 at 12:01 +0800, Li Zhong wrote: This patch tries to fix

Re: [PATCH powerpc] fix a lockdep complaint in start_secondary

2012-05-17 Thread Deepthi Dharwar
On 05/17/2012 09:58 AM, Benjamin Herrenschmidt wrote: On Thu, 2012-05-17 at 12:01 +0800, Li Zhong wrote: This patch tries to fix following lockdep complaints: .../... pseries_notify_cpu_idle_add_cpu() actually does cpuidle_disable_device(), and then cpuidle_enable_device(), which

Re: [PATCH powerpc] fix a lockdep complaint in start_secondary

2012-05-17 Thread Li Zhong
On Thu, 2012-05-17 at 15:52 +0530, Deepthi Dharwar wrote: On 05/17/2012 09:58 AM, Benjamin Herrenschmidt wrote: On Thu, 2012-05-17 at 12:01 +0800, Li Zhong wrote: This patch tries to fix following lockdep complaints: .../... pseries_notify_cpu_idle_add_cpu() actually does

[PATCH powerpc] fix a lockdep complaint in start_secondary

2012-05-16 Thread Li Zhong
This patch tries to fix following lockdep complaints: [ 81.882506] = [ 81.882508] [ INFO: inconsistent lock state ] [ 81.882511] 3.4.0-rc4-autokern1 #1 Not tainted [ 81.882513] - [ 81.882516] inconsistent {IN-HARDIRQ-W} -

Re: [PATCH powerpc] fix a lockdep complaint in start_secondary

2012-05-16 Thread Benjamin Herrenschmidt
On Thu, 2012-05-17 at 12:01 +0800, Li Zhong wrote: This patch tries to fix following lockdep complaints: .../... pseries_notify_cpu_idle_add_cpu() actually does cpuidle_disable_device(), and then cpuidle_enable_device(), which releases and allocates the resources respectively. ( Also, all