On Mon 17-04-17 17:06:20, David Rientjes wrote: > The purpose of the code that commit 623762517e23 ("revert 'mm: vmscan: do > not swap anon pages just because free+file is low'") reintroduces is to > prefer swapping anonymous memory rather than trashing the file lru. > > If all anonymous memory is unevictable, however, this insistance on > SCAN_ANON ends up thrashing that lru instead. Why would be the anonymous memory unevictable? If the swap is depleted then we enforce file scanning AFAIR. Are those pages pinned somehow, by who? It would be great if you could describe the workload which triggers a problem which you are trying to fix. > Check that enough evictable anon memory is actually on this lruvec before > insisting on SCAN_ANON. SWAP_CLUSTER_MAX is used as the threshold to > determine if only scanning anon is beneficial. > > Otherwise, fallback to balanced reclaim so the file lru doesn't remain > untouched. Why should we treat anonymous and file pages any different here. In other words why should file pages check for high wmark and anonymous for SWAP_CLUSTER_MAX. [...] -- Michal Hocko SUSE Labs -- 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>