|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>|
|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>|
|User-agent:||Opera Mail/9.24 (Win32)|
On Fri, 16 May 2008 00:11:21 +1000, Christoph Hellwig <hch@xxxxxxxxxxxxx>
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 entryis 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>|
|Previous by Date:||Re: [PATCH] scale down 074 on lower end machines, David Chinner|
|Next by Date:||[PATCH] xfs_dm_rdwr() needs to pass a vfsmount to dentry_open(), Lachlan McIlroy|
|Previous by Thread:||Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache, Christoph Hellwig|
|Next by Thread:||Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache, Anton Altaparmakov|
|Indexes:||[Date] [Thread] [Top] [All Lists]|