qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] Fix CPU topology initialization
@ 2010-01-05 16:26 Jiri Denemark
  2010-01-11 16:01 ` Anthony Liguori
  0 siblings, 1 reply; 2+ messages in thread
From: Jiri Denemark @ 2010-01-05 16:26 UTC (permalink / raw)
  To: qemu-devel

Late initialization of CPU topology in CPUState prevents KVM guests to
actually see the topology.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
---
 vl.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/vl.c b/vl.c
index e881e45..a03d7a6 100644
--- a/vl.c
+++ b/vl.c
@@ -3484,10 +3484,10 @@ void qemu_init_vcpu(void *_env)
 {
     CPUState *env = _env;
 
-    if (kvm_enabled())
-        kvm_init_vcpu(env);
     env->nr_cores = smp_cores;
     env->nr_threads = smp_threads;
+    if (kvm_enabled())
+        kvm_init_vcpu(env);
     return;
 }
 
@@ -3813,12 +3813,12 @@ void qemu_init_vcpu(void *_env)
 {
     CPUState *env = _env;
 
+    env->nr_cores = smp_cores;
+    env->nr_threads = smp_threads;
     if (kvm_enabled())
         kvm_start_vcpu(env);
     else
         tcg_init_vcpu(env);
-    env->nr_cores = smp_cores;
-    env->nr_threads = smp_threads;
 }
 
 void qemu_notify_event(void)
-- 
1.6.6

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [PATCH] Fix CPU topology initialization
  2010-01-05 16:26 [Qemu-devel] [PATCH] Fix CPU topology initialization Jiri Denemark
@ 2010-01-11 16:01 ` Anthony Liguori
  0 siblings, 0 replies; 2+ messages in thread
From: Anthony Liguori @ 2010-01-11 16:01 UTC (permalink / raw)
  To: Jiri Denemark; +Cc: qemu-devel

On 01/05/2010 10:26 AM, Jiri Denemark wrote:
> Late initialization of CPU topology in CPUState prevents KVM guests to
> actually see the topology.
>
> Signed-off-by: Jiri Denemark<jdenemar@redhat.com>
>    

Applied.  Thanks.

Regards,

Anthony Liguori
> ---
>   vl.c |    8 ++++----
>   1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/vl.c b/vl.c
> index e881e45..a03d7a6 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -3484,10 +3484,10 @@ void qemu_init_vcpu(void *_env)
>   {
>       CPUState *env = _env;
>
> -    if (kvm_enabled())
> -        kvm_init_vcpu(env);
>       env->nr_cores = smp_cores;
>       env->nr_threads = smp_threads;
> +    if (kvm_enabled())
> +        kvm_init_vcpu(env);
>       return;
>   }
>
> @@ -3813,12 +3813,12 @@ void qemu_init_vcpu(void *_env)
>   {
>       CPUState *env = _env;
>
> +    env->nr_cores = smp_cores;
> +    env->nr_threads = smp_threads;
>       if (kvm_enabled())
>           kvm_start_vcpu(env);
>       else
>           tcg_init_vcpu(env);
> -    env->nr_cores = smp_cores;
> -    env->nr_threads = smp_threads;
>   }
>
>   void qemu_notify_event(void)
>    

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-01-11 16:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-05 16:26 [Qemu-devel] [PATCH] Fix CPU topology initialization Jiri Denemark
2010-01-11 16:01 ` Anthony Liguori

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).