|To:||Christoph Hellwig <hch@xxxxxxxxxxxxx>|
|Subject:||Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache|
|From:||Anton Altaparmakov <aia21@xxxxxxxxx>|
|Date:||Fri, 16 May 2008 08:25:55 +0100|
|Cc:||Barry Naujok <bnaujok@xxxxxxx>, xfs@xxxxxxxxxxx, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>|
|References:||<firstname.lastname@example.org> <email@example.com> <20080513085724.GC21919@infradead.org> <firstname.lastname@example.org> <20080515045700.GA4328@infradead.org> <email@example.com> <DCB15FFF-F942-47BD-B8FB-38AADC24B9D6@cam.ac.uk> <20080515141121.GA14198@infradead.org>|
On 15 May 2008, at 15:11, Christoph Hellwig 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
Perhaps, although I am not convinced that wouldn't be worse than just throwing them all away. For example think of a very active directory with thousands of negative dentries in it. A create comes in and we either throw away thousands of entries or we perform a case insensitive comparison thousands of times and discard only a few entries. I am concerned that the "thousands of case insensitive comparisons" would actually be very costly and far outweigh the cost of throwing all negative dentries away and letting them be created again if they are requested again.
At lest in NTFS the case insensitive comparison is very expensive as it involves converting both the UTF8 strings into little endian 2-byte fixed width Unicode, and then for each Unicode character of each of the strings being compared, individually performing a look up in the 128kiB large Unicode upcase table and then the two upcased characters are compared and if they match we move to the next character.
Doing that a thousand times would be way more expensive then simply throwing all negative dentries away I would think.
In the end it probably depends on the usage scenario as to what will be more efficient and perhaps on the file system as well so it may be worth allowing the file system to decide whether to try and do comparisons or to simply throw all the negative dentries away.
Anton -- Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @) Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK Linux NTFS maintainer, http://www.linux-ntfs.org/
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:||Re: [PATCH] merge xfs_mntupdate into xfs_fs_remount, Christoph Hellwig|
|Next by Date:||Re: XFS shutdown in xfs_iunlink_remove() (was Re: 2.6.25: swapper: page allocation failure. order:3, mode:0x4020), Marco Berizzi|
|Previous by Thread:||Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache, Barry Naujok|
|Next by Thread:||Re: [PATCH 3/4] XFS: Return case-insensitive match for dentry cache, Barry Naujok|
|Indexes:||[Date] [Thread] [Top] [All Lists]|