Re: [Qemu-devel] [RFC 0/6] x86 CPU subclasses, v4

[Eduardo Habkost]

On Fri, Jan 04, 2013 at 05:56:16PM -0200, Eduardo Habkost wrote:
> This is a new RFC for the x86 CPU subclasses. This version handles the "host"
> subclass differently, and I believe this made the code much simpler.
> 
> However, a few tricks were necessary:
> 
>  - To keep the "default_kvm_features" compat functions working (this shouldn't
>    be necessary once we start using global properties for the compat code).
>  - For the vendor property setting (because it depends on a kvm_enabled() 
> check,
>    that can't be made on class_init). I don't know how we will be able to
>    convert that logic using global properties/defaults in the future.
>  - To keep the original model names in the "-cpu ?" output, for compatibility.
> 
> This series depends on multiple series:
>  - pc-1.4 kvm_mmu_opt disable series I sent today;
>  - -cpu "enforce" fix series I sent today;
>  - A experimental rebased version of Igor's "wave 2" x86 CPU init cleanup.
> 
> Because of the tricky dependencies, I recommend you use the git tree for 
> testing:
>   git://github.com/ehabkost/qemu-hacks.git x86-cpu-model-classes.RFC.v4
>   https://github.com/ehabkost/qemu-hacks/tree/x86-cpu-model-classes.RFC.v4

I submitted newer versions of the dependencies. There's a git branch of
this RFC rebased on top of the newer dependencies, here:

   git://github.com/ehabkost/qemu-hacks.git x86-cpu-model-classes.RFC.v5
   https://github.com/ehabkost/qemu-hacks/tree/x86-cpu-model-classes.RFC.v5

> 
> Eduardo Habkost (6):
>   target-i386: Move CPU object creation to cpu.c
>   target-i386: Make cpu_x86_create() get Error argument
>   target-i386: Simplify cpu_x86_find_by_name() logic
>   target-i386: Set feature string parsing results directly on CPU
>     object
>   target-i386: Move kvm_features/hypervisor initialization to
>     cpu_x86_find_by_name()
>   target-i386: CPU model subclasses
> 
>  include/sysemu/kvm.h  |    3 +
>  target-i386/cpu-qom.h |   26 +
>  target-i386/cpu.c     | 1548 
> ++++++++++++++++++++++++++++++-------------------
>  target-i386/cpu.h     |    4 +-
>  target-i386/helper.c  |   26 +-
>  target-i386/kvm.c     |    2 +
>  6 files changed, 988 insertions(+), 621 deletions(-)
> 
> -- 
> 1.7.11.7
> 
> 

-- 
Eduardo