[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v7 00/14] alternate layout (post-introspection clean
From: |
Eric Blake |
Subject: |
[Qemu-devel] [PATCH v7 00/14] alternate layout (post-introspection cleanups, subset C) |
Date: |
Fri, 16 Oct 2015 22:35:35 -0600 |
Pending prerequisite: subset B' v9:
https://lists.gnu.org/archive/html/qemu-devel/2015-10/msg03730.html
Pending prerequisite: Paolo's misc patches pull request which includes
a rewrite of qemu-char:
https://lists.gnu.org/archive/html/qemu-devel/2015-10/msg03808.html
Also available as a tag at this location:
git fetch git://repo.or.cz/qemu/ericb.git qapi-cleanupv7c
and I plan to eventually forcefully update my branch with the rest
of the v5 series, at:
http://repo.or.cz/qemu/ericb.git/shortlog/refs/heads/qapi
v7 notes:
Patches 1-3 of the previous round have moved to subset B; patch 7
is gone, and a couple of new patches are present in this round. The
latest version of subset B made it much easier to reason about
collision detection (namely, tag values can't collide with QMP values
thanks to a named rather than anonymous union in the C type), and I
like how things turned out. I suspect the hardest part of the review
will be patches 5/14 and 7/14, although none of this has really
had much review in any earlier versions.
001/14:[0004] [FC] 'qapi: Use generated TestStruct machinery in tests'
002/14:[down] 'qapi: Strengthen test of TestStructList'
003/14:[----] [--] 'qapi: Provide nicer array names in introspection'
004/14:[----] [--] 'qapi-introspect: Guarantee particular sorting'
005/14:[down] 'qapi: Rework collision assertions'
006/14:[down] 'qapi: Update tests related to QMP/branch collisions'
007/14:[0071] [FC] 'qapi: Simplify visiting of alternate types'
008/14:[0008] [FC] 'qapi: Fix alternates that accept 'number' but not 'int''
009/14:[----] [--] 'qapi: Remove dead visitor code'
010/14:[----] [-C] 'qapi: Plug leaks in test-qmp-*'
011/14:[0004] [FC] 'qapi: Simplify error testing in test-qmp-*'
012/14:[----] [--] 'qapi: Test failure in middle of array parse'
013/14:[----] [--] 'qapi: More tests of input arrays'
014/14:[0004] [FC] 'qapi: Simplify visits of optional fields'
v6 notes:
https://lists.gnu.org/archive/html/qemu-devel/2015-10/msg01980.html
Add some patches and rebase onto work on subset B. Rearrange some
patches from v5 (this set includes 17-20, 23, 25-27). Backport diff
gets a bit confused by one patch title changing.
Not much direct review comments, although some of the changes here
are updated based on comments made on other patches in the v5 series.
Subset D (and more?) will come later.
In v5:
https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg05410.html
I _did_ rearrange patches to try and group related features:
1-2: Groundwork cleanups
3-5: Add more test cases
6-16: Front-end cleanups
17-18: Introspection output cleanups
19-20: 'alternate' type cleanups
21-29: qapi visitor cleanups
30-45: qapi-ify netdev_add
46: add qapi shorthand for flat unions
Lots of fixes based on additional testing, and rebased to
track other changes that happened in the meantime. The series
is huge; I can split off smaller portions as requested.
In v4:
https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg02580.html
add some more clean up patches
rebase to Markus' recent work
pull in part of Zoltán's work to make netdev_add a flat union,
further enhancing it to be introspectible
I might be able to rearrange some of these patches, or separate
it into smaller independent series, if requested; but I'm
posting now to get review started.
In v3:
https://lists.gnu.org/archive/html/qemu-devel/2015-08/msg02059.html
redo cleanup of dealloc of partial struct
add patches to make all visit_type_*() avoid leaks on failure
add patches to allow boxed command arguments and events
In v2:
https://lists.gnu.org/archive/html/qemu-devel/2015-08/msg00900.html
rebase to Markus' v3 series
rework how comments are emitted for fields inherited from base
additional patches added for deleting colliding 'void *data'
documentation updates to match code changes
v1 was here:
https://lists.gnu.org/archive/html/qemu-devel/2015-07/msg05266.html
https://lists.gnu.org/archive/html/qemu-devel/2015-07/msg05325.html
Eric Blake (14):
qapi: Use generated TestStruct machinery in tests
qapi: Strengthen test of TestStructList
qapi: Provide nicer array names in introspection
qapi-introspect: Guarantee particular sorting
qapi: Rework collision assertions
qapi: Update tests related to QMP/branch collisions
qapi: Simplify visiting of alternate types
qapi: Fix alternates that accept 'number' but not 'int'
qapi: Remove dead visitor code
qapi: Plug leaks in test-qmp-*
qapi: Simplify error testing in test-qmp-*
qapi: Test failure in middle of array parse
qapi: More tests of input arrays
qapi: Simplify visits of optional fields
docs/qapi-code-gen.txt | 31 ++-
include/qapi/visitor-impl.h | 27 ++-
include/qapi/visitor.h | 22 +-
qapi/introspect.json | 22 +-
qapi/opts-visitor.c | 2 +-
qapi/qapi-visit-core.c | 141 +++++--------
qapi/qmp-input-visitor.c | 11 +-
qapi/string-input-visitor.c | 3 +-
scripts/qapi-introspect.py | 17 +-
scripts/qapi-types.py | 36 +---
scripts/qapi-visit.py | 23 ++-
scripts/qapi.py | 106 +++++-----
tests/Makefile | 3 -
tests/qapi-schema/alternate-empty.out | 1 -
tests/qapi-schema/flat-union-clash-branch.err | 1 -
tests/qapi-schema/flat-union-clash-branch.exit | 1 -
tests/qapi-schema/flat-union-clash-branch.json | 16 --
tests/qapi-schema/flat-union-clash-branch.out | 0
tests/qapi-schema/flat-union-clash-type.err | 1 -
tests/qapi-schema/flat-union-clash-type.exit | 1 -
tests/qapi-schema/flat-union-clash-type.json | 14 --
tests/qapi-schema/flat-union-clash-type.out | 0
tests/qapi-schema/qapi-schema-test.json | 20 +-
tests/qapi-schema/qapi-schema-test.out | 30 ++-
tests/qapi-schema/union-clash-type.err | 1 -
tests/qapi-schema/union-clash-type.exit | 1 -
tests/qapi-schema/union-clash-type.json | 7 -
tests/qapi-schema/union-clash-type.out | 0
tests/test-qmp-input-strict.c | 108 +++-------
tests/test-qmp-input-visitor.c | 268 +++++++++++--------------
tests/test-qmp-output-visitor.c | 153 ++++----------
tests/test-visitor-serialization.c | 76 ++-----
32 files changed, 450 insertions(+), 693 deletions(-)
delete mode 100644 tests/qapi-schema/flat-union-clash-branch.err
delete mode 100644 tests/qapi-schema/flat-union-clash-branch.exit
delete mode 100644 tests/qapi-schema/flat-union-clash-branch.json
delete mode 100644 tests/qapi-schema/flat-union-clash-branch.out
delete mode 100644 tests/qapi-schema/flat-union-clash-type.err
delete mode 100644 tests/qapi-schema/flat-union-clash-type.exit
delete mode 100644 tests/qapi-schema/flat-union-clash-type.json
delete mode 100644 tests/qapi-schema/flat-union-clash-type.out
delete mode 100644 tests/qapi-schema/union-clash-type.err
delete mode 100644 tests/qapi-schema/union-clash-type.exit
delete mode 100644 tests/qapi-schema/union-clash-type.json
delete mode 100644 tests/qapi-schema/union-clash-type.out
--
2.4.3
- [Qemu-devel] [PATCH v7 00/14] alternate layout (post-introspection cleanups, subset C),
Eric Blake <=
- [Qemu-devel] [PATCH v7 04/14] qapi-introspect: Guarantee particular sorting, Eric Blake, 2015/10/17
- [Qemu-devel] [PATCH v7 01/14] qapi: Use generated TestStruct machinery in tests, Eric Blake, 2015/10/17
- [Qemu-devel] [PATCH v7 03/14] qapi: Provide nicer array names in introspection, Eric Blake, 2015/10/17
- [Qemu-devel] [PATCH v7 06/14] qapi: Update tests related to QMP/branch collisions, Eric Blake, 2015/10/17
- [Qemu-devel] [PATCH v7 08/14] qapi: Fix alternates that accept 'number' but not 'int', Eric Blake, 2015/10/17
- [Qemu-devel] [PATCH v7 09/14] qapi: Remove dead visitor code, Eric Blake, 2015/10/17
- [Qemu-devel] [PATCH v7 14/14] qapi: Simplify visits of optional fields, Eric Blake, 2015/10/17
- [Qemu-devel] [PATCH v7 05/14] qapi: Rework collision assertions, Eric Blake, 2015/10/17