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. -- 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
