On Wed, Apr 10, 2013 at 11:03:39PM +0900, JoonSoo Kim wrote: > Another one what I found is that they don't account "nr_reclaimed" precisely. > There is no code which check whether "current->reclaim_state" exist or not, > except prune_inode(). That's because prune_inode() can free page cache pages when the inode mapping is invalidated. Hence it accounts this in addition to the slab objects being freed. IOWs, if you have a shrinker that frees pages from the page cache, you need to do this. Last time I checked, only inode cache reclaim caused extra page cache reclaim to occur, so most (all?) other shrinkers do not need to do this. It's just another wart that we need to clean up.... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
