Re: [PATCH v2 03/18] mm, page_alloc: don't retry initial attempt in slowpath

2016-06-01 Thread Vlastimil Babka
On 06/01/2016 03:26 PM, Michal Hocko wrote: > On Tue 31-05-16 15:08:03, Vlastimil Babka wrote: > [...] >> diff --git a/mm/page_alloc.c b/mm/page_alloc.c >> index da3a62a94b4a..9f83259a18a8 100644 >> --- a/mm/page_alloc.c >> +++ b/mm/page_alloc.c >> @@ -3367,10 +3367,9 @@ __alloc_pages_direct_reclai

Re: [PATCH v2 03/18] mm, page_alloc: don't retry initial attempt in slowpath

2016-06-01 Thread Michal Hocko
On Tue 31-05-16 15:08:03, Vlastimil Babka wrote: [...] > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index da3a62a94b4a..9f83259a18a8 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -3367,10 +3367,9 @@ __alloc_pages_direct_reclaim(gfp_t gfp_mask, unsigned > int order, > bool

[PATCH v2 03/18] mm, page_alloc: don't retry initial attempt in slowpath

2016-05-31 Thread Vlastimil Babka
After __alloc_pages_slowpath() sets up new alloc_flags and wakes up kswapd, it first tries get_page_from_freelist() with the new alloc_flags, as it may succeed e.g. due to using min watermark instead of low watermark. This attempt does not have to be retried on each loop, since direct reclaim, dire