lkml.org 
[lkml]   [2015]   [Feb]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 1/2] mm: remove GFP_THISNODE
On Wed, 25 Feb 2015, Christoph Lameter wrote:

> On Wed, 25 Feb 2015, David Rientjes wrote:
>
> > NOTE: this is not about __GFP_THISNODE, this is only about GFP_THISNODE.
>
> Well but then its not removing it. You are replacing it with an inline
> function.
>

Removing GFP_THISNODE, not __GFP_THISNODE. GFP_THISNODE, as the commit
message says, is a special collection of flags that means "never try
reclaim" and people confuse it for __GFP_THISNODE.

There are legitimate usecases where we want __GFP_THISNODE, in other words
restricting the allocation to only a specific node, and try reclaim but
not warn in failure or retry. The most notable example is in the followup
patch for thp, both for page faults and khugepaged, where we want to
target the local node but silently fallback to small pages instead.

This removes the special "no reclaim" behavior of __GFP_THISNODE |
__GFP_NORETRY | __GFP_NOWARN and relies on clearing __GFP_WAIT instead.


\
 
 \ /
  Last update: 2015-02-26 02:21    [W:0.160 / U:0.184 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site