On 16.06.2014 [17:53:49 +1000], Alexey Kardashevskiy wrote:
> Current QEMU does not support memoryless NUMA nodes.
> This prepares SPAPR for that.
>
> This moves 2 calls of spapr_populate_memory_node() into
> the existing loop which handles nodes other than than
> the first one.
<snip>
> @@ -719,6 +704,12 @@ static int spapr_populate_memory(sPAPREnvironment
> *spapr, void *fdt)
> node_size = ram_size - mem_start;
> }
> }
> + if (!mem_start) {
> + /* ppc_spapr_init() checks for rma_size <= node0_size already */
> + spapr_populate_memory_node(fdt, i, 0, spapr->rma_size);
> + mem_start += spapr->rma_size;
> + node_size -= spapr->rma_size;
> + }
Why is this needed to be separate? The RMA fits in the first node, per
the comment and the prior checks, so can't we just leave the first node
alone?
-Nish
