Re: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath

2016-06-01 Thread Joonsoo Kim
gt;>Subject: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath > >> > >>In __alloc_pages_slowpath(), alloc_flags doesn't change after it's > >>initialized, > >>so move the initialization above the retry: label. Also make the commen

Re: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath

2016-05-31 Thread Vlastimil Babka
On 05/31/2016 08:20 AM, Joonsoo Kim wrote: >From 68f09f1d4381c7451238b4575557580380d8bf30 Mon Sep 17 00:00:00 2001 From: Vlastimil Babka Date: Fri, 29 Apr 2016 11:51:17 +0200 Subject: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath In __alloc_pages_slowpath(), alloc_fl

Re: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath

2016-05-30 Thread Joonsoo Kim
f correct. Note it causes a conflict in > patch 03/13 but it's simple to resolve. > > Thanks > > 8< > >From 68f09f1d4381c7451238b4575557580380d8bf30 Mon Sep 17 00:00:00 2001 > From: Vlastimil Babka > Date: Fri, 29 Apr 2016 11:51:17 +0200 > Subject: [RF

Re: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath

2016-05-12 Thread Michal Hocko
On Tue 10-05-16 14:30:11, Vlastimil Babka wrote: [...] > >From 68f09f1d4381c7451238b4575557580380d8bf30 Mon Sep 17 00:00:00 2001 > From: Vlastimil Babka > Date: Fri, 29 Apr 2016 11:51:17 +0200 > Subject: [RFC 02/13] mm, page_alloc: set alloc_flags only once in s

Re: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath

2016-05-10 Thread Vlastimil Babka
n Sep 17 00:00:00 2001 From: Vlastimil Babka Date: Fri, 29 Apr 2016 11:51:17 +0200 Subject: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath In __alloc_pages_slowpath(), alloc_flags doesn't change after it's initialized, so move the initialization above the retr

Re: [RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath

2016-05-10 Thread Tetsuo Handa
Vlastimil Babka wrote: > In __alloc_pages_slowpath(), alloc_flags doesn't change after it's > initialized, > so move the initialization above the retry: label. Also make the comment above > the initialization more descriptive. Not true. gfp_to_alloc_flags() will include ALLOC_NO_WATERMARKS if cur

[RFC 02/13] mm, page_alloc: set alloc_flags only once in slowpath

2016-05-10 Thread Vlastimil Babka
In __alloc_pages_slowpath(), alloc_flags doesn't change after it's initialized, so move the initialization above the retry: label. Also make the comment above the initialization more descriptive. Signed-off-by: Vlastimil Babka --- mm/page_alloc.c | 14 +++--- 1 file changed, 7 insertions