qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v10 00/25] qapi visitor cleanups part 1 (post-introspection cleanups subset E)
@ 2016-01-29 13:48 Eric Blake
  2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 01/25] qobject: Document more shortcomings in our number handling Eric Blake
                   ` (25 more replies)
  0 siblings, 26 replies; 37+ messages in thread
From: Eric Blake @ 2016-01-29 13:48 UTC (permalink / raw)
  To: qemu-devel; +Cc: marcandre.lureau, armbru, Eduardo Habkost

Based on qemu.git master. No pending prerequisites

Also available as a tag at this location:
git fetch git://repo.or.cz/qemu/ericb.git qapi-cleanupv10e

and will soon be part of my branch with the rest of the v5 series, at:
http://repo.or.cz/qemu/ericb.git/shortlog/refs/heads/qapi

v10 notes:
This is patches 1-20 of v9; 21-37 of that series will come later,
but this half was relatively clean and should be ready to merge.
Plus, this half includes the argument reordering, which touches
a lot of the tree, so getting it in sooner rather than later will
minimize rebase churn.  A couple patches were split differently
or retitled.  Reviewed-by was kept on patches that didn't change
in content (even if the content was split across different patch
boundaries).

Most of the work here was addressing Markus' review comments,
or rebasing later patches on top of earlier changes.

001/25:[0013] [FC] 'qobject: Document more shortcomings in our number handling'
002/25:[----] [--] 'qapi: Avoid use of misnamed DO_UPCAST()'
003/25:[----] [--] 'qapi: Drop dead dealloc visitor variable'
004/25:[down] 'qapi: Dealloc visitor does not need a type_size()'
005/25:[down] 'qapi: Drop dead parameter in gen_params()'
006/25:[down] 'hmp: Drop pointless allocation during qapi visit'
007/25:[down] 'hmp: Cache use of qapi visitor'
008/25:[down] 'vl: Ensure qapi visitor properly ends struct visit'
009/25:[0003] [FC] 'balloon: Improve use of qapi visitor'
010/25:[0004] [FC] 'qapi: Improve generated event use of qapi visitor'
011/25:[0004] [FC] 'qapi: Track all failures between visit_start/stop'
012/25:[down] 'qapi-visit: Kill unused visit_end_union()'
013/25:[0012] [FC] 'qapi: Prefer type_int64 over type_int in visitors'
014/25:[0012] [FC] 'qapi: Make all visitors supply uint64 callbacks'
015/25:[0014] [FC] 'qapi: Consolidate visitor small integer callbacks'
016/25:[0006] [FC] 'qapi: Don't cast Enum* to int*'
017/25:[----] [--] 'qom: Use typedef for Visitor'
018/25:[0004] [FC] 'qapi: Swap visit_* arguments for consistent 'name' placement'
019/25:[0005] [FC] 'qom: Swap 'name' next to visitor in ObjectPropertyAccessor'
020/25:[0010] [FC] 'qapi: Swap 'name' in visit_* callbacks to match public API'
021/25:[0018] [FC] 'qapi: Drop unused 'kind' for struct/enum visit'
022/25:[down] 'qapi: Tighten qmp_input_end_list()'
023/25:[0046] [FC] 'qapi: Drop unused error argument for list and implicit struct'
024/25:[0008] [FC] 'qmp: Fix reference-counting of qnull on empty output visit'
025/25:[0018] [FC] 'qmp: Don't abuse stack to track qmp-output root'

v9 notes:
https://lists.gnu.org/archive/html/qemu-devel/2016-01/msg03504.html
Rebase to master, incorporate findings from Marc-André.

v8 notes:
https://lists.gnu.org/archive/html/qemu-devel/2015-12/msg03863.html
For notes here and earlier, look in the archives

 include/qapi/visitor.h             |  60 +++++----
 include/qapi/visitor-impl.h        |  63 +++++-----
 scripts/qapi-commands.py           |   4 +-
 scripts/qapi-event.py              |  16 ++-
 scripts/qapi-types.py              |   2 +-
 scripts/qapi-visit.py              |  72 ++++++-----
 scripts/qapi.py                    |  13 +-
 include/qom/object.h               |  13 +-
 qapi/qapi-visit-core.c             | 252 +++++++++++++++----------------------
 backends/hostmem.c                 |  24 ++--
 block/qapi.c                       |   2 +-
 blockdev.c                         |   4 +-
 bootdevice.c                       |  12 +-
 hmp.c                              |  18 +--
 hw/acpi/core.c                     |   4 +-
 hw/acpi/ich9.c                     |  49 ++++----
 hw/block/nvme.c                    |  12 +-
 hw/core/machine.c                  |  24 ++--
 hw/core/qdev-properties-system.c   |  44 +++----
 hw/core/qdev-properties.c          | 180 +++++++++++++-------------
 hw/core/qdev.c                     |   7 +-
 hw/i386/pc.c                       |  43 +++----
 hw/ide/qdev.c                      |  12 +-
 hw/intc/xics.c                     |  20 +--
 hw/isa/lpc_ich9.c                  |   7 +-
 hw/mem/pc-dimm.c                   |   6 +-
 hw/misc/edu.c                      |   6 +-
 hw/misc/tmp105.c                   |  12 +-
 hw/net/ne2000-isa.c                |  14 ++-
 hw/pci-host/piix.c                 |  18 +--
 hw/pci-host/q35.c                  |  23 ++--
 hw/ppc/spapr_drc.c                 |  34 +++--
 hw/usb/dev-storage.c               |  12 +-
 hw/virtio/virtio-balloon.c         |  30 ++---
 memory.c                           |  26 ++--
 net/dump.c                         |  12 +-
 net/filter-buffer.c                |  14 ++-
 net/net.c                          |   4 +-
 numa.c                             |   6 +-
 qapi/opts-visitor.c                |  52 ++++----
 qapi/qapi-dealloc-visitor.c        |  48 ++++---
 qapi/qmp-input-visitor.c           |  54 ++++----
 qapi/qmp-output-visitor.c          | 120 +++++++++---------
 qapi/string-input-visitor.c        |  62 +++++----
 qapi/string-output-visitor.c       |  54 ++++----
 qemu-img.c                         |  11 +-
 qobject/json-parser.c              |   6 +-
 qobject/qjson.c                    |  11 +-
 qom/object.c                       | 124 +++++++++---------
 replay/replay-input.c              |   4 +-
 target-i386/cpu.c                  |  97 +++++++-------
 target-ppc/translate_init.c        |  12 +-
 tests/test-opts-visitor.c          |   6 +-
 tests/test-qdev-global-props.c     |  18 +--
 tests/test-qmp-commands.c          |   2 +-
 tests/test-qmp-input-strict.c      |  28 ++---
 tests/test-qmp-input-visitor.c     |  80 ++++++------
 tests/test-qmp-output-visitor.c    |  36 +++---
 tests/test-string-input-visitor.c  |  34 ++---
 tests/test-string-output-visitor.c |  16 +--
 tests/test-visitor-serialization.c |  54 ++++----
 util/qemu-sockets.c                |   4 +-
 vl.c                               |  26 ++--
 63 files changed, 1067 insertions(+), 1066 deletions(-)

-- 
2.5.0

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

end of thread, other threads:[~2016-02-02  9:56 UTC | newest]

Thread overview: 37+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-29 13:48 [Qemu-devel] [PATCH v10 00/25] qapi visitor cleanups part 1 (post-introspection cleanups subset E) Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 01/25] qobject: Document more shortcomings in our number handling Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 02/25] qapi: Avoid use of misnamed DO_UPCAST() Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 03/25] qapi: Drop dead dealloc visitor variable Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 04/25] qapi: Dealloc visitor does not need a type_size() Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 05/25] qapi: Drop dead parameter in gen_params() Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 06/25] hmp: Drop pointless allocation during qapi visit Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 07/25] hmp: Cache use of qapi visitor Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 08/25] vl: Ensure qapi visitor properly ends struct visit Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 09/25] balloon: Improve use of qapi visitor Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 10/25] qapi: Improve generated event " Eric Blake
2016-02-01 12:31   ` Markus Armbruster
2016-02-01 22:50     ` Eric Blake
2016-02-02  7:52       ` Markus Armbruster
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 11/25] qapi: Track all failures between visit_start/stop Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 12/25] qapi-visit: Kill unused visit_end_union() Eric Blake
2016-02-01 12:38   ` Markus Armbruster
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 13/25] qapi: Prefer type_int64 over type_int in visitors Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 14/25] qapi: Make all visitors supply uint64 callbacks Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 15/25] qapi: Consolidate visitor small integer callbacks Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 16/25] qapi: Don't cast Enum* to int* Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 17/25] qom: Use typedef for Visitor Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 18/25] qapi: Swap visit_* arguments for consistent 'name' placement Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 19/25] qom: Swap 'name' next to visitor in ObjectPropertyAccessor Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 20/25] qapi: Swap 'name' in visit_* callbacks to match public API Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 21/25] qapi: Drop unused 'kind' for struct/enum visit Eric Blake
2016-02-01 12:56   ` Markus Armbruster
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 22/25] qapi: Tighten qmp_input_end_list() Eric Blake
2016-01-29 13:48 ` [Qemu-devel] [PATCH v10 23/25] qapi: Drop unused error argument for list and implicit struct Eric Blake
2016-02-01 13:07   ` Markus Armbruster
2016-02-01 17:22     ` Eric Blake
2016-02-01 23:14     ` Eric Blake
2016-02-02  9:00       ` Markus Armbruster
2016-01-29 13:49 ` [Qemu-devel] [PATCH v10 24/25] qmp: Fix reference-counting of qnull on empty output visit Eric Blake
2016-01-29 13:49 ` [Qemu-devel] [PATCH v10 25/25] qmp: Don't abuse stack to track qmp-output root Eric Blake
2016-02-01 13:25 ` [Qemu-devel] [PATCH v10 00/25] qapi visitor cleanups part 1 (post-introspection cleanups subset E) Markus Armbruster
2016-02-02  9:55   ` Markus Armbruster

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