From: Trond Myklebust <trond.myklebust@fys.uio.no>
Subject: RE: [PATCH 2/3] NFS: Add NFS_MOUNT_NONEGDE flag to avoid caching
	negative dentries
Date: Wed, 16 Jan 2008 15:43:39 -0500
Message-ID: <1200516219.6932.22.camel@heimdal.trondhjem.org>
References: <20080115163013.GC18911@newbie.thebellsplace.net>
	 <1200415972.7702.7.camel@heimdal.trondhjem.org>
	 <CB3B3B75-ECE5-4A46-BEA5-7DEDA8DB7DCA@oracle.com>
	 <20080116012209.GB26010@newbie.thebellsplace.net>
	 <1200451350.28088.43.camel@heimdal.trondhjem.org>
	 <478DFD5A.1040501@RedHat.com>
	 <01AE8AF878612047A442668306EAEB05018178E7@SACEXMV01.hq.netapp.com>
Mime-Version: 1.0
Content-Type: text/plain
Cc: Steve Dickson <SteveD@redhat.com>,
	Bob Bell <b_linuxnfs-Y/+76LoPTq9wBoktGHYdvgC/G2K4zDHf@public.gmane.org>,
	Chuck Lever <chuck.lever@oracle.com>,
	NFS list <linux-nfs@vger.kernel.org>
To: "Muntz, Daniel" <Dan.Muntz@netapp.com>
In-Reply-To: <01AE8AF878612047A442668306EAEB05018178E7-Ye0EzW0T4yEQ4vjYWPuN5KYtpRd4g51m@public.gmane.org>
Sender: linux-nfs-owner@vger.kernel.org


On Wed, 2008-01-16 at 11:43 -0800, Muntz, Daniel wrote:
> My $.02
> 
>  -odircache=on
>  -odircache=hitsonly (or positive, or hits, or ?)
>  -odircache=off


I like the 'dircache' bit, but to me 'on/off' suggests a binary switch:
they don't mix well with a third state.

How about

        -odircache=aggr[essive]	/* Full caching */
        -odircache=noneg[ative]	/* Positive lookups only */
        -odircache=off		/* strict lookup revalidation */

?


> -----Original Message-----
> From: Steve Dickson [mailto:SteveD@redhat.com] 
> Sent: Wednesday, January 16, 2008 4:50 AM
> To: Trond Myklebust
> Cc: Bob Bell; Chuck Lever; NFS list
> Subject: Re: [PATCH 2/3] NFS: Add NFS_MOUNT_NONEGDE flag to avoid
> caching negative dentries
> 
> Trond Myklebust wrote:
> > On Tue, 2008-01-15 at 20:22 -0500, Bob Bell wrote:
> >>>> 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.
> > 
> > I'm still not happy with that name. What do people think about 
> > introducing a generic 'cache' option that can be used for fine control
> 
> > of caching options?
> > 
> > I'm thinking somewhat along the lines of
> > 
> > 	-ocache=lookup_all		/* Cache all dentries */
> > 	-ocache=lookup_positive		/* Cache positive dentries */
> > 	-ocache=lookup_none		/* Force all lookups */
> Maybe we should define the type of cache were are controlling, in case
> we want to add controls for other caches. So how about:
> 
>         -olpcache=all        /* Cache all directory entries */
>         -olpcache=found      /* Cache only directory entires that are
> found */
>         -olpcache=off        /* Turn off the caching of directory
> entries */ 
> 
> 
> steved.
> -
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org More majordomo info
> at  http://vger.kernel.org/majordomo-info.html
> -
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html