On Mon, Jul 21, 2014 at 11:02:19AM -0400, Jeff Layton wrote: > Releasing locks when we unhash the stateid instead of doing so only when > the stateid is actually released will be problematic in later patches > when we need to protect the unhashing with spinlocks. Move it into the > sc_free operation instead. > > Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxxxxxxx> Looks reasonable except that sc_free is now initialized twice for lock stateids. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html