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 teardown sequence is: > set ctx->dead > hlist_del_rcu(); > synchronize_rcu(); > > Now we know no system calls can take a new ref, and it's safe to drop > the initial ref: > put_ioctx(); > > We also need to ensure there are no more outstanding kiocbs. This was > done incorrectly - it was being done in kill_ctx(), and before dropping > the initial refcount. At this point, other syscalls may still be > submitting kiocbs! > > Now, we cancel and wait for outstanding kiocbs in free_ioctx(), after > kioctx->users has dropped to 0 and we know no more iocbs could be > submitted. > > Signed-off-by: Kent Overstreet <koverstr...@google.com> > Cc: Zach Brown <z...@redhat.com> > Cc: Felipe Balbi <ba...@ti.com> > Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org> > Cc: Mark Fasheh <mfas...@suse.com> > Cc: Joel Becker <jl...@evilplan.org> > Cc: Rusty Russell <ru...@rustcorp.com.au> > Cc: Jens Axboe <ax...@kernel.dk> > Cc: Asai Thambi S P <asamymuth...@micron.com> > Cc: Selvan Mani <sm...@micron.com> > Cc: Sam Bradshaw <sbrads...@micron.com> > Cc: Jeff Moyer <jmo...@redhat.com> > Cc: Al Viro <v...@zeniv.linux.org.uk> > Cc: Benjamin LaHaise <b...@kvack.org> > Cc: Theodore Ts'o <ty...@mit.edu> > Signed-off-by: Andrew Morton <a...@linux-foundation.org>
Reviewed-by: "Theodore Ts'o" <ty...@mit.edu> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
