On Thu, Mar 06, 2014 at 01:56:35PM -0800, Andrew Morton wrote: > On Thu, 6 Mar 2014 16:49:27 -0500 Johannes Weiner <hannes@xxxxxxxxxxx> wrote: > > > On Thu, Mar 06, 2014 at 12:37:57PM -0800, akpm@xxxxxxxxxxxxxxxxxxxx wrote: > > > Subject: [merged] mm-page_alloc-reset-aging-cycle-with-gfp_thisnode-v2.patch removed from -mm tree > > > To: hannes@xxxxxxxxxxx,jstancek@xxxxxxxxxx,mgorman@xxxxxxx,riel@xxxxxxxxxx,stable@xxxxxxxxxx,mm-commits@xxxxxxxxxxxxxxx > > > From: akpm@xxxxxxxxxxxxxxxxxxxx > > > Date: Thu, 06 Mar 2014 12:37:57 -0800 > > > > > > > > > The patch titled > > > Subject: mm: page_alloc: exempt GFP_THISNODE allocations from zone fairness > > > has been removed from the -mm tree. Its filename was > > > mm-page_alloc-reset-aging-cycle-with-gfp_thisnode-v2.patch > > > > > > This patch was dropped because it was merged into mainline or a subsystem tree > > > > Would it make sense to also merge > > > > mm-fix-gfp_thisnode-callers-and-clarify.patch > > > > at this point? It's not as critical as the GFP_THISNODE exemption, > > which is why I didn't tag it for stable, but it's a bugfix as well. > > Changelog fail! > > : GFP_THISNODE is for callers that implement their own clever fallback to > : remote nodes, and so no direct reclaim is invoked. There are many current > : users that only want node exclusiveness but still want reclaim to make the > : allocation happen. Convert them over to __GFP_THISNODE and update the > : documentation to clarify GFP_THISNODE semantics. > > what bug does it fix and what are the user-visible effects?? Ok, maybe this is better? --- GFP_THISNODE is for callers that implement their own clever fallback to remote nodes. It restricts the allocation to the specified node and does not invoke reclaim, assuming that the caller will take care of it when the fallback fails, e.g. through a subsequent allocation request without GFP_THISNODE set. However, many current GFP_THISNODE users only want the node exclusive aspect of the flag, without actually implementing their own fallback or triggering reclaim if necessary. This results in things like page migration failing prematurely even when there is easily reclaimable memory available, unless kswapd happens to be running already or a concurrent allocation attempt triggers the necessary reclaim. Convert all callsites that don't implement their own fallback strategy to __GFP_THISNODE. This restricts the allocation a single node too, but at the same time allows the allocator to enter the slowpath, wake kswapd, and invoke direct reclaim if necessary, to make the allocation happen when memory is full. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>