Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock

2013-07-08 Thread Waiman Long
On 07/05/2013 04:33 PM, Thomas Gleixner wrote: On Fri, 5 Jul 2013, Waiman Long wrote: patch 1:Introduce the new lockref data structure patch 2:Enable x86 architecture to use the feature patch 3:Rename all d_count references to d_refcount And after that the mail threading

Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock

2013-07-05 Thread Thomas Gleixner
On Fri, 5 Jul 2013, Waiman Long wrote: > patch 1: Introduce the new lockref data structure > patch 2: Enable x86 architecture to use the feature > patch 3: Rename all d_count references to d_refcount And after that the mail threading does not work anymore, because you managed to los

Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock

2013-07-05 Thread Waiman Long
On 07/05/2013 01:54 PM, Al Viro wrote: On Fri, Jul 05, 2013 at 11:29:10AM -0400, Waiman Long wrote: I've just pushed such commit into vfs.git#for-linus; please, do the rest on top of it. And keep it bisectable, i.e. so that at any intermediate point the tree would build and work. I am sorry. I

Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock

2013-07-05 Thread Al Viro
On Fri, Jul 05, 2013 at 11:29:10AM -0400, Waiman Long wrote: > >I've just pushed such commit into vfs.git#for-linus; please, do the rest > >on top of it. And keep it bisectable, i.e. so that at any intermediate > >point the tree would build and work. > > I am sorry. I didn't change anything in th

Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock

2013-07-05 Thread Waiman Long
On 07/05/2013 11:29 AM, Waiman Long wrote: On 07/05/2013 11:02 AM, Al Viro wrote: On Fri, Jul 05, 2013 at 10:47:33AM -0400, Waiman Long wrote: v4->v5: - Add a d_count() helper for readonly access of reference count and change all references to d_count outside of dcache.c, dcache.h and

Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock

2013-07-05 Thread Waiman Long
On 07/05/2013 11:02 AM, Al Viro wrote: On Fri, Jul 05, 2013 at 10:47:33AM -0400, Waiman Long wrote: v4->v5: - Add a d_count() helper for readonly access of reference count and change all references to d_count outside of dcache.c, dcache.h and namei.c to use d_count(). Sigh... You are

Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock

2013-07-05 Thread Al Viro
On Fri, Jul 05, 2013 at 10:47:33AM -0400, Waiman Long wrote: > v4->v5: > - Add a d_count() helper for readonly access of reference count and >change all references to d_count outside of dcache.c, dcache.h >and namei.c to use d_count(). Sigh... You are breaking bisectability again. This