qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
@ 2017-08-04  1:25 Eric Blake
  2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 01/22] qobject: Accept "%"PRId64 in qobject_from_jsonf() Eric Blake
                   ` (23 more replies)
  0 siblings, 24 replies; 66+ messages in thread
From: Eric Blake @ 2017-08-04  1:25 UTC (permalink / raw)
  To: qemu-devel; +Cc: armbru

Definitely 2.11 material now.  v4 is a rather big overhaul
of what has been previously posted, rebasing things to make
it easier to pass the command name separately from the args
when calling qmp() derivatives in the testsuite, and adding
support for %% handling to make -Wformat checking more robust.

Also available as a tag at:
git fetch git://repo.or.cz/qemu/ericb.git qapi-qmp-cleanup-v4

Diffstat since v3:

001/22:[0011] [FC] 'qobject: Accept "%"PRId64 in qobject_from_jsonf()'
002/22:[----] [--] 'tests: Clean up wait for event'
003/22:[----] [--] 'tests/libqtest: Clean up how we read the QMP greeting'
004/22:[down] 'tests: Add assertion for no qmp("")'
005/22:[down] 'qobject: Simplify qobject_from_jsonv()'
006/22:[down] 'qobject: Perform %% interpolation in qobject_from_jsonf()'
007/22:[down] 'numa-test: Use hmp()'
008/22:[----] [--] 'qtest: Avoid passing raw strings through hmp()'
009/22:[0022] [FC] 'qtest: Document calling conventions'
010/22:[down] 'libqtest: Skip round-trip through QObject'
011/22:[down] 'test-qga: Simplify command construction'
012/22:[down] 'libqtest: Change qmp_fd_send() to drop varargs'
013/22:[down] 'libqtest: Add qmp_raw()'
014/22:[down] 'libqtest: Separate qmp_discard_response() from command'
015/22:[down] 'libqtest: Delete qtest_qmp() wrappers'
016/22:[down] 'libqtest: Add qmp_cmd() helper'
017/22:[down] 'libqtest: Add qmp_args() helper'
018/22:[0014] [FC] 'tests/libqos/usb: Clean up string interpolation into QMP input'
019/22:[down] 'libqtest: Add qmp_args_dict() helper'
020/22:[0004] [FC] 'tests/libqos/pci: Clean up string interpolation into QMP input'
021/22:[down] 'libqtest: Drop now-unused qmp()'
022/22:[down] 'libqtest: Rename qmp_cmd() to qmp()'

Eric Blake (18):
  qobject: Accept "%"PRId64 in qobject_from_jsonf()
  tests: Add assertion for no qmp("")
  qobject: Simplify qobject_from_jsonv()
  qobject: Perform %% interpolation in qobject_from_jsonf()
  numa-test: Use hmp()
  qtest: Avoid passing raw strings through hmp()
  qtest: Document calling conventions
  libqtest: Skip round-trip through QObject
  test-qga: Simplify command construction
  libqtest: Change qmp_fd_send() to drop varargs
  libqtest: Add qmp_raw()
  libqtest: Separate qmp_discard_response() from command
  libqtest: Delete qtest_qmp() wrappers
  libqtest: Add qmp_cmd() helper
  libqtest: Add qmp_args() helper
  libqtest: Add qmp_args_dict() helper
  libqtest: Drop now-unused qmp()
  libqtest: Rename qmp_cmd() to qmp()

Markus Armbruster (4):
  tests: Clean up wait for event
  tests/libqtest: Clean up how we read the QMP greeting
  tests/libqos/usb: Clean up string interpolation into QMP input
  tests/libqos/pci: Clean up string interpolation into QMP input

 configure                          |  24 ++++
 include/qapi/qmp/qjson.h           |   2 +-
 tests/libqos/pci.h                 |   2 +-
 tests/libqtest.h                   | 163 +++++++++++---------------
 tests/libqtest.c                   | 234 ++++++++++++++-----------------------
 qobject/json-lexer.c               |  24 ++--
 qobject/json-parser.c              |  59 +++++-----
 qobject/qjson.c                    |  49 +++++++-
 tests/ahci-test.c                  |  29 ++---
 tests/boot-order-test.c            |   4 +-
 tests/check-qjson.c                |  57 +++++++++
 tests/device-introspect-test.c     |   7 +-
 tests/drive_del-test.c             |  11 +-
 tests/fdc-test.c                   |  11 +-
 tests/ide-test.c                   |   5 +-
 tests/ivshmem-test.c               |  10 +-
 tests/libqos/ahci.c                |   4 +-
 tests/libqos/libqos.c              |  23 +---
 tests/libqos/pci-pc.c              |  14 +--
 tests/libqos/pci.c                 |  32 ++---
 tests/libqos/usb.c                 |  26 ++---
 tests/numa-test.c                  |  23 +---
 tests/pc-cpu-test.c                |   8 +-
 tests/postcopy-test.c              |  41 +++----
 tests/q35-test.c                   |   2 +-
 tests/qmp-test.c                   |  27 +++--
 tests/qom-test.c                   |  11 +-
 tests/tco-test.c                   |   3 +-
 tests/test-filter-mirror.c         |   3 +-
 tests/test-filter-redirector.c     |   6 +-
 tests/test-hmp.c                   |   4 +-
 tests/test-netfilter.c             | 139 +++++++++-------------
 tests/test-qga.c                   | 220 +++++++++++++++++-----------------
 tests/test-qobject-input-visitor.c |  18 +--
 tests/test-x86-cpuid-compat.c      |   8 +-
 tests/tmp105-test.c                |   9 +-
 tests/usb-hcd-uhci-test.c          |  20 +---
 tests/usb-hcd-xhci-test.c          |  37 ++----
 tests/vhost-user-test.c            |  12 +-
 tests/virtio-blk-test.c            |  20 ++--
 tests/virtio-net-test.c            |  13 +--
 tests/virtio-scsi-test.c           |  13 +--
 tests/virtio-serial-test.c         |  12 +-
 tests/vmgenid-test.c               |   2 +-
 tests/wdt_ib700-test.c             |  37 ++----
 45 files changed, 678 insertions(+), 800 deletions(-)

-- 
2.13.3

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

end of thread, other threads:[~2018-01-08 16:46 UTC | newest]

Thread overview: 66+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-04  1:25 [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp() Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 01/22] qobject: Accept "%"PRId64 in qobject_from_jsonf() Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 02/22] tests: Clean up wait for event Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 03/22] tests/libqtest: Clean up how we read the QMP greeting Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 04/22] tests: Add assertion for no qmp("") Eric Blake
2017-08-09  7:13   ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 05/22] qobject: Simplify qobject_from_jsonv() Eric Blake
2017-08-09  7:59   ` Markus Armbruster
2017-08-09 13:14     ` Eric Blake
2017-10-02  5:46       ` Markus Armbruster
2017-10-02 14:30         ` Eric Blake
2018-01-08 16:46           ` Eric Blake
2017-09-11 21:52   ` Eric Blake
2017-10-02  7:15     ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 06/22] qobject: Perform %% interpolation in qobject_from_jsonf() Eric Blake
2017-08-09  9:06   ` Markus Armbruster
2017-08-09 13:21     ` Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 07/22] numa-test: Use hmp() Eric Blake
2017-08-09  9:07   ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 08/22] qtest: Avoid passing raw strings through hmp() Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 09/22] qtest: Document calling conventions Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 10/22] libqtest: Skip round-trip through QObject Eric Blake
2017-08-09 10:10   ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 11/22] test-qga: Simplify command construction Eric Blake
2017-08-09 11:40   ` Markus Armbruster
2017-08-09 13:29     ` Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 12/22] libqtest: Change qmp_fd_send() to drop varargs Eric Blake
2017-08-09 13:18   ` Markus Armbruster
2017-08-09 13:44     ` Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 13/22] libqtest: Add qmp_raw() Eric Blake
2017-08-09 14:54   ` Markus Armbruster
2017-08-09 15:18     ` Eric Blake
2017-08-10  7:29       ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 14/22] libqtest: Separate qmp_discard_response() from command Eric Blake
2017-08-09 15:15   ` Markus Armbruster
2017-08-09 15:32     ` Eric Blake
2017-08-10  7:40       ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 15/22] libqtest: Delete qtest_qmp() wrappers Eric Blake
2017-08-09 15:34   ` Markus Armbruster
2017-08-09 16:35     ` Eric Blake
2017-08-10  7:47       ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 16/22] libqtest: Add qmp_cmd() helper Eric Blake
2017-08-09 15:40   ` Markus Armbruster
2017-08-09 16:39     ` Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 17/22] libqtest: Add qmp_args() helper Eric Blake
2017-08-09 15:57   ` Markus Armbruster
2017-08-09 21:57     ` Eric Blake
2017-08-10  8:17       ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 18/22] tests/libqos/usb: Clean up string interpolation into QMP input Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 19/22] libqtest: Add qmp_args_dict() helper Eric Blake
2017-08-09 15:59   ` Markus Armbruster
2017-08-09 16:41     ` Eric Blake
2017-08-10  8:19       ` Markus Armbruster
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 20/22] tests/libqos/pci: Clean up string interpolation into QMP input Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 21/22] libqtest: Drop now-unused qmp() Eric Blake
2017-08-09 16:01   ` Markus Armbruster
2017-08-09 16:43     ` Eric Blake
2017-08-04  1:25 ` [Qemu-devel] [PATCH v4 22/22] libqtest: Rename qmp_cmd() to qmp() Eric Blake
2017-08-04  1:54 ` [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp() no-reply
2017-08-04 11:50   ` Eric Blake
2017-08-04 12:10     ` Fam Zheng
2017-08-07  6:43       ` Fam Zheng
2017-08-07  7:33         ` Fam Zheng
2017-08-07 14:08           ` Philippe Mathieu-Daudé
2017-08-07  8:09   ` Fam Zheng
2017-08-04  2:02 ` no-reply

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).