Re: [PATCH 3/4] Embed zone_id information within the zonelist->zones pointer

Andi Kleen
2007-08-13T22:58:41Z

On Mon, Aug 13, 2007 at 03:00:14PM -0700, Christoph Lameter wrote:
> You said that ZONE_DMA will still be there right? So the zone will be

There will be a (variable sized) dma zone, but not a ZONE_DMA entry in pgdat
or in the the fallback lists.

>
> > There are still other architectures that use it. Biggest offender
> > is s390. I'll leave them to their respective maintainers.
>
> IA64 also uses ZONE_DMA to support 32bit controllers.

ZONE_DMA32 I thought? That one is not changed.

>
> So I think we can only get rid of ZONE_DMA in its 16MB incarnation for
> i386 and x86_64.

Correct.

>
> But you will be keeping ZONE_DMA32?

Yes.

> If so then it may be better to drop ZONE_DMA32 and make ZONE_DMA be below
> 4GB like other 64bit arches.

That might be possible as a followup, but would change the driver
API. Is it worth it?

-Andi