On Thu, 21 Oct 2010 12:59:17 -0500 (CDT) Christoph Lameter <cl@xxxxxxxxx> wrote: > Slab objects (and other caches) are always allocated from ZONE_NORMAL. > Not from any other zone. Calling the shrinkers for those zones may put > unnecessary pressure on the caches. > > Check the zone if we are in a reclaim situation where we are targeting > a specific node. Can occur f.e. in kswapd and in zone reclaim. I have a vague feeling that there was a reason for shrinking the slab for highmem reclaim. Perhaps some scenario in which freeing a slab object would make a highmem page freeable. Something like stripping buffer_heads from a pagecache page, but it wasn't that. I can't immediately find mention in code comments or in ancient changelogs. hrm. Obviously we do want to shrink slab when someone's trying to allocate with __GFP_HIGHMEM because that allocation can also use ZONE_NORMAL. But vmscan will do that as it advances from ZONE_HIGHMEM down to ZONE_NORMAL. The patch doesn't patch direct reclaim, in do_try_to_free_pages(). How come? <ancient memories are stirring> OK, maybe this. Suppose we have a machine with 800M lowmem and 200M highmem. And suppose the lowmem region is stuffed full of clean icache/dcache. A __GFP_HIGHMEM allocation should put pressure on lowmem to get some of those pages back. What we don't want to do is to keep on reclaiming the highmem zone and allocating pages from there, because the machine would effectively end up with only 200M available for pagecache. Please convince us that your patch doesn't screw up zone balancing? -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>