[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH v2 00/17] [uq/master] Prepare for more qemu-kvm
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH v2 00/17] [uq/master] Prepare for more qemu-kvm merging |
Date: |
Tue, 04 Jan 2011 09:42:05 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Am 03.01.2011 17:09, Avi Kivity wrote:
> On 01/03/2011 10:32 AM, Jan Kiszka wrote:
>> All previously sent patches against current uq combined, some smaller
>> tweaks applied, and 4 new patches added. Major news is support for
>> using up to 16M BIOSes and optional code dump for cpu_dump_state. Those
>> features are already present qemu-kvm but were reworked to provide
>> cleaner upstream versions.
>>
>
> Looks reasonable overall.
>
>> Based on this series, I've an experimental tree here where I eliminated
>> another 1500 LOC from qemu-kvm code. Specifically, that tree sets an end
>> to duplicate KVM and VCPU initialization functions, KVMState copies, and
>> redundant state saving/loading functions. Will be rolled out after some
>> more review and testing.
>
> Sounds really frightening... this glue code is a real breeding ground
> for subtle bugs and merge problems.
...and that's why we are more than late with unifying it into a single
code base.
After that upcoming cleanup round we will have three major remaining to-dos:
- in-kernel irqchip and PIT support with the subtopics
- VAPIC/TPR-patching support
- IRQ routing
- PCI device assignment
- VCPU & IO-thread execution model
While I'm not that concerned about the first two as they basically add
code that "just" needs to be refactored, the last one is worrying me. We
need to find a smart way to morph one into the other. The code is full
of subtle differences and is likely quite performance sensitive.
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] Re: [PATCH v2 17/17] kvm: Drop dependencies on very old capabilities, (continued)
[Qemu-devel] [PATCH v2 12/17] kvm: x86: Drop MCE MSRs write back restrictions, Jan Kiszka, 2011/01/03
[Qemu-devel] [PATCH v2 02/17] kvm: Drop return value of kvm_cpu_exec, Jan Kiszka, 2011/01/03
[Qemu-devel] [PATCH v2 03/17] kvm: Stop on all fatal exit reasons, Jan Kiszka, 2011/01/03
[Qemu-devel] [PATCH v2 04/17] kvm: Improve reporting of fatal errors, Jan Kiszka, 2011/01/03
[Qemu-devel] [PATCH 18/17] kvm: Flush coalesced mmio buffer on IO window exits, Jan Kiszka, 2011/01/03
[Qemu-devel] Re: [PATCH v2 00/17] [uq/master] Prepare for more qemu-kvm merging, Avi Kivity, 2011/01/03
- [Qemu-devel] Re: [PATCH v2 00/17] [uq/master] Prepare for more qemu-kvm merging,
Jan Kiszka <=