Re: [PATCH 2/3] mm, vmscan: consider eligible zones in get_scan_count

2017-02-06 Thread Andrew Morton
On Mon, 6 Feb 2017 09:10:07 +0100 Michal Hocko wrote: > Hi Andrew, > it turned out that this is not a theoretical issue after all. Trevor > (added to the CC) was seeing pre-mature OOM killer triggering [1] > bisected to b2e18757f2c9 ("mm, vmscan: begin reclaiming pages on a > per-node basis"). >

Re: [PATCH 2/3] mm, vmscan: consider eligible zones in get_scan_count

2017-02-06 Thread Michal Hocko
Hi Andrew, it turned out that this is not a theoretical issue after all. Trevor (added to the CC) was seeing pre-mature OOM killer triggering [1] bisected to b2e18757f2c9 ("mm, vmscan: begin reclaiming pages on a per-node basis"). After some going back and forth it turned out that b4536f0c829c ("mm

Re: [PATCH 2/3] mm, vmscan: consider eligible zones in get_scan_count

2017-01-18 Thread Johannes Weiner
On Tue, Jan 17, 2017 at 11:37:01AM +0100, Michal Hocko wrote: > From: Michal Hocko > > get_scan_count considers the whole node LRU size when > - doing SCAN_FILE due to many page cache inactive pages > - calculating the number of pages to scan > > in both cases this might lead to unexpected behav

[PATCH 2/3] mm, vmscan: consider eligible zones in get_scan_count

2017-01-17 Thread Michal Hocko
From: Michal Hocko get_scan_count considers the whole node LRU size when - doing SCAN_FILE due to many page cache inactive pages - calculating the number of pages to scan in both cases this might lead to unexpected behavior especially on 32b systems where we can expect lowmem memory pressure ver

Re: [PATCH 2/3] mm, vmscan: consider eligible zones in get_scan_count

2017-01-16 Thread Hillf Danton
On Tuesday, January 17, 2017 3:33 AM Michal Hocko wrote: > > From: Michal Hocko > > get_scan_count considers the whole node LRU size when > - doing SCAN_FILE due to many page cache inactive pages > - calculating the number of pages to scan > > in both cases this might lead to unexpected behav

[PATCH 2/3] mm, vmscan: consider eligible zones in get_scan_count

2017-01-16 Thread Michal Hocko
From: Michal Hocko get_scan_count considers the whole node LRU size when - doing SCAN_FILE due to many page cache inactive pages - calculating the number of pages to scan in both cases this might lead to unexpected behavior especially on 32b systems where we can expect lowmem memory pressure ver