| 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 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. |
| 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] |