From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Blunck Subject: Re: [PATCH 30/30] VFS: Destroy the dentries contributed by a superblock on unmounting [try #11] Date: Tue, 1 Aug 2006 11:03:00 +0200 Message-ID: <20060801090259.GB10032@X40.localnet> References: <20060727205222.8443.29381.stgit@warthog.cambridge.redhat.com> <20060727205333.8443.97943.stgit@warthog.cambridge.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: torvalds@osdl.org, akpm@osdl.org, steved@redhat.com, trond.myklebust@fys.uio.no, linux-fsdevel@vger.kernel.org, linux-cachefs@redhat.com, nfsv4@linux-nfs.org, linux-kernel@vger.kernel.org Return-path: To: David Howells Content-Disposition: inline In-Reply-To: <20060727205333.8443.97943.stgit@warthog.cambridge.redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Thu, Jul 27, David Howells wrote: > diff --git a/include/linux/dcache.h b/include/linux/dcache.h > index 44605be..63f64a9 100644 > --- a/include/linux/dcache.h > +++ b/include/linux/dcache.h > @@ -230,6 +230,7 @@ extern struct dentry * d_alloc_anon(stru > extern struct dentry * d_splice_alias(struct inode *, struct dentry *); > extern void shrink_dcache_sb(struct super_block *); > extern void shrink_dcache_parent(struct dentry *); > +extern void shrink_dcache_for_umount(struct super_block *); > extern int d_invalidate(struct dentry *); > > /* only used at mount-time */ I don't see the point that we need two different versions of shrink_dcache_sb(). IMHO it is better to fix shrink_dcache_for_umount() so that it is a replacement for shrink_dcache_sb(). BTW: Talking about shrink_dcache_sb(): is it really necessary to call shrink_dcache_sb() when remounting a filesystem? The only reason I can see are changes to the lookup mechanism (hash algorithm etc) but a quick look into the different filesystems forbid the change of this options during remount. Jan