[Qemu-devel] [PATCH v2 00/15] QOM'ify x86 CPU, part 2: properties
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(-) -- 1.7.7
Re: [Qemu-devel] [PATCH v2 00/15] QOM'ify x86 CPU, part 2: properties
Am 24.04.2012 11:33, schrieb Andreas Färber: 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 Sorry, forgot the change log: v1 - v2: * Added error_free() in cpu_x86_register() (spotted by Michael). * Added missing shift in model property getter (spotted by Michael). * Extended some commit messages. /-F 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(-) -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
Re: [Qemu-devel] [PATCH v2 00/15] QOM'ify x86 CPU, part 2: properties
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 Liguorianth...@codemonkey.ws Cc: Jan Kiszkajan.kis...@siemens.com Cc: Igor Mammedovimamm...@redhat.com Cc: Liu Jinsongjinsong@intel.com Cc: Lai Jiangshanla...@cn.fujitsu.com Cc: Vasilis Liaskovitisvasilis.liaskovi...@profitbricks.com Cc: Eduardo Habkostehabk...@redhat.com Cc: Michael Rothmdr...@linux.vnet.ibm.com Cc: Paolo Bonzinipbonz...@redhat.com Cc: Vadim Rozenfeldvroze...@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