This fixes the recently-reported "fsstress memory leak" problem. It has been there since November 2002. shrink_dcache() has a heuristic to prevent the dcache (and hence icache) from getting shrunk too far: it refuses to allow the dcache to shrink below 2*nr_used. Problem is, _all_ non-leaf dentries (directories) count as used. So when you have really deep directory hierarchies (fsstress creates these), nr_used is really high, and there is no upper bound to the amount of pinned dcache. The patch just rips out the heuristic. This means that dcache (and hence icache (and hence pagecache)) will be shrunk more aggressively. This could be a problem, and tons of testing is needed - a new heuristic may be needed. However I am not able to reproduce the problem which cause me to add this heuristic in the first place: Simple testcase: run a huge `dd' while running a concurrent `watch -n1 cat /proc/meminfo'. The program text for `cat' gets loaded from disk once per second. fs/dcache.c | 21 +-------------------- 1 files changed, 1 insertion(+), 20 deletions(-) diff -puN fs/dcache.c~dentry-bloat-fix-2 fs/dcache.c --- 25/fs/dcache.c~dentry-bloat-fix-2 2003-10-26 04:00:13.000000000 -0800 +++ 25-akpm/fs/dcache.c 2003-10-26 16:31:05.000000000 -0800 @@ -639,24 +639,9 @@ void shrink_dcache_anon(struct hlist_hea /* * This is called from kswapd when we think we need some more memory. - * - * We don't want the VM to steal _all_ unused dcache. Because that leads to - * the VM stealing all unused inodes, which shoots down recently-used - * pagecache. So what we do is to tell fibs to the VM about how many reapable - * objects there are in this cache. If the number of unused dentries is - * less than half of the total dentry count then return zero. The net effect - * is that the number of unused dentries will be, at a minimum, equal to the - * number of used ones. - * - * If unused_ratio is set to 5, the number of unused dentries will not fall - * below 5* the number of used ones. */ static int shrink_dcache_memory(int nr, unsigned int gfp_mask) { - int nr_used; - int nr_unused; - const int unused_ratio = 1; - if (nr) { /* * Nasty deadlock avoidance. @@ -672,11 +657,7 @@ static int shrink_dcache_memory(int nr, if (gfp_mask & __GFP_FS) prune_dcache(nr); } - nr_unused = dentry_stat.nr_unused; - nr_used = dentry_stat.nr_dentry - nr_unused; - if (nr_unused < nr_used * unused_ratio) - return 0; - return nr_unused - nr_used * unused_ratio; + return dentry_stat.nr_unused; } #define NAME_ALLOC_LEN(len) ((len+16) & ~15) _