From: Jan Kiszka <jan.kiszka@siemens.com>
To: Alexander Graf <agraf@suse.de>
Cc: Avi Kivity <avi@redhat.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
Anthony Liguori <aliguori@us.ibm.com>,
Glauber Costa <glommer@redhat.com>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: Re: [PATCH 00/21] qemu-kvm: Hook cleanups and extended use of upstream code
Date: Tue, 02 Feb 2010 12:19:56 +0100 [thread overview]
Message-ID: <4B680A5C.7090102@siemens.com> (raw)
In-Reply-To: <15E55176-5115-41A5-8F36-923FE9FC093E@suse.de>
Alexander Graf wrote:
> On 02.02.2010, at 09:18, Jan Kiszka wrote:
>
>> Let's start with the overall stats:
>>
>> 31 files changed, 274 insertions(+), 822 deletions(-)
>>
>> So this series drops far more than 500 lines of redundant code, moving
>> qemu-kvm yet a bit closer to upstream.
>>
>> The other highlight is the simplification of synchronization between
>> in-kernel and user space VCPU states. This area used to call a lot of
>> problems in the past because it was tricky to get things right,
>> specifically during the multi-threaded startup. The new approach pushes
>> all the sync work around reset and vmsave/load into generic code, not
>> only removing the burden from developers of, say, in-kernel APIC
>> support, but also dropping most of our kvm-specific hooks, especially in
>> the qemu-kvm tree.
>>
>> While I tested this on various VMs around, and things look good so far,
>> I wouldn't be surprised if there are some regressions remaining,
>> specifically in the non-x86 parts that I wasn't able to test or even
>> build. Please have a careful look!
>
> The good news on that part is that apart from IA64, all other archs are broken in qemu-kvm anyways, but work on upstream qemu. So moving towards upstream definitely helps here.
>
OK, then you probably want my corresponding uq/master series in order to
test. Will try to roll them out ASAP.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
WARNING: multiple messages have this Message-ID (diff)
From: Jan Kiszka <jan.kiszka@siemens.com>
To: Alexander Graf <agraf@suse.de>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
Glauber Costa <glommer@redhat.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
Avi Kivity <avi@redhat.com>
Subject: [Qemu-devel] Re: [PATCH 00/21] qemu-kvm: Hook cleanups and extended use of upstream code
Date: Tue, 02 Feb 2010 12:19:56 +0100 [thread overview]
Message-ID: <4B680A5C.7090102@siemens.com> (raw)
In-Reply-To: <15E55176-5115-41A5-8F36-923FE9FC093E@suse.de>
Alexander Graf wrote:
> On 02.02.2010, at 09:18, Jan Kiszka wrote:
>
>> Let's start with the overall stats:
>>
>> 31 files changed, 274 insertions(+), 822 deletions(-)
>>
>> So this series drops far more than 500 lines of redundant code, moving
>> qemu-kvm yet a bit closer to upstream.
>>
>> The other highlight is the simplification of synchronization between
>> in-kernel and user space VCPU states. This area used to call a lot of
>> problems in the past because it was tricky to get things right,
>> specifically during the multi-threaded startup. The new approach pushes
>> all the sync work around reset and vmsave/load into generic code, not
>> only removing the burden from developers of, say, in-kernel APIC
>> support, but also dropping most of our kvm-specific hooks, especially in
>> the qemu-kvm tree.
>>
>> While I tested this on various VMs around, and things look good so far,
>> I wouldn't be surprised if there are some regressions remaining,
>> specifically in the non-x86 parts that I wasn't able to test or even
>> build. Please have a careful look!
>
> The good news on that part is that apart from IA64, all other archs are broken in qemu-kvm anyways, but work on upstream qemu. So moving towards upstream definitely helps here.
>
OK, then you probably want my corresponding uq/master series in order to
test. Will try to roll them out ASAP.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
next prev parent reply other threads:[~2010-02-02 11:20 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-02 8:18 [PATCH 00/21] qemu-kvm: Hook cleanups and extended use of upstream code Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 01/21] qemu-kvm: Drop vmport changes Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 02/21] KVM: Make vmport KVM-compatible Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 03/21] qemu-kvm: Clean up register access API Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 11:06 ` Gleb Natapov
2010-02-02 11:06 ` [Qemu-devel] " Gleb Natapov
2010-02-02 11:18 ` Jan Kiszka
2010-02-02 11:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 04/21] KVM: x86: Fix up misreported CPU features Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 05/21] qemu-kvm: Use upstream kvm_enabled and cpu_synchronize_state Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 06/21] qemu-kvm: Use upstream kvm_setup_guest_memory Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 07/21] qemu-kvm: Use some more upstream prototypes Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 08/21] qemu-kvm: Use upstream kvm_arch_get_supported_cpuid Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 09/21] qemu-kvm: Use upstream kvm_pit_in_kernel Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 10/21] KVM: Move and rename regs_modified Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 11/21] KVM: Rework of guest debug state writing Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 12/21] qemu-kvm: Use upstream kvm_vcpu_dirty Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:18 ` [PATCH 13/21] qemu-kvm: Use upstream guest debug code Jan Kiszka
2010-02-02 8:18 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 14/21] qemu-kvm: Rework VCPU state writeback API Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 15/21] qemu-kvm: Clean up mpstate synchronization Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 12:23 ` Gleb Natapov
2010-02-02 12:23 ` [Qemu-devel] " Gleb Natapov
2010-02-02 12:31 ` Jan Kiszka
2010-02-02 12:31 ` [Qemu-devel] " Jan Kiszka
2010-02-02 12:37 ` Gleb Natapov
2010-02-02 12:37 ` [Qemu-devel] " Gleb Natapov
2010-02-02 12:40 ` Jan Kiszka
2010-02-02 12:40 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 16/21] KVM: x86: Restrict writeback of VCPU state Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 17/21] qemu-kvm: Use VCPU event state for reset and vmsave/load Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 18/21] qemu-kvm: Cleanup/fix TSC and PV clock writeback Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 19/21] qemu-kvm: Clean up KVM's APIC hooks Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 20/21] qemu-kvm: Move kvm_set_boot_cpu_id Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 14:11 ` Gleb Natapov
2010-02-02 14:11 ` [Qemu-devel] " Gleb Natapov
2010-02-02 14:20 ` Jan Kiszka
2010-02-02 14:20 ` [Qemu-devel] " Jan Kiszka
2010-02-02 14:28 ` Gleb Natapov
2010-02-02 14:28 ` [Qemu-devel] " Gleb Natapov
2010-02-02 14:33 ` Jan Kiszka
2010-02-02 14:33 ` [Qemu-devel] " Jan Kiszka
2010-02-02 8:19 ` [PATCH 21/21] qemu-kvm: Bring qemu_init_vcpu back home Jan Kiszka
2010-02-02 8:19 ` [Qemu-devel] " Jan Kiszka
2010-02-02 10:52 ` [PATCH 00/21] qemu-kvm: Hook cleanups and extended use of upstream code Alexander Graf
2010-02-02 10:52 ` [Qemu-devel] " Alexander Graf
2010-02-02 11:19 ` Jan Kiszka [this message]
2010-02-02 11:19 ` Jan Kiszka
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B680A5C.7090102@siemens.com \
--to=jan.kiszka@siemens.com \
--cc=agraf@suse.de \
--cc=aliguori@us.ibm.com \
--cc=avi@redhat.com \
--cc=glommer@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mtosatti@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.