Re: [PATCH] mm: cma: honor __GFP_ZERO flag in cma_alloc()

From: Christoph Hellwig
Date: Wed Jun 13 2018 - 08:55:53 EST


On Wed, Jun 13, 2018 at 02:40:00PM +0200, Marek Szyprowski wrote:
> It is not only the matter of the spinlocks. GFP_ATOMIC is not supported
> by the
> memory compaction code, which is used in alloc_contig_range(). Right, this
> should be also noted in the documentation.

Documentation is good, asserts are better. The code should reject any
flag not explicitly supported, or even better have its own flags type
with the few actually supported flags.