[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-11-09 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-11-09 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Lee Schermerhorn
On Fri, 2007-09-28 at 19:28 +0100, Mel Gorman wrote: > On (28/09/07 11:37), Lee Schermerhorn didst pronounce: > > Still need to fix 'nodes_intersect' -> 'nodes_intersects'. See below. > > > > > diff -rup -X /usr/src/patchset-0.6/bin//dontdiff > > >

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Paul Jackson
Mel replied to Lee: > > > + return nodes_intersect(nodemask, current->mems_allowed); > > ^^^ -- should be nodes_intersects, I think. > > Crap, you're right, I missed the warning about implicit declarations. I > apologise. This is the corrected version I found myself

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Mel Gorman
On (28/09/07 11:37), Lee Schermerhorn didst pronounce: > Still need to fix 'nodes_intersect' -> 'nodes_intersects'. See below. > > On Fri, 2007-09-28 at 15:25 +0100, Mel Gorman wrote: > > The MPOL_BIND policy creates a zonelist that is used for allocations > > belonging > > to that thread that

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Lee Schermerhorn
Still need to fix 'nodes_intersect' -> 'nodes_intersects'. See below. On Fri, 2007-09-28 at 15:25 +0100, Mel Gorman wrote: > The MPOL_BIND policy creates a zonelist that is used for allocations belonging > to that thread that can use the policy_zone. As the per-node zonelist is > already being

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Lee Schermerhorn
Still need to fix 'nodes_intersect' - 'nodes_intersects'. See below. On Fri, 2007-09-28 at 15:25 +0100, Mel Gorman wrote: The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Mel Gorman
On (28/09/07 11:37), Lee Schermerhorn didst pronounce: Still need to fix 'nodes_intersect' - 'nodes_intersects'. See below. On Fri, 2007-09-28 at 15:25 +0100, Mel Gorman wrote: The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Paul Jackson
Mel replied to Lee: + return nodes_intersect(nodemask, current-mems_allowed); ^^^ -- should be nodes_intersects, I think. Crap, you're right, I missed the warning about implicit declarations. I apologise. This is the corrected version I found myself making

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-28 Thread Lee Schermerhorn
On Fri, 2007-09-28 at 19:28 +0100, Mel Gorman wrote: On (28/09/07 11:37), Lee Schermerhorn didst pronounce: Still need to fix 'nodes_intersect' - 'nodes_intersects'. See below. snip diff -rup -X /usr/src/patchset-0.6/bin//dontdiff

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-13 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-13 Thread Lee Schermerhorn
On Wed, 2007-09-12 at 22:06 +0100, Mel Gorman wrote: > The MPOL_BIND policy creates a zonelist that is used for allocations belonging > to that thread that can use the policy_zone. As the per-node zonelist is > already being filtered based on a zone id, this patch adds a version of >

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-13 Thread Mel Gorman
On (12/09/07 14:23), Christoph Lameter didst pronounce: > On Wed, 12 Sep 2007, Mel Gorman wrote: > > > - z++) > > - ; > > + if (likely(nodes == NULL)) > > + for (; zonelist_zone_idx(z) > highest_zoneidx; > > + z++) > > +

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-13 Thread Mel Gorman
On (12/09/07 14:23), Christoph Lameter didst pronounce: On Wed, 12 Sep 2007, Mel Gorman wrote: - z++) - ; + if (likely(nodes == NULL)) + for (; zonelist_zone_idx(z) highest_zoneidx; + z++) + ; +

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-13 Thread Lee Schermerhorn
On Wed, 2007-09-12 at 22:06 +0100, Mel Gorman wrote: The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages()

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-13 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-12 Thread Christoph Lameter
On Wed, 12 Sep 2007, Mel Gorman wrote: > - z++) > - ; > + if (likely(nodes == NULL)) > + for (; zonelist_zone_idx(z) > highest_zoneidx; > + z++) > + ; > + else > + for (;

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-12 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-12 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-12 Thread Christoph Lameter
On Wed, 12 Sep 2007, Mel Gorman wrote: - z++) - ; + if (likely(nodes == NULL)) + for (; zonelist_zone_idx(z) highest_zoneidx; + z++) + ; + else + for (; zonelist_zone_idx(z)

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-11 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-11 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-11 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-09-11 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the per-node zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-08-21 Thread Mel Gorman
On (17/08/07 14:29), Christoph Lameter didst pronounce: > On Fri, 17 Aug 2007, Mel Gorman wrote: > > > @@ -696,6 +696,16 @@ static inline struct zonelist *node_zone > > return _DATA(nid)->node_zonelist; > > } > > > > +static inline int zone_in_nodemask(unsigned long zone_addr, > > +

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-08-21 Thread Mel Gorman
On (17/08/07 14:29), Christoph Lameter didst pronounce: On Fri, 17 Aug 2007, Mel Gorman wrote: @@ -696,6 +696,16 @@ static inline struct zonelist *node_zone return NODE_DATA(nid)-node_zonelist; } +static inline int zone_in_nodemask(unsigned long zone_addr, +

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-08-17 Thread Christoph Lameter
On Fri, 17 Aug 2007, Mel Gorman wrote: > @@ -696,6 +696,16 @@ static inline struct zonelist *node_zone > return _DATA(nid)->node_zonelist; > } > > +static inline int zone_in_nodemask(unsigned long zone_addr, > + nodemask_t *nodes) > +{ > +#ifdef CONFIG_NUMA >

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-08-17 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This eliminates the

[PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-08-17 Thread Mel Gorman
The MPOL_BIND policy creates a zonelist that is used for allocations belonging to that thread that can use the policy_zone. As the zonelist is already being filtered based on a zone id, this patch adds a version of __alloc_pages() that takes a nodemask for further filtering. This eliminates the

Re: [PATCH 5/6] Filter based on a nodemask as well as a gfp_mask

2007-08-17 Thread Christoph Lameter
On Fri, 17 Aug 2007, Mel Gorman wrote: @@ -696,6 +696,16 @@ static inline struct zonelist *node_zone return NODE_DATA(nid)-node_zonelist; } +static inline int zone_in_nodemask(unsigned long zone_addr, + nodemask_t *nodes) +{ +#ifdef CONFIG_NUMA +