J. Bruce Fields wrote: > On Wed, Apr 09, 2008 at 05:29:47PM -0400, Peter Staubach wrote: >> Steve Dickson wrote: >>> Jeff Layton wrote: >>> >>>> Yes. But like I mentioned in the description, there is a potential >>>> corner case here. If someone does a foreground mount and >>>> explicitly sets retry=10000 then it will be reset to 2. It's not very >>>> likely, but is simple enough to prevent... >>>> >>> hmm.. I guess... but programing for every single brain dead >>> corner case can really make the code more complicated than >>> it need to be, which I think is the case here... I think I would >>> rather just added the three lines and live that corner case... >> Customers tend to be remarkably good at finding corner cases and >> then wondering why they don't work as advertised... > > And 10000 is a nice round number. Given more than a few users, > *somebody* is going to try it eventually, and get extremely confused. Ok... I'll leave the patch as is... steved. -- 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
