| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 1/5] inode: Make unused inode LRU per superblock |
| From: | Nick Piggin <npiggin@xxxxxxx> |
| Date: | Thu, 27 May 2010 02:17:33 +1000 |
| Cc: | linux-kernel@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, xfs@xxxxxxxxxxx |
| In-reply-to: | <1274777588-21494-2-git-send-email-david@xxxxxxxxxxxxx> |
| References: | <1274777588-21494-1-git-send-email-david@xxxxxxxxxxxxx> <1274777588-21494-2-git-send-email-david@xxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.20 (2009-06-14) |
On Tue, May 25, 2010 at 06:53:04PM +1000, Dave Chinner wrote: > From: Dave Chinner <dchinner@xxxxxxxxxx> > > The inode unused list is currently a global LRU. This does not match > the other global filesystem cache - the dentry cache - which uses > per-superblock LRU lists. Hence we have related filesystem object > types using different LRU reclaimatin schemes. Is this an improvement I wonder? The dcache is using per sb lists because it specifically requires sb traversal. What allocation/reclaim really wants (for good scalability and NUMA characteristics) is per-zone lists for these things. It's easy to convert a single list into per-zone lists. It is much harder to convert per-sb lists into per-sb x per-zone lists. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 11/17] fs/xfs/quota: Add missing mutex_unlock, Julia Lawall |
|---|---|
| Next by Date: | Re: [PATCH 3/5] superblock: introduce per-sb cache shrinker infrastructure, Nick Piggin |
| Previous by Thread: | [PATCH 1/5] inode: Make unused inode LRU per superblock, Dave Chinner |
| Next by Thread: | Re: [PATCH 1/5] inode: Make unused inode LRU per superblock, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |