[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 00/10] Correct two minor QMP interface design
From: |
John Snow |
Subject: |
Re: [Qemu-block] [PATCH v2 00/10] Correct two minor QMP interface design flaws |
Date: |
Wed, 16 Aug 2017 17:34:29 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
Looks like this series got no replies (maybe a failure of my mail
filtering?) but it has since been merged, so purely for my own selfish
purposes;
Merged upstream as:
01fa55982692fb51a16049b63b571651a1053989 migration: Use JSON null
instead of "" to reset parameter to default
1bda8b3c6950f74482ba19e8529db72b511ba977 migration: Unshare
MigrationParameters struct for now
e87fae4c488ff8a10b921311a63f16b94031c611 migration: Add TODO comments on
duplication of QAPI_CLONE()
8cc99dcdc264bc896926e43e7576c7b7ab633d70 migration: Clean up around
tls_creds, tls_hostname
7e91e82044f8d23acfb9949a2cdd667a6b239acd hmp: Clean up and simplify
hmp_migrate_set_parameter()
c42e8742f527476839bcc5f91c3d2ea456ca6a45 block: Use JSON null instead of
"" to disable backing file
06f80154b2ada1d58ac504e98ff6e943b069b96c
tests/test-qobject-input-visitor: Drop redundant test
4d2d5c41a9e8ee201cda8be8701f7f9fc92e71aa qapi: Introduce a first class
'null' type
d2f95f4d482374485234790a6fc3cca29ebb7355 qapi: Use QNull for a more
regular visit_type_null()
006ca09f3027d86346fce707e9295975c6558f42 qapi: Separate type QNull from
QObject
--js
On 07/20/2017 03:53 AM, Markus Armbruster wrote:
> 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.
>
Did this get dropped in favor of a more comprehensive 2.11 fix?
> v2, because my pull request clashed with Juan's:
> * Rebased, non-trivial conflicts in migration/migration.c
> * PATCH 02: Commit message typo [Eric]
> Fix ppc [patchew]
> * PATCH 03: qapi-code-gen.txt update [Eric]
> * PATCH 09: Conflict resolution, somewhat ugly
> * PATCH 10: Doc improvement [Eric]
> Conflict resolution
>
> 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 +++++
> docs/devel/qapi-code-gen.txt | 10 +--
> 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 | 96 +++++++++++++++++++++++++---
> qapi-schema.json | 108
> ++++++++++++++++++++++++++++----
> 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 +-
> target/ppc/translate_init.c | 5 +-
> 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 +++-
> 32 files changed, 378 insertions(+), 137 deletions(-)
>
- Re: [Qemu-block] [PATCH v2 00/10] Correct two minor QMP interface design flaws,
John Snow <=