Re: [Cbe-oss-dev] [PATCH] Cell OProfile: SPU mutex lock fix

2008-04-08 Thread Arnd Bergmann
On Tuesday 08 April 2008, Carl Love wrote: > On Fri, 2008-04-04 at 08:38 +0200, Arnd Bergmann wrote: > Our first thought to fix the bug was to just grab the mutex lock when > adding the switch notification data to the queue. The kernel gives us > an oops message saying something along the line of

Re: [Cbe-oss-dev] [PATCH] Cell OProfile: SPU mutex lock fix

2008-04-08 Thread Carl Love
On Fri, 2008-04-04 at 08:38 +0200, Arnd Bergmann wrote: > On Wednesday 02 April 2008, Carl Love wrote: > > On Wed, 2008-04-02 at 07:21 +0200, Arnd Bergmann wrote: > > > On Tuesday 25 March 2008, Carl Love wrote: > > > > This patch fixes a bug in the code that records the SPU data and > > > > cont

Re: [Cbe-oss-dev] [PATCH] Cell OProfile: SPU mutex lock fix

2008-04-03 Thread Arnd Bergmann
On Wednesday 02 April 2008, Carl Love wrote: > On Wed, 2008-04-02 at 07:21 +0200, Arnd Bergmann wrote: > > On Tuesday 25 March 2008, Carl Love wrote: > > > This patch fixes a bug in the code that records the SPU data and > > > context switches. The buffer_mutex lock must be held when the > > > ker

Re: [Cbe-oss-dev] [PATCH] Cell OProfile: SPU mutex lock fix

2008-04-02 Thread Carl Love
On Wed, 2008-04-02 at 09:42 -0700, Carl Love wrote: > On Wed, 2008-04-02 at 07:21 +0200, Arnd Bergmann wrote: > > On Tuesday 25 March 2008, Carl Love wrote: > > > This patch fixes a bug in the code that records the SPU data and > > > context switches. The buffer_mutex lock must be held when the >

Re: [Cbe-oss-dev] [PATCH] Cell OProfile: SPU mutex lock fix

2008-04-02 Thread Carl Love
On Wed, 2008-04-02 at 07:21 +0200, Arnd Bergmann wrote: > On Tuesday 25 March 2008, Carl Love wrote: > > This patch fixes a bug in the code that records the SPU data and > > context switches. The buffer_mutex lock must be held when the > > kernel is adding data to the buffer between the kernel an

Re: [Cbe-oss-dev] [PATCH] Cell OProfile: SPU mutex lock fix

2008-04-01 Thread Arnd Bergmann
On Tuesday 25 March 2008, Carl Love wrote: > This patch fixes a bug in the code that records the SPU data and > context switches. The buffer_mutex lock must be held when the > kernel is adding data to the buffer between the kernel and the > OProfile daemon. The lock is not being held in the curre