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

Reply via email to