diff options
author | Dou Liyang <douly.fnst@cn.fujitsu.com> | 2017-01-17 22:42:31 +0800 |
---|---|---|
committer | Eduardo Habkost <ehabkost@redhat.com> | 2017-01-23 21:25:36 -0200 |
commit | 9c6703fe82b29909cf2cf35b192892327841f71a (patch) | |
tree | 8c8083b0bc5e685f572824c1519899815d54d164 /vl.c | |
parent | bd72159db41d3d3c5f9afb1671d0320a77167569 (diff) | |
download | qemu-9c6703fe82b29909cf2cf35b192892327841f71a.zip qemu-9c6703fe82b29909cf2cf35b192892327841f71a.tar.gz qemu-9c6703fe82b29909cf2cf35b192892327841f71a.tar.bz2 |
vl: Ensure the numa_post_machine_init func in the appropriate location
In the numa_post_machine_init(), we use CPU_FOREACH macro to set all
CPUs' namu_node. So, we should make sure that we call it after Qemu
has already initialied all the CPUs.
As we all know, the CPUs can be created by "-smp"(pc_new_cpu) or
"-device"(qdev_device_add) command. But, before the device init,
Qemu execute the numa_post_machine_init earlier. It makes the mapping
of NUMA nodes and CPUs incorrect.
The patch move the numa_post_machine_init func in the appropriate
location.
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Message-Id: <1484664152-24446-2-git-send-email-douly.fnst@cn.fujitsu.com>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Diffstat (limited to 'vl.c')
-rw-r--r-- | vl.c | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -4513,8 +4513,6 @@ int main(int argc, char **argv, char **envp) cpu_synchronize_all_post_init(); - numa_post_machine_init(); - if (hax_enabled()) { hax_sync_vcpus(); } @@ -4539,6 +4537,9 @@ int main(int argc, char **argv, char **envp) device_init_func, NULL, NULL)) { exit(1); } + + numa_post_machine_init(); + rom_reset_order_override(); /* Did we create any drives that we failed to create a device for? */ |