It's rebase of remnants of "[PATCH qom-cpu 00/17] x86 CPU cleanup, part 3" [1] taking in account comments of reviewers.
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. git tree for testing: https://github.com/imammedo/qemu/tree/x86_cpu_cleanup.part4 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" *1 - http://lists.gnu.org/archive/html/qemu-devel/2013-01/msg01732.html v6: - patches 1-3 from previous series already in master, dropped - patches 4-5 from previous series are sqashed in one - patches 7-8,10-17 from previous series are sqashed in one and cpu_x86_parse_featurestr() rewritten to set properties directly on CPU 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. Igor Mammedov (5): target-i386: print deprecated warning if xlevel < 0x80000000 target-i386: replace uint32_t vendor fields by vendor string in x86_def_t target-i386: remove vendor_override field from CPUX86State target-i386: set custom features/properties without intermediate x86_def_t target-i386: remove setting tsc-frequency from x86_def_t target-i386/cpu.c | 322 ++++++++++++++++++++++------------------------------- target-i386/cpu.h | 7 +- 2 files changed, 138 insertions(+), 191 deletions(-)