Re: [PATCH 4/8] mm: MIGRATE_CMA migration type added

From: Russell King - ARM Linux
Date: Tue Jul 05 2011 - 08:28:21 EST


On Tue, Jul 05, 2011 at 01:44:31PM +0200, Arnd Bergmann wrote:
> > @@ -198,6 +198,12 @@ config MIGRATION
> > pages as migration can relocate pages to satisfy a huge page
> > allocation instead of reclaiming.
> >
> > +config CMA_MIGRATE_TYPE
> > + bool
> > + help
> > + This enables the use the MIGRATE_CMA migrate type, which lets lets CMA
> > + work on almost arbitrary memory range and not only inside ZONE_MOVABLE.
> > +
> > config PHYS_ADDR_T_64BIT
> > def_bool 64BIT || ARCH_PHYS_ADDR_T_64BIT
>
> This is currently only selected on ARM with your patch set.

That's because CMA is targeted at solving the "we need massive contiguous
DMA areas" problem on ARM SoCs.

And it does this without addressing the technical architecture problems
surrounding multiple aliasing mappings with differing attributes which
actually make it unsuitable for use on ARM. This is not the first time
I've pointed that out, and I'm now at the point of basically ignoring
this CMA work because I'm tired of constantly pointing this out.

My silence on this subject must not be taken as placid acceptance of the
approach, but revulsion at seemingly being constantly ignored and having
these patches pushed time and time again with nothing really changing on
that issue.

It will be a sad day if these patches make their way into mainline without
that being addressed, and will show contempt for architecture maintainers
if it does.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/