Re: [PATCH 3/8] mm, page_alloc: don't retry initial attempt in slowpath

2016-07-20 Thread David Rientjes
On Wed, 20 Jul 2016, Vlastimil Babka wrote: > >> diff --git a/mm/page_alloc.c b/mm/page_alloc.c > >> index eb1968a1041e..30443804f156 100644 > >> --- a/mm/page_alloc.c > >> +++ b/mm/page_alloc.c > >> @@ -3541,35 +3541,42 @@ __alloc_pages_slowpath(gfp_t gfp_mask, unsigned > >> int order, > >>

Re: [PATCH 3/8] mm, page_alloc: don't retry initial attempt in slowpath

2016-07-20 Thread Vlastimil Babka
On 07/20/2016 12:36 AM, David Rientjes wrote: > On Mon, 18 Jul 2016, Vlastimil Babka wrote: > >> 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

Re: [PATCH 3/8] mm, page_alloc: don't retry initial attempt in slowpath

2016-07-19 Thread David Rientjes
On Mon, 18 Jul 2016, Vlastimil Babka wrote: > 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. It makes > sense to to do this

Re: [PATCH 3/8] mm, page_alloc: don't retry initial attempt in slowpath

2016-07-18 Thread Vlastimil Babka
On 07/18/2016 01:29 PM, Michal Hocko wrote: On Mon 18-07-16 13:22:57, Vlastimil Babka wrote: 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

Re: [PATCH 3/8] mm, page_alloc: don't retry initial attempt in slowpath

2016-07-18 Thread Michal Hocko
On Mon 18-07-16 13:22:57, Vlastimil Babka wrote: > 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. It makes > sense to to do

[PATCH 3/8] mm, page_alloc: don't retry initial attempt in slowpath

2016-07-18 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. It makes sense to to do this attempt before adjusting zonelist based on alloc_fla