Re: [RFC PATCH] ARM: keep __my_cpu_offset consistent with generic one

2013-03-06 Thread Ming Lei
Hi, On Thu, Mar 7, 2013 at 11:44 AM, Will Deacon wrote: > > For the record, can you include a backtrace or something please? The lock_release_holdtime() will access percpu variable, so every lock release may trigger the hang if CONFIG_LOCK_STAT is enabled, and there are many lock

Re: [RFC PATCH] ARM: keep __my_cpu_offset consistent with generic one

2013-03-06 Thread Will Deacon
Hello, On Thu, Mar 07, 2013 at 02:47:48AM +, Ming Lei wrote: > Commit 14318efb(ARM: 7587/1: implement optimized percpu variable access) > introduces arm's __my_cpu_offset to optimize percpu vaiable access, > which really works well on hackbench test, but may cause __my_cpu_offset > to return

[RFC PATCH] ARM: keep __my_cpu_offset consistent with generic one

2013-03-06 Thread Ming Lei
Commit 14318efb(ARM: 7587/1: implement optimized percpu variable access) introduces arm's __my_cpu_offset to optimize percpu vaiable access, which really works well on hackbench test, but may cause __my_cpu_offset to return garbage value before it is initialized in cpu_init() called by setup_arch,

[RFC PATCH] ARM: keep __my_cpu_offset consistent with generic one

2013-03-06 Thread Ming Lei
Commit 14318efb(ARM: 7587/1: implement optimized percpu variable access) introduces arm's __my_cpu_offset to optimize percpu vaiable access, which really works well on hackbench test, but may cause __my_cpu_offset to return garbage value before it is initialized in cpu_init() called by setup_arch,

Re: [RFC PATCH] ARM: keep __my_cpu_offset consistent with generic one

2013-03-06 Thread Will Deacon
Hello, On Thu, Mar 07, 2013 at 02:47:48AM +, Ming Lei wrote: Commit 14318efb(ARM: 7587/1: implement optimized percpu variable access) introduces arm's __my_cpu_offset to optimize percpu vaiable access, which really works well on hackbench test, but may cause __my_cpu_offset to return

Re: [RFC PATCH] ARM: keep __my_cpu_offset consistent with generic one

2013-03-06 Thread Ming Lei
Hi, On Thu, Mar 7, 2013 at 11:44 AM, Will Deacon will.dea...@arm.com wrote: For the record, can you include a backtrace or something please? The lock_release_holdtime() will access percpu variable, so every lock release may trigger the hang if CONFIG_LOCK_STAT is enabled, and there are many