On Mon, 23 Jun 2014 09:04:07 -0700 Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Thu, Jun 19, 2014 at 10:49:11AM -0400, Jeff Layton wrote: > > From: Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> > > > > Signed-off-by: Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> > > Looks good, but should have a patch description that better tells > why this is done (looks like preparing for nfs_lock_state() removal > again, not fixing some existing race) > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> Correct. I'll add that into the desciption if you think it's warranted, but again that's the case with many of the patches in this series. -- 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