On Saturday 08 July 2006 02:05, Christoph Lameter wrote:
> Make ZONE_DMA optional in the page allocator
Hmm, we should rename you the "ifdef warrior"
> - ifdef all code for ZONE_DMA and related definition
>
> - Without ZONE_DMA, ZONE_HIGHMEM and ZONE_DMA32 we fall back
> to an empty GFP_ZONEMASK and a ZONES_SHIFT of zero (since there
> is only one zone....).
>
> - We need to fix the use of ZONE_DMA in the memory policy layer.
> ZONE_DMA is used there as the first zone so use 0 instead.
Is the barely better code really worth all the ugliness from the ifdefs? I have doubts.
I think your idea of saving some cache lines by not having the unused
zones in the fallback lists etc. is a good one, but your current implementation
with its ifdef maze is extremly ugly. Surely this can be a done less
intrusively?
-Andi
-
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/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
