On Mon, 23 Jun 2014 09:06:36 -0700 Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > Missed the why the locking is added. > > Also I pointed out a way to better factor this code in reply to one > of Tronds previous postings of this one. > I saw your comments and even did some experimentation to that end. After playing around with some patches that tried to use FMODE flags instead, I tossed them out. It may have consolidated the code a bit, but the result was just too difficult to follow. I think it's better to go with clearer code here, even if it's a little more verbose. -- 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