Series looks good to me.
Reviewed-By: Igor Mammedov <imamm...@redhat.com>


On 04/24/2012 11:33 AM, Andreas Färber wrote:
Hello,

This series introduces some QOM properties for X86CPU, so that our built-in
init code exercises the same code paths as QMP, as suggested by Eduardo:
* "family",
* "model",
* "stepping" and
* "model-id" (rather than "model_id")
This QOM'ifies my previously introduced helper functions, adding getters.

In the same spirit I've also introduced numeric QOM properties for:
* "level"
* "xlevel"
* "tsc-frequency" (rather than "tsc_freq")

Further I've prepared one QOM property that's currently unused:
* "vendor" (converting three words to string and back seemed too much overhead)

By constrast, the HyperV -cpu property "hv_spinlocks" and flags "hv_relaxed"
and "hv_vapic" do not seem to be per-CPU properties.

v2 fixes minor issues and if I can get an Acked-by or Reviewed-by for 09/15
then I'll send a PULL tomorrow.

Available from:
git://github.com/afaerber/qemu-cpu.git qom-cpu-x86-prop.v2
https://github.com/afaerber/qemu-cpu/commits/qom-cpu-x86-prop.v2

Regards,
Andreas

Cc: Anthony Liguori<anth...@codemonkey.ws>
Cc: Jan Kiszka<jan.kis...@siemens.com>
Cc: Igor Mammedov<imamm...@redhat.com>
Cc: Liu Jinsong<jinsong....@intel.com>
Cc: Lai Jiangshan<la...@cn.fujitsu.com>
Cc: Vasilis Liaskovitis<vasilis.liaskovi...@profitbricks.com>
Cc: Eduardo Habkost<ehabk...@redhat.com>
Cc: Michael Roth<mdr...@linux.vnet.ibm.com>
Cc: Paolo Bonzini<pbonz...@redhat.com>
Cc: Vadim Rozenfeld<vroze...@redhat.com>

Andreas Färber (15):
   target-i386: Fix x86_cpuid_set_model_id()
   target-i386: Pass X86CPU to cpu_x86_register()
   target-i386: Add range check for -cpu ,family=x
   target-i386: Add "family" property to X86CPU
   target-i386: Add "model" property to X86CPU
   target-i386: Add "stepping" property to X86CPU
   target-i386: Add "model-id" property to X86CPU
   target-i386: Add property getter for CPU family
   target-i386: Add property getter for CPU model
   target-i386: Add property getter for CPU stepping
   target-i386: Add property getter for CPU model-id
   target-i386: Introduce "level" property for X86CPU
   target-i386: Introduce "xlevel" property for X86CPU
   target-i386: Prepare "vendor" property for X86CPU
   target-i386: Introduce "tsc-frequency" property for X86CPU

  target-i386/cpu.c    |  320 +++++++++++++++++++++++++++++++++++++++++++++++---
  target-i386/cpu.h    |    2 +-
  target-i386/helper.c |    2 +-
  3 files changed, 304 insertions(+), 20 deletions(-)


--
-----
 Igor

Reply via email to