Re: [PATCH 1/2] mm: get rid of __alloc_pages_high_priority

2015-11-18 Thread Vlastimil Babka
On 11/16/2015 02:22 PM, mho...@kernel.org wrote: > From: Michal Hocko > > __alloc_pages_high_priority doesn't do anything special other than it > calls get_page_from_freelist and loops around GFP_NOFAIL allocation > until it succeeds. It would be better if the first part was done in > __alloc_pag

Re: [PATCH 1/2] mm: get rid of __alloc_pages_high_priority

2015-11-16 Thread David Rientjes
On Mon, 16 Nov 2015, mho...@kernel.org wrote: > From: Michal Hocko > > __alloc_pages_high_priority doesn't do anything special other than it > calls get_page_from_freelist and loops around GFP_NOFAIL allocation > until it succeeds. It would be better if the first part was done in > __alloc_pages

Re: [PATCH 1/2] mm: get rid of __alloc_pages_high_priority

2015-11-16 Thread Mel Gorman
On Mon, Nov 16, 2015 at 02:22:18PM +0100, mho...@kernel.org wrote: > From: Michal Hocko > > __alloc_pages_high_priority doesn't do anything special other than it > calls get_page_from_freelist and loops around GFP_NOFAIL allocation > until it succeeds. It would be better if the first part was don

[PATCH 1/2] mm: get rid of __alloc_pages_high_priority

2015-11-16 Thread mhocko
From: Michal Hocko __alloc_pages_high_priority doesn't do anything special other than it calls get_page_from_freelist and loops around GFP_NOFAIL allocation until it succeeds. It would be better if the first part was done in __alloc_pages_slowpath where we modify the zonelist because this would b