xfs
[Top] [All Lists]

Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache

To: "Christoph Hellwig" <hch@xxxxxxxxxxxxx>, "Anton Altaparmakov" <aia21@xxxxxxxxx>
Subject: Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache
From: "Barry Naujok" <bnaujok@xxxxxxx>
Date: Fri, 16 May 2008 10:30:37 +1000
Cc: xfs@xxxxxxxxxxx, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>
In-reply-to: <20080515141121.GA14198@xxxxxxxxxxxxx>
Organization: SGI
References: <20080513075749.477238845@xxxxxxxxxxxxxxxxxxxxxxx> <20080513080152.911303131@xxxxxxxxxxxxxxxxxxxxxxx> <20080513085724.GC21919@xxxxxxxxxxxxx> <op.ua4wa7t03jf8g2@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20080515045700.GA4328@xxxxxxxxxxxxx> <op.ua6ji4r93jf8g2@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <DCB15FFF-F942-47BD-B8FB-38AADC24B9D6@xxxxxxxxx> <20080515141121.GA14198@xxxxxxxxxxxxx>
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Opera Mail/9.24 (Win32)
On Fri, 16 May 2008 00:11:21 +1000, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:

On Thu, May 15, 2008 at 02:43:44PM +0100, Anton Altaparmakov wrote:
Yes, and you can get the performance back if you allow negative dentries to be created. You just have to make sure that every time a directory entry
is created in directory X, all negative dentries which are children of
directory X are thrown away.

We might even be able to optimize this a little by calling d_compare on
each alias to see if it hashes down to the same one down in the fs.

So, I gather from this comment is that the CI FS implementation overrides
d_compare but not d_hash, and the CI purge function will behave differently
if d_compare is overridden or not? If overridden, use it as per your
suggestion and if not (due to oversight or unexpected problems), purge
all negative children?

Barry.


<Prev in Thread] Current Thread [Next in Thread>