[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH for-2.10 00/10] Correct two minor QMP interface desi
From: |
Markus Armbruster |
Subject: |
[Qemu-block] [PATCH for-2.10 00/10] Correct two minor QMP interface design flaws |
Date: |
Tue, 18 Jul 2017 15:41:16 +0200 |
blockdev-add and migrate-set-parameters overload empty strings to mean
something entirely different. See my memo "qapi: Stop abusing
"special" values for something entirely different" for details.
Message-ID: <address@hidden>
https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg04526.html
This series deprecates these usages of "" in favour of JSON null.
Because we're so close to the 2.10 freeze, the implementation is
intentionally stupid: rewrite null to "" at first opportunity. The
proper way to do it would be rewriting "" to null, but that requires
fixing up code to work with null. There are TODO comments for that.
I'm willing to take care of them in the next development cycle.
Markus Armbruster (10):
qapi: Separate type QNull from QObject
qapi: Use QNull for a more regular visit_type_null()
qapi: Introduce a first class 'null' type
tests/test-qobject-input-visitor: Drop redundant test
block: Use JSON null instead of "" to disable backing file
hmp: Clean up and simplify hmp_migrate_set_parameter()
migration: Clean up around tls_creds, tls_hostname
migration: Add TODO comments on duplication of QAPI_CLONE()
migration: Unshare MigrationParameters struct for now
migration: Use JSON null instead of "" to reset parameter to default
blockdev.c | 14 +++++
hmp.c | 88 +++++++++++++----------------
hw/ppc/spapr_drc.c | 4 +-
include/qapi/qmp/qobject.h | 10 +++-
include/qapi/visitor-impl.h | 3 +-
include/qapi/visitor.h | 8 +--
include/qemu/typedefs.h | 1 +
migration/migration.c | 34 ++++++++---
qapi-schema.json | 99 +++++++++++++++++++++++++++++----
qapi/block-core.json | 29 +++++++---
qapi/qapi-clone-visitor.c | 5 +-
qapi/qapi-dealloc-visitor.c | 6 +-
qapi/qapi-visit-core.c | 7 ++-
qapi/qobject-input-visitor.c | 6 +-
qapi/qobject-output-visitor.c | 5 +-
qapi/string-input-visitor.c | 8 ++-
qapi/string-output-visitor.c | 3 +-
qapi/trace-events | 2 +-
qobject/json-parser.c | 2 +-
qobject/qnull.c | 8 ++-
scripts/qapi.py | 5 +-
target/i386/cpu.c | 4 +-
tests/check-qjson.c | 6 +-
tests/check-qnull.c | 27 +++++----
tests/qapi-schema/qapi-schema-test.json | 3 +-
tests/qapi-schema/qapi-schema-test.out | 1 +
tests/qemu-iotests/085 | 2 +-
tests/qemu-iotests/139 | 2 +-
tests/test-qobject-input-visitor.c | 24 ++++----
tests/test-qobject-output-visitor.c | 13 ++++-
30 files changed, 299 insertions(+), 130 deletions(-)
--
2.7.5
- [Qemu-block] [PATCH for-2.10 00/10] Correct two minor QMP interface design flaws,
Markus Armbruster <=