Home » Mailing list archives » linux-kernel » 2007 » October » 10

Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Lee Schermerhorn <Lee.Schermerhorn@...>
To: Mel Gorman <mel@...>
Cc: Nishanth Aravamudan <nacc@...>, <akpm@...>, <linux-kernel@...>, <linux-mm@...>, <rientjes@...>, <kamezawa.hiroyu@...>, <clameter@...>
Subject: Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask
Date: Wednesday, October 10, 2007 - 11:53 am

On Tue, 2007-10-09 at 16:40 +0100, Mel Gorman wrote:
<snip>
<snip>

Mel:  I applied this patch [to your v8 series--the most recent, I
think?] and it does fix the problem.  However, now I'm tripping over
this warning in __alloc_pages_nodemask:

	/* Specifying both __GFP_THISNODE and nodemask is stupid. Warn user */
	WARN_ON(gfp_mask & __GFP_THISNODE);

for each huge page allocated.  Rather slow as my console is a virtual
serial line and the warning includes the stack traceback.

I think we want to just drop this warning, but maybe you have a tighter
condition that you want to warn about?

Lee

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 0/6] Use one zonelist per node instead of multiple zo..., Mel Gorman, (Fri Sep 28, 10:23 am)
[PATCH 4/6] Have zonelist contains structs with both a zone ..., Mel Gorman, (Fri Sep 28, 10:24 am)
Re: [PATCH 4/6] Have zonelist contains structs with both a z..., David Rientjes, (Tue Oct 16, 11:22 pm)
[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask, Mel Gorman, (Fri Sep 28, 10:25 am)
Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_..., Lee Schermerhorn, (Fri Sep 28, 11:37 am)
Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_..., Mel Gorman, (Fri Sep 28, 2:28 pm)
Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_..., Lee Schermerhorn, (Fri Sep 28, 5:03 pm)
Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_..., Paul Jackson, (Fri Sep 28, 2:38 pm)
[PATCH 6/6] Use one zonelist that is filtered by nodemask, Mel Gorman, (Fri Sep 28, 10:25 am)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Nishanth Aravamudan, (Mon Oct 8, 9:11 pm)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Mel Gorman, (Tue Oct 9, 11:40 am)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Lee Schermerhorn, (Wed Oct 10, 11:53 am)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Mel Gorman, (Wed Oct 10, 12:09 pm)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Nishanth Aravamudan, (Wed Oct 10, 12:05 pm)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Nishanth Aravamudan, (Tue Oct 9, 2:12 pm)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Nishanth Aravamudan, (Tue Oct 9, 12:25 pm)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Christoph Lameter, (Tue Oct 9, 2:47 pm)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Christoph Lameter, (Mon Oct 8, 9:56 pm)
Re: [PATCH 6/6] Use one zonelist that is filtered by nodemask, Nishanth Aravamudan, (Mon Oct 8, 11:17 pm)
[PATCH 2/6] Introduce node_zonelist() for accessing the zone..., Mel Gorman, (Fri Sep 28, 10:24 am)
[PATCH 3/6] Use two zonelist that are filtered by GFP mask, Mel Gorman, (Fri Sep 28, 10:24 am)
[PATCH 1/6] Use zonelists instead of zones when direct recla..., Mel Gorman, (Fri Sep 28, 10:23 am)