Re: [PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-11-20 Thread Lee Schermerhorn
On Fri, 2007-11-09 at 14:33 +, Mel Gorman wrote: > Filtering zonelists requires very frequent use of zone_idx(). This is costly > as it involves a lookup of another structure and a substraction operation. As > the zone_idx is often required, it should be quickly accessible. The node > idx

Re: [PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-11-20 Thread Lee Schermerhorn
On Fri, 2007-11-09 at 14:33 +, Mel Gorman wrote: Filtering zonelists requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-11-09 Thread Mel Gorman
Filtering zonelists requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that accessing

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-11-09 Thread Mel Gorman
Filtering zonelists requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that accessing

Re: [PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-10-16 Thread David Rientjes
On Fri, 28 Sep 2007, Mel Gorman wrote: > > Filtering zonelists requires very frequent use of zone_idx(). This is costly > as it involves a lookup of another structure and a substraction operation. As > the zone_idx is often required, it should be quickly accessible. The node > idx could also be

Re: [PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-10-16 Thread David Rientjes
On Fri, 28 Sep 2007, Mel Gorman wrote: Filtering zonelists requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-28 Thread Mel Gorman
Filtering zonelists requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that accessing

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-28 Thread Mel Gorman
Filtering zonelists requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that accessing

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-13 Thread Mel Gorman
Using two zonelists per node requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-13 Thread Mel Gorman
Using two zonelists per node requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that

Re: [PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-12 Thread Christoph Lameter
On Wed, 12 Sep 2007, Mel Gorman wrote: > /* > + * This struct contains information about a zone in a zonelist. It is stored > + * here to avoid dereferences into large structures and lookups of tables > + */ > +struct zoneref { > + struct zone *zone; /* Pointer to actual zone */ > +

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-12 Thread Mel Gorman
Using two zonelists per node requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that

[PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-12 Thread Mel Gorman
Using two zonelists per node requires very frequent use of zone_idx(). This is costly as it involves a lookup of another structure and a substraction operation. As the zone_idx is often required, it should be quickly accessible. The node idx could also be stored here if it was found that

Re: [PATCH 4/6] Have zonelist contains structs with both a zone pointer and zone_idx

2007-09-12 Thread Christoph Lameter
On Wed, 12 Sep 2007, Mel Gorman wrote: /* + * This struct contains information about a zone in a zonelist. It is stored + * here to avoid dereferences into large structures and lookups of tables + */ +struct zoneref { + struct zone *zone; /* Pointer to actual zone */ + int