On Mon, 23 Jun 2014 09:23:32 -0700 Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Mon, Jun 23, 2014 at 12:20:39PM -0400, Jeff Layton wrote: > > 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. > > It might be fairly obvious in the context of the series, it's not > if someone goes back in history with a git-blame. > > And it's not a 100% obvious with the series either, at least earlier > versions also fixes pre-existing races, although all of that might be > upstream now. > > So as far as I am concerned a simple one-line blurb mentioning what > the new locking is for would be useful to be added to all patches > just changing the locking. Fair enough. I'll do that. Thanks, -- 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
