Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-20 Thread Russell King - ARM Linux
On Tue, Oct 19, 2010 at 04:31:30PM -0700, Daniel Walker wrote: On Mon, 2010-10-18 at 09:44 +0200, Ohad Ben-Cohen wrote: OMAP4 introduces a Spinlock hardware module, which provides hardware assistance for synchronization and mutual exclusion between heterogeneous processors and those not

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-20 Thread Ohad Ben-Cohen
[resubmitting due to l-o being dropped from this discussion fork. Thanks Russell for catching this] On Wed, Oct 20, 2010 at 1:31 AM, Daniel Walker dwal...@codeaurora.org wrote: On Mon, 2010-10-18 at 09:44 +0200, Ohad Ben-Cohen wrote: OMAP4 introduces a Spinlock hardware module, which provides

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-20 Thread Daniel Walker
On Wed, 2010-10-20 at 10:53 +0100, Russell King - ARM Linux wrote: To: Ohad Ben-Cohen o...@wizery.com Cc: Hari Kanigeri h-kanige...@ti.com, Benoit Cousson b-cous...@ti.com, Tony Lindgren t...@atomide.com, Greg KH g...@kroah.com, linux-ker...@vger.kernel.org, Grant

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-20 Thread Bryan Huntsman
Indeed. Where is that MSM code ? We don't have an equivalent feature on MSM, nor any plans for one. Daniel was thinking of an internal test feature that had been deprecated a while ago. - Bryan -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center,

[PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Ohad Ben-Cohen
OMAP4 introduces a Spinlock hardware module, which provides hardware assistance for synchronization and mutual exclusion between heterogeneous processors and those not operating under a single, shared operating system (e.g. OMAP4 has dual Cortex-A9, dual Cortex-M3 and a C64x+ DSP). The intention

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Peter Zijlstra
On Mon, 2010-10-18 at 09:44 +0200, Ohad Ben-Cohen wrote: OMAP4 introduces a Spinlock hardware module, which provides hardware assistance for synchronization and mutual exclusion between heterogeneous processors and those not operating under a single, shared operating system (e.g. OMAP4 has

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Russell King - ARM Linux
On Mon, Oct 18, 2010 at 02:46:55PM +0200, Peter Zijlstra wrote: On Mon, 2010-10-18 at 09:44 +0200, Ohad Ben-Cohen wrote: OMAP4 introduces a Spinlock hardware module, which provides hardware assistance for synchronization and mutual exclusion between heterogeneous processors and those not

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Peter Zijlstra
On Mon, 2010-10-18 at 14:35 +0100, Russell King - ARM Linux wrote: On Mon, Oct 18, 2010 at 02:46:55PM +0200, Peter Zijlstra wrote: On Mon, 2010-10-18 at 09:44 +0200, Ohad Ben-Cohen wrote: OMAP4 introduces a Spinlock hardware module, which provides hardware assistance for synchronization

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Ohad Ben-Cohen
On Mon, Oct 18, 2010 at 3:43 PM, Peter Zijlstra pet...@infradead.org wrote: Right, so the problem is that there simply is no way to do atomic memory access from these auxiliary processing units wrt the main CPU? Yes. There are a few relevant system-wide limitations, one of them is that simply

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Peter Zijlstra
On Mon, 2010-10-18 at 16:28 +0200, Ohad Ben-Cohen wrote: On Mon, Oct 18, 2010 at 3:43 PM, Peter Zijlstra pet...@infradead.org wrote: Right, so the problem is that there simply is no way to do atomic memory access from these auxiliary processing units wrt the main CPU? Yes. There are a few

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Catalin Marinas
Peter Zijlstra pet...@infradead.org wrote: On Mon, 2010-10-18 at 14:35 +0100, Russell King - ARM Linux wrote: In any case, Linux's spinlock API (or more accurately, the ARM exclusive access instructions) relies upon hardware coherency support (a piece of hardware called an exclusive monitor)

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Peter Zijlstra
On Mon, 2010-10-18 at 16:27 +0100, Catalin Marinas wrote: Peter Zijlstra pet...@infradead.org wrote: On Mon, 2010-10-18 at 14:35 +0100, Russell King - ARM Linux wrote: In any case, Linux's spinlock API (or more accurately, the ARM exclusive access instructions) relies upon hardware

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Ohad Ben-Cohen
On Mon, Oct 18, 2010 at 5:32 PM, Peter Zijlstra pet...@infradead.org wrote: Sure, but you can 'easily' extend your coherency protocols with support In our case, there are no coherency protocols supported between the A9, M3 and the DSP. -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Peter Zijlstra
On Mon, 2010-10-18 at 17:35 +0200, Ohad Ben-Cohen wrote: On Mon, Oct 18, 2010 at 5:32 PM, Peter Zijlstra pet...@infradead.org wrote: Sure, but you can 'easily' extend your coherency protocols with support In our case, there are no coherency protocols supported between the A9, M3 and the

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Catalin Marinas
Peter Zijlstra pet...@infradead.org wrote: On Mon, 2010-10-18 at 16:27 +0100, Catalin Marinas wrote: Peter Zijlstra pet...@infradead.org wrote: On Mon, 2010-10-18 at 14:35 +0100, Russell King - ARM Linux wrote: In any case, Linux's spinlock API (or more accurately, the ARM exclusive access

Re: [PATCH 0/3] Add OMAP hardware spinlock misc driver

2010-10-18 Thread Peter Zijlstra
On Mon, 2010-10-18 at 16:51 +0100, Catalin Marinas wrote: Peter Zijlstra pet...@infradead.org wrote: On Mon, 2010-10-18 at 16:27 +0100, Catalin Marinas wrote: Peter Zijlstra pet...@infradead.org wrote: On Mon, 2010-10-18 at 14:35 +0100, Russell King - ARM Linux wrote: In any case,