Poison more CONFIG switches

[PATCH for-6.1 0/4] Poison more CONFIG switches

Thomas Huth posted 4 patches 3 years ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210414112004.943383-1-thuth@redhat.com

Maintainers: "Daniel P. Berrangé" <berrange@redhat.com>, Markus Armbruster <armbru@redhat.com>, Eric Blake <eblake@redhat.com>, Sunil Muthuswamy <sunilmut@microsoft.com>, Colin Xu <colin.xu@intel.com>, Cameron Esfahani <dirty@apple.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Eduardo Habkost <ehabkost@redhat.com>, Wenchao Wang <wenchao.wang@intel.com>, Juan Quintela <quintela@redhat.com>, Roman Bolshakov <r.bolshakov@yadro.com>

Makefile              |  2 +-
configure             |  5 +++++
include/exec/poison.h |  6 ++++++
include/sysemu/hax.h  |  4 ++++
include/sysemu/hvf.h  |  4 ++++
include/sysemu/whpx.h |  4 ++++
migration/meson.build |  3 ++-
migration/migration.c | 15 ---------------
migration/migration.h |  2 ++
migration/target.c    | 25 +++++++++++++++++++++++++
qapi/qom.json         |  4 ++--
11 files changed, 55 insertions(+), 19 deletions(-)
create mode 100644 migration/target.c

Expand all Fold all

[PATCH for-6.1 0/4] Poison more CONFIG switches

Posted by Thomas Huth 3 years ago

Using target-specific CONFIG_xxx switches in common code via "#ifdef"s
is wrong, since these macros are only defined for target-specific code.
We already poison many switches in common code to avoid the bugs with
dead code here, but these problems still keep creeping in ...
This series now improves the situation by poisoning more symbols,
especially by generating these from the target-specific config headers
automatically.

Thomas Huth (4):
  include/sysemu: Poison all accelerator CONFIG switches in common code
  migration: Move populate_vfio_info() into a separate file
  qapi/qom.json: Do not use CONFIG_VIRTIO_CRYPTO in common code
  configure: Poison all current target-specific #defines

 Makefile              |  2 +-
 configure             |  5 +++++
 include/exec/poison.h |  6 ++++++
 include/sysemu/hax.h  |  4 ++++
 include/sysemu/hvf.h  |  4 ++++
 include/sysemu/whpx.h |  4 ++++
 migration/meson.build |  3 ++-
 migration/migration.c | 15 ---------------
 migration/migration.h |  2 ++
 migration/target.c    | 25 +++++++++++++++++++++++++
 qapi/qom.json         |  4 ++--
 11 files changed, 55 insertions(+), 19 deletions(-)
 create mode 100644 migration/target.c

-- 
2.27.0