qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] hax: unbreak accelerator cpu code after cpus.c split
@ 2020-10-16  8:00 Claudio Fontana
  2020-10-16  8:08 ` Paolo Bonzini
  2020-10-16  8:23 ` Philippe Mathieu-Daudé
  0 siblings, 2 replies; 3+ messages in thread
From: Claudio Fontana @ 2020-10-16  8:00 UTC (permalink / raw)
  To: Volker Rümelin, Paolo Bonzini
  Cc: Richard Henderson, haxm-team, Claudio Fontana, qemu-devel

during my split of cpus.c, code line
"current_cpu = cpu"
was removed by mistake, causing hax to break.

This commit fixes the situation restoring it.

Reported-by: Volker Rümelin <vr_qemu@t-online.de>
Fixes: e92558e4bf8059ce4f0b310afe218802b72766bc
Signed-off-by: Claudio Fontana <cfontana@suse.de>
---
 target/i386/hax-cpus.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/target/i386/hax-cpus.c b/target/i386/hax-cpus.c
index 99770e590c..f72c85bd49 100644
--- a/target/i386/hax-cpus.c
+++ b/target/i386/hax-cpus.c
@@ -38,6 +38,7 @@ static void *hax_cpu_thread_fn(void *arg)
     qemu_thread_get_self(cpu->thread);
 
     cpu->thread_id = qemu_get_thread_id();
+    current_cpu = cpu;
     hax_init_vcpu(cpu);
     cpu_thread_signal_created(cpu);
     qemu_guest_random_seed_thread_part2(cpu->random_seed);
-- 
2.26.2



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

* Re: [PATCH] hax: unbreak accelerator cpu code after cpus.c split
  2020-10-16  8:00 [PATCH] hax: unbreak accelerator cpu code after cpus.c split Claudio Fontana
@ 2020-10-16  8:08 ` Paolo Bonzini
  2020-10-16  8:23 ` Philippe Mathieu-Daudé
  1 sibling, 0 replies; 3+ messages in thread
From: Paolo Bonzini @ 2020-10-16  8:08 UTC (permalink / raw)
  To: Claudio Fontana, Volker Rümelin
  Cc: Richard Henderson, haxm-team, qemu-devel

On 16/10/20 10:00, Claudio Fontana wrote:
> during my split of cpus.c, code line
> "current_cpu = cpu"
> was removed by mistake, causing hax to break.
> 
> This commit fixes the situation restoring it.
> 
> Reported-by: Volker Rümelin <vr_qemu@t-online.de>
> Fixes: e92558e4bf8059ce4f0b310afe218802b72766bc
> Signed-off-by: Claudio Fontana <cfontana@suse.de>
> ---
>  target/i386/hax-cpus.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/target/i386/hax-cpus.c b/target/i386/hax-cpus.c
> index 99770e590c..f72c85bd49 100644
> --- a/target/i386/hax-cpus.c
> +++ b/target/i386/hax-cpus.c
> @@ -38,6 +38,7 @@ static void *hax_cpu_thread_fn(void *arg)
>      qemu_thread_get_self(cpu->thread);
>  
>      cpu->thread_id = qemu_get_thread_id();
> +    current_cpu = cpu;
>      hax_init_vcpu(cpu);
>      cpu_thread_signal_created(cpu);
>      qemu_guest_random_seed_thread_part2(cpu->random_seed);
> 

Queued, thanks.

Paolo



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

* Re: [PATCH] hax: unbreak accelerator cpu code after cpus.c split
  2020-10-16  8:00 [PATCH] hax: unbreak accelerator cpu code after cpus.c split Claudio Fontana
  2020-10-16  8:08 ` Paolo Bonzini
@ 2020-10-16  8:23 ` Philippe Mathieu-Daudé
  1 sibling, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-10-16  8:23 UTC (permalink / raw)
  To: Claudio Fontana
  Cc: Paolo Bonzini, haxm-team, Volker Rümelin, Richard Henderson,
	qemu-devel

[-- Attachment #1: Type: text/plain, Size: 1047 bytes --]

Le ven. 16 oct. 2020 10:03, Claudio Fontana <cfontana@suse.de> a écrit :

> during my split of cpus.c, code line
> "current_cpu = cpu"
> was removed by mistake, causing hax to break.
>
> This commit fixes the situation restoring it.
>
> Reported-by: Volker Rümelin <vr_qemu@t-online.de>
> Fixes: e92558e4bf8059ce4f0b310afe218802b72766bc
> Signed-off-by: Claudio Fontana <cfontana@suse.de>
>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

---
>  target/i386/hax-cpus.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/target/i386/hax-cpus.c b/target/i386/hax-cpus.c
> index 99770e590c..f72c85bd49 100644
> --- a/target/i386/hax-cpus.c
> +++ b/target/i386/hax-cpus.c
> @@ -38,6 +38,7 @@ static void *hax_cpu_thread_fn(void *arg)
>      qemu_thread_get_self(cpu->thread);
>
>      cpu->thread_id = qemu_get_thread_id();
> +    current_cpu = cpu;
>      hax_init_vcpu(cpu);
>      cpu_thread_signal_created(cpu);
>      qemu_guest_random_seed_thread_part2(cpu->random_seed);
> --
> 2.26.2
>
>
>

[-- Attachment #2: Type: text/html, Size: 1852 bytes --]

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

end of thread, other threads:[~2020-10-16  8:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-10-16  8:00 [PATCH] hax: unbreak accelerator cpu code after cpus.c split Claudio Fontana
2020-10-16  8:08 ` Paolo Bonzini
2020-10-16  8:23 ` Philippe Mathieu-Daudé

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).