On Fri, 26 Jan 2007, Christoph Lameter wrote:
On Thu, 25 Jan 2007, Mel Gorman wrote:@@ -166,6 +168,8 @@ enum zone_type { #define ZONES_SHIFT 1 #elif __ZONE_COUNT <= 4 #define ZONES_SHIFT 2 +#elif __ZONE_COUNT <= 8 +#define ZONES_SHIFT 3 #elseYou do not need a shift of 3. Even with ZONE_MOVABLE the maximum number of zones is still 4. x86_64 has DMA, DMA32, NORMAL, MOVABLE i386 has DMA, NORMAL, HIGHMEM, MOVABLE x86_64 is the only platform that has DMA32.
Good point. I'll recheck this to be sure but if it's true, it means that the only major collision point between these patches and the optional ZONE_DMA patches goes away.
-- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

