On Mon, Jun 23, 2014 at 12:24:58PM -0400, Jeff Layton wrote: > 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. I appreciate that it's tedious but yes, thanks, that would be helpful. --b. -- 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