PG_DMA

In 2.3.32-pre, I see that the PageDMA(page) macro has been changed to

#define PageDMA(page) (contig_page_data.node_zones + ZONE_DMA == (page)->zone)

Why was this done? I would still prefer to see the PG_DMA bit, because for discontig platforms, there is not a "contig_page_data". In short, this will break any platform that does use the CONFIG_DISCONTIGMEM code.

Thanks.

Kanoj
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://humbolt.geo.uu.nl/Linux-MM/

Actually, the REAL reason this was done is that PageDMA should just go away completely.

If you grep for where it is actually used, you'll see that it's only used in the page freeing code to see if this is a page we're interested in freeing. And that needs to be eithe rremoved or revamped anyway - whether it's going to be per-zone or global or whatever, the current test is just noth worth it (what KIND of DMA? Is it the 20-bit ISA DMA or the 31-bit broken PCI dma or the 32-bit real PCI dma or what?)

The DMA'ness of the page should be encoded as just the kind of zone it is part of.

Linus

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://humbolt.geo.uu.nl/Linux-MM/