On Fri, Oct 30, 2015 at 04:01:41PM +0900, Minchan Kim wrote: > MADV_FREE is a hint that it's okay to discard pages if there is memory > pressure and we use reclaimers(ie, kswapd and direct reclaim) to free them > so there is no value keeping them in the active anonymous LRU so this > patch moves them to inactive LRU list's head. > > This means that MADV_FREE-ed pages which were living on the inactive list > are reclaimed first because they are more likely to be cold rather than > recently active pages. > > An arguable issue for the approach would be whether we should put the page > to the head or tail of the inactive list. I chose head because the kernel > cannot make sure it's really cold or warm for every MADV_FREE usecase but > at least we know it's not *hot*, so landing of inactive head would be a > comprimise for various usecases. Even if we're wrong about the aging of those MADV_FREE pages, their contents are invalidated; they can be discarded freely, and restoring them is a mere GFP_ZERO allocation. All other anonymous pages have to be written to disk, and potentially be read back. [ Arguably, MADV_FREE pages should even be reclaimed before inactive page cache. It's the same cost to discard both types of pages, but restoring page cache involves IO. ] It probably makes sense to stop thinking about them as anonymous pages entirely at this point when it comes to aging. They're really not. The LRU lists are split to differentiate access patterns and cost of page stealing (and restoring). From that angle, MADV_FREE pages really have nothing in common with in-use anonymous pages, and so they shouldn't be on the same LRU list. That would also fix the very unfortunate and unexpected consequence of tying the lazy free optimization to the availability of swap space. I would prefer to see this addressed before the code goes upstream. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>