Re: [PATCH v2 4/4] mm: make unreserve highatomic functions reliable

2016-10-12 Thread Michal Hocko
On Wed 12-10-16 14:33:36, Minchan Kim wrote: [...] > @@ -2138,8 +2146,10 @@ static bool unreserve_highatomic_pageblock(const > struct alloc_context *ac) >*/ > set_pageblock_migratetype(page, ac->migratetype); > ret = move_freepage

Re: [PATCH v2 4/4] mm: make unreserve highatomic functions reliable

2016-10-12 Thread Minchan Kim
On Wed, Oct 12, 2016 at 09:33:28AM +0200, Michal Hocko wrote: > On Wed 12-10-16 14:33:36, Minchan Kim wrote: > [...] > > @@ -2138,8 +2146,10 @@ static bool unreserve_highatomic_pageblock(const > > struct alloc_context *ac) > > */ > > set_pageblock_migratety

Re: [PATCH v2 4/4] mm: make unreserve highatomic functions reliable

2016-10-12 Thread Vlastimil Babka
On 10/12/2016 07:33 AM, Minchan Kim wrote: Currently, unreserve_highatomic_pageblock bails out if it found highatomic pageblock regardless of really moving free pages from the one so that it could mitigate unreserve logic's goal which saves OOM of a process. This patch makes unreserve functions

[PATCH v2 4/4] mm: make unreserve highatomic functions reliable

2016-10-11 Thread Minchan Kim
Currently, unreserve_highatomic_pageblock bails out if it found highatomic pageblock regardless of really moving free pages from the one so that it could mitigate unreserve logic's goal which saves OOM of a process. This patch makes unreserve functions bail out only if it moves some pages out of !