On Sun, 24 Aug 2014 09:11:34 -0700 Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Sun, Aug 24, 2014 at 09:06:34AM -0700, Christoph Hellwig wrote: > > We really should split a lm_release from lm_change, the way it is > > used is highly confusing. In addition I think a lot of code > > currently in lease_modify should move here instead, e.g. something like: > > At this point the old lm_change actually becomes superflous if we > simply disallow changes for delegation. > This sounds like a reasonable change, but I think it'll be easier to implement a little later once we've done some cleanup and change how file locks are handled. I'm going to leave this out of this current patchset, but I'll keep it in mind for a later cleanup. -- Jeff Layton <jlayton@xxxxxxxxxxxxxxx> -- 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