Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-13 Thread Arnd Bergmann
On Thursday 13 September 2007, Michael Ellerman wrote: > Well that'd be nice, but I don't see anywhere that that happens. AFAICT > the acquire we do in the first coredump callback is the first the SPU > contexts know about their PPE process dying. And spufs is still live, so > I think we

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-13 Thread Arnd Bergmann
On Thursday 13 September 2007, Michael Ellerman wrote: Well that'd be nice, but I don't see anywhere that that happens. AFAICT the acquire we do in the first coredump callback is the first the SPU contexts know about their PPE process dying. And spufs is still live, so I think we definitely

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
On Wed, 2007-09-12 at 10:47 +0200, Arnd Bergmann wrote: > On Wednesday 12 September 2007, Michael Ellerman wrote: > > On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: > > > This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around > > > DEFINE_SIMPLE_ATTRIBUTE which does the specified

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: > On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: > > This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around > > DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get > > routine for us. > > > > Unfortunately we

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: > This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around > DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get > routine for us. > > Unfortunately we need two get routines (a locked and unlocked version) to > support

[PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get routine for us. Unfortunately we need two get routines (a locked and unlocked version) to support the coredump code. This patch hides one of those (the locked version)

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
On Wed, 2007-09-12 at 10:47 +0200, Arnd Bergmann wrote: On Wednesday 12 September 2007, Michael Ellerman wrote: On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for

[PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get routine for us. Unfortunately we need two get routines (a locked and unlocked version) to support the coredump code. This patch hides one of those (the locked version)

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Michael Ellerman
On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get routine for us. Unfortunately we need two get routines (a locked and unlocked version) to support the

Re: [PATCH 15/15] Add DEFINE_SPUFS_ATTRIBUTE()

2007-09-12 Thread Arnd Bergmann
On Wednesday 12 September 2007, Michael Ellerman wrote: On Wed, 2007-09-12 at 17:43 +1000, Michael Ellerman wrote: This patch adds DEFINE_SPUFS_ATTRIBUTE(), a wraper around DEFINE_SIMPLE_ATTRIBUTE which does the specified locking for the get routine for us. Unfortunately we need two