Re: [Qemu-devel] [PATCH v2 0/11]

[Glauber Costa <glommer@gmail.com>]

On Wed, Dec 2, 2009 at 8:59 AM, Gleb Natapov <gleb@redhat.com> wrote:
> On Tue, Dec 01, 2009 at 10:51:26AM -0200, Glauber Costa wrote:
>> This is a repost of the -smp series. Note that it depends on irqchip-in-kernel,
>> that is already in staging. Also, you'll have to enable the io-thread, for the time
>> being.
>>
>> >From the last version, main change is that I am not calling queue_work automatically
>> from vcpu ioctls. queue_work is only used currently for the gdb stub.
>>
>> All other uses were by-passed by the new qemu_register_vcpu_reset(), since most
>> of it uses (all racy) came from the reset handlers.
>>
> Looks good to me except one thing. I don't see how you are addressing
> the problem fixed by commit b8a7857071b477b28d3055e33ff0298fc91f329a
> in qemu-kvm. The problem is that mp_state can change in kernel between
> call to kvm_cpu_synchronize_state() and vcpu re-entry. In this case old
> mp_state will overwrite new one.

I plan to do it in a patch that already lives on my tree. It basically flushes
register state in early post_load


-- 
Glauber  Costa.
"Free as in Freedom"
http://glommer.net

"The less confident you are, the more serious you have to act."