From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marcelo Tosatti <mtosatti@redhat.com>
Subject: Re: [PATCH v2 00/21] qemu-kvm: Hook cleanups and extended use of
 upstream code
Date: Wed, 3 Feb 2010 14:50:51 -0200
Message-ID: <20100203165051.GA29235@amt.cnet>
References: <cover.1265187223.git.jan.kiszka@siemens.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Avi Kivity <avi@redhat.com>, kvm@vger.kernel.org,
	Anthony Liguori <aliguori@us.ibm.com>,
	Alexander Graf <agraf@suse.de>,
	Glauber Costa <glommer@redhat.com>, qemu-devel@nongnu.org
To: Jan Kiszka <jan.kiszka@siemens.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from mx1.redhat.com ([209.132.183.28]:43393 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1757450Ab0BCQvv (ORCPT <rfc822;kvm@vger.kernel.org>);
	Wed, 3 Feb 2010 11:51:51 -0500
Content-Disposition: inline
In-Reply-To: <cover.1265187223.git.jan.kiszka@siemens.com>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On Wed, Feb 03, 2010 at 09:53:25AM +0100, Jan Kiszka wrote:
> This version addresses the feedback on v2, namely:
> - assert(<vm stopped> || <current thread == env->thread>) on low-level
>   load/save registers
> - fixed mpstate initialization
> 
> Yet untested is -no-kvm-irqchip with smp due to some bug in unpatched
> qemu-kvm or the kernel modules. Still investigating.

Don't recall it ever working properly. Guess that IPI emulation with
signals is problematic.

> 
> Pull URL is still
> 
> 	git://git.kiszka.org/qemu-kvm.git queues/vcpu-state
> 
> PS: The corresponding upstream queue is now available under
> queues/kvm-upstream in the same repository. Will send it out later if
> there are no further remarks on this series to avoid flooding the
> mailing lists. Early testers are nevertheless welcome.

Can the necessary changes for cleanups go in qemu.git first, so after
they're merged in qemu-kvm, we can apply the unification patches.

And later do the same with synchronization rework (API looks fine to
me).

Nice work!