On Tuesday 16 August 2011, Marek Szyprowski wrote:

On Friday, August 12, 2011 5:01 PM Arnd Bergmann wrote:

quoted

How about something like

      if (arch_is_coherent() || nommu())
              ret = alloc_simple_buffer();
      else if (arch_is_v4_v5())
              ret = alloc_remap();
      else if (gfp & GFP_ATOMIC)
              ret = alloc_from_pool();
      else
              ret = alloc_from_contiguous();

This also allows a natural conversion to dma_map_ops when we get there.

Ok. Is it ok to enable CMA permanently for ARMv6+? If CMA is left conditional
the dma pool code will be much more complicated, because it will need to support
both CMA and non-CMA cases.

I think that is ok, yes.

	Arnd

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>