On 23/05/2019 21:47, Igor Mammedov wrote:
On Wed,  8 May 2019 14:17:18 +0800
Tao Xu <tao3...@intel.com> wrote:

...
diff --git a/hw/arm/boot.c b/hw/arm/boot.c
index 8ff08814fd..845b737ab9 100644
--- a/hw/arm/boot.c
+++ b/hw/arm/boot.c
@@ -602,7 +602,7 @@ int arm_load_dtb(hwaddr addr, const struct arm_boot_info 
*binfo,
      if (nb_numa_nodes > 0) {
          mem_base = binfo->loader_start;
          for (i = 0; i < nb_numa_nodes; i++) {
-            mem_len = numa_info[i].node_mem;
+            mem_len = ms->numa_state->nodes[i].node_mem;
in 1/11 I've suggested to add nb_numa_nodes, but it might be to add
a pointer to MachineState there.
It would also help to simplify arm_load_dtb later as there are other
bits that we copy to arm_boot_info from MachineState.


Hi Igor,

Thank you for your review. I will simplify arm_load_dtb() in the next version of patch and improve the other issues.

Tao

Reply via email to