qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH for-2.10 00/10] Correct two minor QMP interface design flaws
@ 2017-07-18 13:41 Markus Armbruster
  2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 01/10] qapi: Separate type QNull from QObject Markus Armbruster
                   ` (11 more replies)
  0 siblings, 12 replies; 54+ messages in thread
From: Markus Armbruster @ 2017-07-18 13:41 UTC (permalink / raw)
  To: qemu-devel
  Cc: eblake, berrange, kwolf, mreitz, qemu-block, quintela, dgilbert

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: <87379zhrhn.fsf@dusky.pond.sub.org>
    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

^ permalink raw reply	[flat|nested] 54+ messages in thread

end of thread, other threads:[~2017-07-19  8:33 UTC | newest]

Thread overview: 54+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-18 13:41 [Qemu-devel] [PATCH for-2.10 00/10] Correct two minor QMP interface design flaws Markus Armbruster
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 01/10] qapi: Separate type QNull from QObject Markus Armbruster
2017-07-18 14:24   ` Eric Blake
2017-07-18 15:44   ` Daniel P. Berrange
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 02/10] qapi: Use QNull for a more regular visit_type_null() Markus Armbruster
2017-07-18 14:36   ` Eric Blake
2017-07-18 15:05     ` Markus Armbruster
2017-07-18 15:46   ` Daniel P. Berrange
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 03/10] qapi: Introduce a first class 'null' type Markus Armbruster
2017-07-18 14:53   ` Eric Blake
2017-07-18 14:54     ` Eric Blake
2017-07-18 15:21       ` Markus Armbruster
2017-07-18 19:43         ` Markus Armbruster
2017-07-18 19:47           ` Eric Blake
2017-07-18 20:02             ` Markus Armbruster
2017-07-18 20:08           ` Eric Blake
2017-07-18 20:27             ` Markus Armbruster
2017-07-18 15:20     ` Markus Armbruster
2017-07-18 15:47   ` Daniel P. Berrange
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 04/10] tests/test-qobject-input-visitor: Drop redundant test Markus Armbruster
2017-07-18 15:13   ` Eric Blake
2017-07-18 15:47   ` Daniel P. Berrange
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 05/10] block: Use JSON null instead of "" to disable backing file Markus Armbruster
2017-07-18 15:53   ` Daniel P. Berrange
2017-07-18 17:27   ` Eric Blake
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 06/10] hmp: Clean up and simplify hmp_migrate_set_parameter() Markus Armbruster
2017-07-18 15:54   ` Daniel P. Berrange
2017-07-18 17:09   ` Dr. David Alan Gilbert
2017-07-18 17:34   ` Eric Blake
2017-07-18 18:28     ` Markus Armbruster
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 07/10] migration: Clean up around tls_creds, tls_hostname Markus Armbruster
2017-07-18 15:57   ` Daniel P. Berrange
2017-07-18 17:36   ` Eric Blake
2017-07-18 17:37   ` Dr. David Alan Gilbert
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 08/10] migration: Add TODO comments on duplication of QAPI_CLONE() Markus Armbruster
2017-07-18 15:57   ` Daniel P. Berrange
2017-07-18 17:36   ` Eric Blake
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 09/10] migration: Unshare MigrationParameters struct for now Markus Armbruster
2017-07-18 16:01   ` Daniel P. Berrange
2017-07-18 17:42   ` Eric Blake
2017-07-18 13:41 ` [Qemu-devel] [PATCH for-2.10 10/10] migration: Use JSON null instead of "" to reset parameter to default Markus Armbruster
2017-07-18 16:03   ` Daniel P. Berrange
2017-07-18 17:46   ` Eric Blake
2017-07-18 18:37     ` Markus Armbruster
2017-07-18 18:39     ` Markus Armbruster
2017-07-18 19:09       ` Eric Blake
2017-07-18 19:32         ` Markus Armbruster
2017-07-18 19:55           ` Eric Blake
2017-07-18 17:52   ` Dr. David Alan Gilbert
2017-07-18 19:24     ` Markus Armbruster
2017-07-19  8:32       ` Daniel P. Berrange
2017-07-18 14:02 ` [Qemu-devel] [PATCH for-2.10 00/10] Correct two minor QMP interface design flaws no-reply
2017-07-18 16:08 ` Daniel P. Berrange
2017-07-19  6:12   ` Kevin Wolf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).