qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PULL 07/37] cpus: extract out hax-specific code to target/i386/


From: Volker Rümelin
Subject: Re: [PULL 07/37] cpus: extract out hax-specific code to target/i386/
Date: Sat, 17 Oct 2020 09:17:54 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.2

>> Hi Claudio,
>>
>> is there a reason why you removed current_cpu = cpu; from 
>> hax_cpu_thread_fn() when you moved that function to target/i386/hax-cpus.c? 
>> This change broke HAX on Windows. Adding back that line makes it work again.
>
> Hello Volker, I see the change in the history and it was clearly an ugly 
> mistake on my part.
> There was no reason or intention to remove the current_cpu = cpu assignment
>
> The fix seems indeed to just + current_cpu = cpu;
> and I will send a patch momentarily that does just that,
>
> but I don't know of any CI coverage for Windows + hax currently,
> so it would be good if you could spin the change to verify that it fixes the 
> problem.
>

That was very quick. Only 5 hours from bug report to pull request. I've tested 
current master together with the patch from Paolo's PR. HAX on Windows works.

With best regards,
Volker



reply via email to

[Prev in Thread] Current Thread [Next in Thread]