[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v4] numa: enable sparse node numbering on pp
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [RFC PATCH v4] numa: enable sparse node numbering on ppc |
Date: |
Tue, 1 Jul 2014 12:24:48 -0300 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, Jun 30, 2014 at 02:34:56PM -0700, Nishanth Aravamudan wrote:
[...]
> }
> + assert(numa_info[i].present);
> numa_info[i].node_mem = ram_size - usedmem;
> }
Tab here.
>
[...]
> + for (i = 0, j = 0; i < max_cpus; i++) {
> + do {
> + j = (j + 1) % (max_numa_nodeid);
> + } while (!numa_info[j].present);
Ditto.
(Those can be detected using checkpatch.pl).
Other than that, the PC changes look good to me (I haven't reviewed the
spapr.c changes). But I have an additional request: can you submit the
nb_numa_nodes/num_numa_nodes rename as a separate patch, to make the
other changes more visible and easier to review?
--
Eduardo
- Re: [Qemu-devel] [RFC PATCH v4] numa: enable sparse node numbering on ppc,
Eduardo Habkost <=