Re: [PATCH 2/3] NFS: Add NFS_MOUNT_NONEGDE flag to avoid caching negative dentries

[Bob Bell <b_linuxnfs-Y/+76LoPTq9wBoktGHYdvgC/G2K4zDHf@public.gmane.org>]

On Tue, Jan 15, 2008 at 12:04:50PM -0500, Chuck Lever wrote:
>On Jan 15, 2008, at 11:52 AM, Trond Myklebust wrote:
>>The patch itself looks OK. My only gripe is the two names 'negde' and
>>'nonegde'. Can we find something that rolls a bit more smoothly off  
>>the
>>tongue?
>
>Yeah, actually I thought it was spelled "non edge" until I looked  
>closer.

Heh -- I mentioned the same thing at the bottom of my (admittedly 
verbose) intro the patch.  In fact, I've mistyped it "nonedge" myself 
a few times, which I figured was a strong hint it needed renaming.

I was going for the terseness I saw in the rest of the options, and 
"nonegde" was the best my limited creativity could come up with.

>>How about something like 'cacheneglookup' 'nocacheneglookup'? Better
>>suggestions, anyone?

As long as no one objects to the length, I'll go with "nocacheneglookup" 
(and NFS_MOUNT_NOCACHENEGLOOKUP, I suppose) for round 2.

>You might consider tying this behavior to "noac" as well.  In other  
>words, make "noac" equivalent to "sync,actimeo=0,nonegde" .

I'll provide this as a separate patch in the patch set, as Trond 
indicated he would prefer.  This might make sense, as this is probably 
the set of options we'll used.

-- 
Bob Bell