All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v7 00/38] Preliminary libqtest cleanups
@ 2017-09-11 17:19 Eric Blake
  2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 01/38] test-qga: Kill broken and dead QGA_TEST_SIDE_EFFECTING code Eric Blake
                   ` (37 more replies)
  0 siblings, 38 replies; 75+ messages in thread
From: Eric Blake @ 2017-09-11 17:19 UTC (permalink / raw)
  To: qemu-devel; +Cc: armbru, pbonzini, thuth

I'm trying to get rid of some of the duplication in libqtest.h
before tackling the project of making qmp() easier to use. v6
got some positive review on the direction of making dependence
on global_qtest explicit (compared to v5 where I went the
opposite direction and tried to use global_qtest even more than
it was already being abused for), so most of the changes in
this posting are related to additional cleanups or addressing
good review comments.

Still to come: I still have work pending on overhauling the
qmp() and qtest_qmp() code, using naming that makes more
sense, as well as getting us to the point that we can reliably
use -Wformat checking on qmp().

Available as a tag at:
git fetch git://repo.or.cz/qemu/ericb.git qapi-qmp-cleanup-v7

Since v6 (https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg00308.html):
- Add review tags where appropriate
- merge a couple of patches [Thomas]
- split ahci into multiple patches [Thomas]
- add qtest_[v]startf rather than overloading (and churn) on
qtest_start [Thomas]
- catch more depedencies on global_qtest under tests/libqos/
- lots more patches added to make use of global_qtest obvious
- drop .gitignore changes for a later series

001/38:[----] [--] 'test-qga: Kill broken and dead QGA_TEST_SIDE_EFFECTING code'
002/38:[----] [--] 'qtest: Don't perform side effects inside assertion'
003/38:[----] [--] 'numa-test: Use hmp()'
004/38:[----] [--] 'tests: Clean up wait for event'
005/38:[----] [--] 'libqtest: Remove dead qtest_instances variable'
006/38:[----] [--] 'libqtest: Use qemu_strtoul()'
007/38:[0020] [FC] 'libqtest: Inline qtest_query_target_endianness()'
008/38:[0019] [FC] 'libqos: Track QTestState with QPCIBus'
009/38:[down] 'libqos: Track QTestState with QVirtioBus'
010/38:[down] 'libqos: Move/rename qpci_unplug_acpi_device_test() to pci.c'
011/38:[0045] [FC] 'libqos: Use explicit QTestState for pci operations'
012/38:[0179] [FC] 'libqos: Use explicit QTestState for virtio operations'
013/38:[0011] [FC] 'libqos: Use explicit QTestState for fw_cfg operations'
014/38:[0015] [FC] 'libqos: Use explicit QTestState for rtas operations'
015/38:[down] 'libqos: Use explicit QTestState for i2c operations'
016/38:[down] 'libqos: Use explicit QTestState for ahci operations'
017/38:[down] 'libqos: Use explicit QTestState for remaining libqos operations'
018/38:[0079] [FC] 'ahci-test: Drop dependence on global_qtest'
019/38:[0012] [FC] 'ivshmem-test: Drop dependence on global_qtest'
020/38:[----] [--] 'postcopy-test: Drop dependence on global_qtest'
021/38:[----] [--] 'vhost-user-test: Drop dependence on global_qtest'
022/38:[----] [-C] 'qmp-test: Drop dependence on global_qtest'
023/38:[----] [--] 'tests/boot-sector: Drop dependence on global_qtest'
024/38:[0079] [FC] 'tests/acpi-utils: Drop dependence on global_qtest'
025/38:[----] [--] 'wdt_ib700-test: Drop dependence on global_qtest'
026/38:[down] 'libqtest: Merge qtest_end() into qtest_quit()'
027/38:[down] 'libqtest: Swap order of qtest_init() and qtest_start()'
028/38:[down] 'libqtest: Add qtest_[v]startf()'
029/38:[down] 'libqtest: Merge qtest_init() into qtest_start()'
030/38:[0008] [FC] 'qtest: Avoid passing raw strings through hmp()'
031/38:[down] 'libqtest: Merge qtest_clock_*() with clock_*()'
032/38:[down] 'libqtest: Merge qtest_irq*() with irq*()'
033/38:[down] 'libqtest: Merge qtest_{in,out}[bwl]() with {in,out}[bwl]()'
034/38:[down] 'libqtest: Merge qtest_{read,write}[bwlq]() with {read,write}[bwlq]()'
035/38:[down] 'libqtest: Merge qtest_{mem,buf}{read,write}() with {mem,buf}{read,write}()'
036/38:[down] 'libqtest: Merge qtest_memset() with qmemset()'
037/38:[0088] [FC] 'libqtest: Separate qmp_discard_response() from command'
038/38:[down] 'libqtest: Merge qtest_hmp() with hmp()'

Eric Blake (37):
  test-qga: Kill broken and dead QGA_TEST_SIDE_EFFECTING code
  qtest: Don't perform side effects inside assertion
  numa-test: Use hmp()
  libqtest: Remove dead qtest_instances variable
  libqtest: Use qemu_strtoul()
  libqtest: Inline qtest_query_target_endianness()
  libqos: Track QTestState with QPCIBus
  libqos: Track QTestState with QVirtioBus
  libqos: Move/rename qpci_unplug_acpi_device_test() to pci.c
  libqos: Use explicit QTestState for pci operations
  libqos: Use explicit QTestState for virtio operations
  libqos: Use explicit QTestState for fw_cfg operations
  libqos: Use explicit QTestState for rtas operations
  libqos: Use explicit QTestState for i2c operations
  libqos: Use explicit QTestState for ahci operations
  libqos: Use explicit QTestState for remaining libqos operations
  ahci-test: Drop dependence on global_qtest
  ivshmem-test: Drop dependence on global_qtest
  postcopy-test: Drop dependence on global_qtest
  vhost-user-test: Drop dependence on global_qtest
  qmp-test: Drop dependence on global_qtest
  tests/boot-sector: Drop dependence on global_qtest
  tests/acpi-utils: Drop dependence on global_qtest
  wdt_ib700-test: Drop dependence on global_qtest
  libqtest: Merge qtest_end() into qtest_quit()
  libqtest: Swap order of qtest_init() and qtest_start()
  libqtest: Add qtest_[v]startf()
  libqtest: Merge qtest_init() into qtest_start()
  qtest: Avoid passing raw strings through hmp()
  libqtest: Merge qtest_clock_*() with clock_*()
  libqtest: Merge qtest_irq*() with irq*()
  libqtest: Merge qtest_{in,out}[bwl]() with {in,out}[bwl]()
  libqtest: Merge qtest_{read,write}[bwlq]() with {read,write}[bwlq]()
  libqtest: Merge qtest_{mem,buf}{read,write}() with
    {mem,buf}{read,write}()
  libqtest: Merge qtest_memset() with qmemset()
  libqtest: Separate qmp_discard_response() from command
  libqtest: Merge qtest_hmp() with hmp()

Markus Armbruster (1):
  tests: Clean up wait for event

 tests/libqtest.h               | 522 +++++++----------------------------------
 tests/acpi-utils.h             |  67 +++---
 tests/boot-sector.h            |   6 +-
 tests/libqos/ahci.h            |   2 +-
 tests/libqos/fw_cfg.h          |  10 +-
 tests/libqos/i2c.h             |   7 +-
 tests/libqos/libqos.h          |   5 +-
 tests/libqos/malloc-pc.h       |   4 +-
 tests/libqos/malloc-spapr.h    |   2 +-
 tests/libqos/malloc.h          |   1 +
 tests/libqos/pci-pc.h          |   2 +-
 tests/libqos/pci-spapr.h       |   2 +-
 tests/libqos/pci.h             |   5 +-
 tests/libqos/rtas.h            |   9 +-
 tests/libqos/virtio-mmio.h     |   6 +-
 tests/libqos/virtio-pci.h      |   2 -
 tests/libqos/virtio.h          |  13 +-
 tests/multiboot/libc.h         |   2 +-
 tests/libqtest.c               | 192 ++++++++-------
 qtest.c                        |  80 +++++--
 tests/ac97-test.c              |   4 +-
 tests/acpi-utils.c             |  19 +-
 tests/ahci-test.c              |  92 ++++----
 tests/bios-tables-test.c       | 199 ++++++++--------
 tests/boot-order-test.c        |  31 ++-
 tests/boot-sector.c            |   8 +-
 tests/boot-serial-test.c       |  12 +-
 tests/device-introspect-test.c |  30 +--
 tests/display-vga-test.c       |  18 +-
 tests/drive_del-test.c         |  26 +-
 tests/ds1338-test.c            |   6 +-
 tests/e1000-test.c             |  10 +-
 tests/e1000e-test.c            |  46 ++--
 tests/eepro100-test.c          |  11 +-
 tests/endianness-test.c        |  57 ++---
 tests/es1370-test.c            |   4 +-
 tests/fdc-test.c               | 144 ++++++------
 tests/fw_cfg-test.c            |  13 +-
 tests/hd-geo-test.c            |  20 +-
 tests/i440fx-test.c            |  24 +-
 tests/i82801b11-test.c         |   5 +-
 tests/ide-test.c               |  58 +++--
 tests/intel-hda-test.c         |   9 +-
 tests/ioh3420-test.c           |   7 +-
 tests/ipmi-bt-test.c           |  29 +--
 tests/ipmi-kcs-test.c          |  23 +-
 tests/ipoctal232-test.c        |   5 +-
 tests/ivshmem-test.c           |  27 +--
 tests/libqos/ahci.c            |  47 ++--
 tests/libqos/fw_cfg.c          |  14 +-
 tests/libqos/i2c-imx.c         |  67 +++---
 tests/libqos/i2c-omap.c        |  45 ++--
 tests/libqos/libqos-pc.c       |   2 +-
 tests/libqos/libqos.c          |  52 +---
 tests/libqos/malloc-pc.c       |   8 +-
 tests/libqos/malloc-spapr.c    |   4 +-
 tests/libqos/pci-pc.c          |  81 +++----
 tests/libqos/pci-spapr.c       |  45 ++--
 tests/libqos/pci.c             |  22 +-
 tests/libqos/rtas.c            |  36 +--
 tests/libqos/virtio-mmio.c     |  61 +++--
 tests/libqos/virtio-pci.c      |  14 +-
 tests/libqos/virtio.c          |  87 ++++---
 tests/m25p80-test.c            |  89 ++++---
 tests/m48t59-test.c            |  14 +-
 tests/megasas-test.c           |   7 +-
 tests/multiboot/libc.c         |   2 +-
 tests/ne2000-test.c            |   4 +-
 tests/numa-test.c              | 102 +++-----
 tests/nvme-test.c              |   7 +-
 tests/pc-cpu-test.c            |  24 +-
 tests/pcnet-test.c             |   4 +-
 tests/pnv-xscom-test.c         |  18 +-
 tests/postcopy-test.c          |  93 ++++----
 tests/prom-env-test.c          |  15 +-
 tests/pvpanic-test.c           |   8 +-
 tests/pxe-test.c               |  16 +-
 tests/q35-test.c               |  24 +-
 tests/qmp-test.c               |  43 ++--
 tests/qom-test.c               |   7 +-
 tests/rtas-test.c              |   3 +-
 tests/rtc-test.c               |  94 ++++----
 tests/rtl8139-test.c           |  17 +-
 tests/spapr-phb-test.c         |   5 +-
 tests/tco-test.c               |  45 ++--
 tests/test-arm-mptimer.c       |  43 ++--
 tests/test-filter-mirror.c     |  19 +-
 tests/test-filter-redirector.c |  66 +++---
 tests/test-hmp.c               |  13 +-
 tests/test-netfilter.c         |   9 +-
 tests/test-qga.c               |  90 -------
 tests/test-x86-cpuid-compat.c  |  13 +-
 tests/tmp105-test.c            |   7 +-
 tests/tpci200-test.c           |   4 +-
 tests/usb-hcd-ehci-test.c      |  27 ++-
 tests/usb-hcd-ohci-test.c      |   4 +-
 tests/usb-hcd-uhci-test.c      |   7 +-
 tests/usb-hcd-xhci-test.c      |  19 +-
 tests/vhost-user-test.c        | 105 +++------
 tests/virtio-9p-test.c         |   5 +-
 tests/virtio-balloon-test.c    |   4 +-
 tests/virtio-blk-test.c        |  83 ++++---
 tests/virtio-console-test.c    |  10 +-
 tests/virtio-net-test.c        |  30 ++-
 tests/virtio-rng-test.c        |   9 +-
 tests/virtio-scsi-test.c       |  22 +-
 tests/virtio-serial-test.c     |   4 +-
 tests/vmgenid-test.c           |  51 ++--
 tests/vmxnet3-test.c           |   4 +-
 tests/wdt_ib700-test.c         |  92 ++++----
 110 files changed, 1632 insertions(+), 2210 deletions(-)

-- 
2.13.5

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

end of thread, other threads:[~2017-09-14  4:36 UTC | newest]

Thread overview: 75+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-11 17:19 [Qemu-devel] [PATCH v7 00/38] Preliminary libqtest cleanups Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 01/38] test-qga: Kill broken and dead QGA_TEST_SIDE_EFFECTING code Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 02/38] qtest: Don't perform side effects inside assertion Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 03/38] numa-test: Use hmp() Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 04/38] tests: Clean up wait for event Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 05/38] libqtest: Remove dead qtest_instances variable Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 06/38] libqtest: Use qemu_strtoul() Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 07/38] libqtest: Inline qtest_query_target_endianness() Eric Blake
2017-09-12  6:32   ` Thomas Huth
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 08/38] libqos: Track QTestState with QPCIBus Eric Blake
2017-09-11 23:46   ` John Snow
2017-09-12  7:05   ` Thomas Huth
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 09/38] libqos: Track QTestState with QVirtioBus Eric Blake
2017-09-12  7:21   ` Thomas Huth
2017-09-12 13:28     ` Eric Blake
2017-09-13  7:10       ` Thomas Huth
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 10/38] libqos: Move/rename qpci_unplug_acpi_device_test() to pci.c Eric Blake
2017-09-12  7:29   ` Thomas Huth
2017-09-12 13:28     ` Eric Blake
2017-09-13  7:15       ` Thomas Huth
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 11/38] libqos: Use explicit QTestState for pci operations Eric Blake
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 12/38] libqos: Use explicit QTestState for virtio operations Eric Blake
2017-09-12  7:38   ` Thomas Huth
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 13/38] libqos: Use explicit QTestState for fw_cfg operations Eric Blake
2017-09-11 23:49   ` John Snow
2017-09-12  8:55   ` Thomas Huth
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 14/38] libqos: Use explicit QTestState for rtas operations Eric Blake
2017-09-12  9:01   ` Thomas Huth
2017-09-11 17:19 ` [Qemu-devel] [PATCH v7 15/38] libqos: Use explicit QTestState for i2c operations Eric Blake
2017-09-12  9:04   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 16/38] libqos: Use explicit QTestState for ahci operations Eric Blake
2017-09-11 23:54   ` John Snow
2017-09-12  9:09   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 17/38] libqos: Use explicit QTestState for remaining libqos operations Eric Blake
2017-09-11 21:30   ` Greg Kurz
2017-09-12  0:01   ` John Snow
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 18/38] ahci-test: Drop dependence on global_qtest Eric Blake
2017-09-12  0:20   ` John Snow
2017-09-12  0:21     ` John Snow
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 19/38] ivshmem-test: " Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 20/38] postcopy-test: " Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 21/38] vhost-user-test: " Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 22/38] qmp-test: " Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 23/38] tests/boot-sector: " Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 24/38] tests/acpi-utils: " Eric Blake
2017-09-12  9:26   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 25/38] wdt_ib700-test: " Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 26/38] libqtest: Merge qtest_end() into qtest_quit() Eric Blake
2017-09-12  0:31   ` John Snow
2017-09-12  9:30   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 27/38] libqtest: Swap order of qtest_init() and qtest_start() Eric Blake
2017-09-12  9:57   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 28/38] libqtest: Add qtest_[v]startf() Eric Blake
2017-09-12 10:14   ` Thomas Huth
2017-09-12 13:32     ` Eric Blake
2017-09-13  7:19       ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 29/38] libqtest: Merge qtest_init() into qtest_start() Eric Blake
2017-09-12 10:37   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 30/38] qtest: Avoid passing raw strings through hmp() Eric Blake
2017-09-11 17:42   ` Dr. David Alan Gilbert
2017-09-12 10:40   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 31/38] libqtest: Merge qtest_clock_*() with clock_*() Eric Blake
2017-09-12 10:45   ` Thomas Huth
2017-09-12 13:35     ` Eric Blake
2017-09-14  4:35       ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 32/38] libqtest: Merge qtest_irq*() with irq*() Eric Blake
2017-09-12 10:47   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 33/38] libqtest: Merge qtest_{in, out}[bwl]() with {in, out}[bwl]() Eric Blake
2017-09-12 10:49   ` Thomas Huth
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 34/38] libqtest: Merge qtest_{read, write}[bwlq]() with {read, write}[bwlq]() Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 35/38] libqtest: Merge qtest_{mem, buf}{read, write}() with {mem, buf}{read, write}() Eric Blake
2017-09-11 21:35   ` Greg Kurz
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 36/38] libqtest: Merge qtest_memset() with qmemset() Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 37/38] libqtest: Separate qmp_discard_response() from command Eric Blake
2017-09-11 17:20 ` [Qemu-devel] [PATCH v7 38/38] libqtest: Merge qtest_hmp() with hmp() Eric Blake

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.