[Qemu-devel] [PATCH qom-cpu 00/17] x86 CPU cleanup, part 3

[Igor Mammedov]

It's rebase of "PATCH 00/20 v4] x86 CPU cleanup (wave 2)" including
changes asked by Andreas on IRC. See below (v5) for detailed changes.

This series is several cleanups, moved out from CPU properties series,
since they do not really depend on CPU properties re-factoring and could
simplify CPU subclasses work as well.

Series doesn't depend on cpu as device or any other series, and applies
to current master.

Depends on "[PATCH qom-cpu 0/7] disable kvm_mmu + -cpu "enforce" fixes (v3)"
  http://lists.gnu.org/archive/html/qemu-devel/2013-01/msg00815.html
  which is applied to qom-cpu now

git tree for testing:
    https://github.com/imammedo/qemu/tree/x86_cpu_cleanup.part3

vendor related changes are tested with:
  https://github.com/imammedo/virt-test/tree/cpuid_features
  following command was used to run test:
   ./run -t kvm --qemu-bin=$QEMU --tests="qemu_cpu.qemu13 cpuid.custom_vendor"

v5:
 - dropped patch
   "[PATCH 06/20] target-i386: move out CPU features initialization in separate 
func"
   due to Andreas objection
 - fixed x86cpu to x86_cpu new function prefixes
 - rebased on top of "disable kvm_mmu + -cpu "enforce" fixes (v3)" due conflicts
 - patches 1-4 from "[PATCH 00/20 v4] x86 CPU cleanup (wave 2)", already
   in master. So dropped from here
 - added patches 16-17 to deal with tsc_freq parsing without introducing
   new new visitor

v4:
 - rename [01/20] from:
     target-i386: filter out not TCG features if running without kvm at
       realize time
   to:
     target-i386: filter out unsupported features at realize time
 - make commit lines shorter for:
     target-i386: move kvm_check_features_against_host() check to realize
       time
 - restore removed by mistake host_cpuid() call in:
     target-i386: replace uint32_t vendor fields by vendor string in x86_def_t
 - fix spelling in:
     target-i386: print depricated warning if xlevel < 0x80000000
 - use qstring_append_int() for converting xlevel to string in:
     target-i386: set custom 'xlevel' without intermediate x86_def_t

v3:
 - [07/20] sets error if cpu name is empty, restore return -1 on error
 - get rid of *vendor_override field in CPUX86State & co
 - mark xlevel < 0x80000000 as depricated
 - squash idef-ing kvm specific functions in [08/20]
 - expand comment of [12/20] and reorder it right before "set custom" patches

v2:
 - cleanup commit message  and style fixes in
     [PATCH 2/6] target-i386: sanitize AMD's ext2_features at realize time
 - extracted more patches [07-20] from cpu properties series, that were
   more cleanups and code reorganizing than conversion to static properties.



*** BLURB HERE ***

Igor Mammedov (17):
  target-i386: move setting defaults out of cpu_x86_parse_featurestr()
  target-i386: cpu_x86_register() consolidate freeing resources
  target-i386: move kvm_check_features_against_host() check to realize
    time
  target-i386: add x86_cpu_vendor_words2str()
  target-i386: replace uint32_t vendor fields by vendor string in
    x86_def_t
  target-i386: remove vendor_override field from CPUX86State
  target-i386: prepare cpu_x86_parse_featurestr() to return a set of
    key,value property pairs
  target-i386: set custom 'vendor' without intermediate x86_def_t
  target-i386: print deprecated warning if xlevel < 0x80000000
  target-i386: set custom 'xlevel' without intermediate x86_def_t
  target-i386: set custom 'level' without intermediate x86_def_t
  target-i386: set custom 'model-id' without intermediate x86_def_t
  target-i386: set custom 'stepping' without intermediate x86_def_t
  target-i386: set custom 'model' without intermediate x86_def_t
  target-i386: set custom 'family' without intermediate x86_def_t
  target-i386: set custom 'tsc-frequency' without intermediate
    x86_def_t
  target-i386: remove setting tsc-frequency from x86_def_t

 target-i386/cpu.c |  314 ++++++++++++++++++++++-------------------------------
 target-i386/cpu.h |    7 +-
 2 files changed, 131 insertions(+), 190 deletions(-)