Re: [PATCH 12/33] aio: refcounting cleanup

2013-04-01 Thread Theodore Ts'o
On Thu, Mar 21, 2013 at 09:35:33AM -0700, Kent Overstreet wrote: > The usage of ctx->dead was fubar - it makes no sense to explicitly check > it all over the place, especially when we're already using RCU. > > Now, ctx->dead only indicates whether we've dropped the initial > refcount. The new tear

[PATCH 12/33] aio: refcounting cleanup

2013-03-21 Thread Kent Overstreet
The usage of ctx->dead was fubar - it makes no sense to explicitly check it all over the place, especially when we're already using RCU. Now, ctx->dead only indicates whether we've dropped the initial refcount. The new teardown sequence is: set ctx->dead hlist_del_rcu(); synchronize_rcu(); Now we