Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-08 Thread Michal Hocko
On Tue 07-03-17 10:57:48, Matthew Wilcox wrote: > On Tue, Mar 07, 2017 at 10:28:41AM -0800, Matthew Wilcox wrote: > > On Tue, Mar 07, 2017 at 03:10:20PM +0100, Michal Hocko wrote: > > > This patch simply uses __GFP_HIGHMEM implicitly when allocating pages to > > > be mapped to the vmalloc space. Cu

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-08 Thread Michal Hocko
On Tue 07-03-17 15:08:45, Andrew Morton wrote: > On Tue, 7 Mar 2017 15:10:20 +0100 Michal Hocko wrote: > > > __vmalloc* allows users to provide gfp flags for the underlying > > allocation. This API is quite popular > > $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l > > 7

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-08 Thread Michal Hocko
On Wed 08-03-17 08:33:58, Vlastimil Babka wrote: > On 03/07/2017 03:10 PM, Michal Hocko wrote: [...] > > index dece26f119d4..a804a4107fbc 100644 > > --- a/drivers/block/drbd/drbd_bitmap.c > > +++ b/drivers/block/drbd/drbd_bitmap.c > > @@ -409,7 +409,7 @@ static struct page **bm_realloc_pages(struct

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-08 Thread Vlastimil Babka
On 03/07/2017 03:10 PM, Michal Hocko wrote: > From: Michal Hocko > > __vmalloc* allows users to provide gfp flags for the underlying > allocation. This API is quite popular > $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l > 77 > > the only problem is that many people are

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-07 Thread Vlastimil Babka
On 03/07/2017 07:57 PM, Matthew Wilcox wrote: > On Tue, Mar 07, 2017 at 10:28:41AM -0800, Matthew Wilcox wrote: >> On Tue, Mar 07, 2017 at 03:10:20PM +0100, Michal Hocko wrote: >>> This patch simply uses __GFP_HIGHMEM implicitly when allocating pages to >>> be mapped to the vmalloc space. Current u

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-07 Thread Matthew Wilcox
On Tue, Mar 07, 2017 at 03:08:45PM -0800, Andrew Morton wrote: > On Tue, 7 Mar 2017 15:10:20 +0100 Michal Hocko wrote: > > > __vmalloc* allows users to provide gfp flags for the underlying > > allocation. This API is quite popular > > $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmallo

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-07 Thread Andrew Morton
On Tue, 7 Mar 2017 15:10:20 +0100 Michal Hocko wrote: > __vmalloc* allows users to provide gfp flags for the underlying > allocation. This API is quite popular > $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l > 77 > > the only problem is that many people are not aware t

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-07 Thread Matthew Wilcox
On Tue, Mar 07, 2017 at 10:28:41AM -0800, Matthew Wilcox wrote: > On Tue, Mar 07, 2017 at 03:10:20PM +0100, Michal Hocko wrote: > > This patch simply uses __GFP_HIGHMEM implicitly when allocating pages to > > be mapped to the vmalloc space. Current users which add __GFP_HIGHMEM > > are simplified a

Re: [PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-07 Thread Matthew Wilcox
On Tue, Mar 07, 2017 at 03:10:20PM +0100, Michal Hocko wrote: > From: Michal Hocko > > __vmalloc* allows users to provide gfp flags for the underlying > allocation. This API is quite popular > $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l > 77 > > the only problem is th

[PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-03-07 Thread Michal Hocko
From: Michal Hocko __vmalloc* allows users to provide gfp flags for the underlying allocation. This API is quite popular $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l 77 the only problem is that many people are not aware that they really want to give __GFP_HIGHMEM along

Re: [RFC PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-02-07 Thread Michal Hocko
On Tue 07-02-17 15:24:14, Vlastimil Babka wrote: > On 02/01/2017 03:05 PM, Michal Hocko wrote: > > From: Michal Hocko > > > > __vmalloc* allows users to provide gfp flags for the underlying > > allocation. This API is quite popular > > $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmallo

Re: [RFC PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-02-07 Thread Vlastimil Babka
On 02/01/2017 03:05 PM, Michal Hocko wrote: From: Michal Hocko __vmalloc* allows users to provide gfp flags for the underlying allocation. This API is quite popular $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l 77 the only problem is that many people are not aware tha

Re: [RFC PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-02-07 Thread Michal Hocko
On Wed 01-02-17 15:05:30, Michal Hocko wrote: > From: Michal Hocko > > __vmalloc* allows users to provide gfp flags for the underlying > allocation. This API is quite popular > $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l > 77 > > the only problem is that many people a

[RFC PATCH] mm, vmalloc: use __GFP_HIGHMEM implicitly

2017-02-01 Thread Michal Hocko
From: Michal Hocko __vmalloc* allows users to provide gfp flags for the underlying allocation. This API is quite popular $ git grep "=[[:space:]]__vmalloc\|return[[:space:]]*__vmalloc" | wc -l 77 the only problem is that many people are not aware that they really want to give __GFP_HIGHMEM along