On Sun, Aug 24, 2014 at 09:43:01PM -0400, Jeff Layton wrote: > > - add a return value to lm_break so that the lock manager can tell the > > core code "you can delete this lease right now". That gets rid of > > the games with the timeout which require all kinds of race avoidance > > code in the users. > > > > I'm not sure I understand what you're suggesting here. Isn't it just as > simple to have lm_break call lease_modify to just remove the lease? Unfortunately it's not. lm_break gets a pointer to the file_lock, and lease_modify needs a pointer to the list position of the file_lock. -- 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