[Qemu-devel] [PATCH v4 0/7] Poison some more target-specific defines

Thomas Huth posted 7 patches 6 years, 9 months ago
Only 6 patches received!
Makefile.objs             |  2 +-
Makefile.target           |  2 +-
accel/tcg/translate-all.c |  8 ++++++++
bootdevice.c              |  2 +-
hw/acpi/ich9.c            |  1 -
hw/i386/pc_q35.c          |  1 +
hw/misc/Makefile.objs     |  2 +-
include/exec/cpu-common.h |  2 ++
include/exec/poison.h     |  8 ++++++++
include/hw/i386/pc.h      | 13 -------------
include/qom/cpu.h         |  8 ++++++++
include/sysemu/kvm.h      | 31 ++++++++++++-------------------
qom/cpu.c                 |  5 ++---
target/i386/kvm_i386.h    | 23 +++++++++++++++++++++++
14 files changed, 68 insertions(+), 40 deletions(-)
[Qemu-devel] [PATCH v4 0/7] Poison some more target-specific defines
Posted by Thomas Huth 6 years, 9 months ago
This series marks some more #defines as poisoned, which are
target-specific (declared in config-target.h) and thus must
not be used in common code.

v4:
- Rebased to master since the v2 versions of the first two patches
  have already been merged. The diff between v2 and v3 can now be
  found in the first patch of this series.

v3:
- Mark some more TARGET_xxx and CONFIG_xxx defines as poisoned in
  the first two patches (as requested by Richard)
- Added the final patch to move the edu device to common-obj-y

v2:
- First two patches are the same as in v1
- Reworked the CONFIG_KVM patches according to Paolo's review feedback
- Added two new patches to finally poison CONFIG_SOFTMMU, too
- Added a final patch to move bootdevice.o to common-obj now
  (based on an earlier patch where I also tried to move numa.o and
  balloon.o, too - but these files are indirectly target-dependent as
  I now know, so they can't be moved)

Thomas Huth (7):
  include/exec/poison: Add some more missing TARGET and CONFIG defines
  Move CONFIG_KVM related definitions to kvm_i386.h
  include/exec/poison: Mark CONFIG_KVM as poisoned, too
  cpu: Introduce a wrapper for tlb_flush() that can be used in common
    code
  include/exec/poison: Mark CONFIG_SOFTMMU as poisoned
  Makefile: Move bootdevice.o to common-obj-y
  hw/misc/edu: Compile the edu device as common object

 Makefile.objs             |  2 +-
 Makefile.target           |  2 +-
 accel/tcg/translate-all.c |  8 ++++++++
 bootdevice.c              |  2 +-
 hw/acpi/ich9.c            |  1 -
 hw/i386/pc_q35.c          |  1 +
 hw/misc/Makefile.objs     |  2 +-
 include/exec/cpu-common.h |  2 ++
 include/exec/poison.h     |  8 ++++++++
 include/hw/i386/pc.h      | 13 -------------
 include/qom/cpu.h         |  8 ++++++++
 include/sysemu/kvm.h      | 31 ++++++++++++-------------------
 qom/cpu.c                 |  5 ++---
 target/i386/kvm_i386.h    | 23 +++++++++++++++++++++++
 14 files changed, 68 insertions(+), 40 deletions(-)

-- 
1.8.3.1