[PATCH 02/15] dma-contiguous: simplify cma_early_percent_memory()

Christoph Hellwig hch at lst.de
Tue Jul 28 02:37:49 EDT 2020


On Tue, Jul 28, 2020 at 08:11:40AM +0300, Mike Rapoport wrote:
> From: Mike Rapoport <rppt at linux.ibm.com>
> 
> The memory size calculation in cma_early_percent_memory() traverses
> memblock.memory rather than simply call memblock_phys_mem_size(). The
> comment in that function suggests that at some point there should have been
> call to memblock_analyze() before memblock_phys_mem_size() could be used.
> As of now, there is no memblock_analyze() at all and
> memblock_phys_mem_size() can be used as soon as cold-plug memory is
> registerd with memblock.
> 
> Replace loop over memblock.memory with a call to memblock_phys_mem_size().
> 
> Signed-off-by: Mike Rapoport <rppt at linux.ibm.com>

Looks good:

Reviewed-by: Christoph Hellwig <hch at lst.de>



More information about the linux-riscv mailing list