All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v6 00/16] qapi netdev_add introspection (post-introspection cleanups subset F)
@ 2015-12-23 20:55 Eric Blake
  2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 01/16] net: use Netdev instead of NetClientOptions in client init Eric Blake
                   ` (16 more replies)
  0 siblings, 17 replies; 20+ messages in thread
From: Eric Blake @ 2015-12-23 20:55 UTC (permalink / raw)
  To: qemu-devel; +Cc: armbru

Prerequisites:
+ my qapi cleanups subset E v8:
https://lists.gnu.org/archive/html/qemu-devel/2015-12/msg03863.html

This is the final subset of my original cleanups series v5.
The focus here is adding a 'box':true parameter and anonymous
flat union base syntax, so that we can finally represent
netdev_add as a full-fledged qapi command that shows up in
introspection rather than as a magic command with undocumented
arguments.

v6 notes:
Rebase, since there are several other series that want features
from this subset.

Backport diffstat:

001/16:[0096] [FC] 'net: use Netdev instead of NetClientOptions in client init'
002/16:[0021] [FC] 'qapi: Avoid use of 'data' member of qapi unions'
003/16:[0010] [FC] 'qapi: Forbid empty unions and useless alternates'
004/16:[0014] [FC] 'qapi: Drop useless 'data' member of unions'
005/16:[0017] [FC] 'qapi: Hide tag_name data member of variants'
006/16:[0049] [FC] 'qapi: Plumb in 'box' to qapi generator lower levels'
007/16:[down] 'qapi: Implement boxed types for commands/events'
008/16:[0008] [FC] 'qapi: support implicit structs in OptsVisitor'
009/16:[0092] [FC] 'qapi: Change Netdev into a flat union'
010/16:[----] [-C] 'net: Use correct type for bool flag'
011/16:[0023] [FC] 'net: Complete qapi-fication of netdev_add'
012/16:[0049] [FC] 'qapi: Allow anonymous base for flat union'
013/16:[down] 'qapi: Use anonymous base in SchemaInfo'
014/16:[down] 'qapi: Use anonymous base in Netdev'
015/16:[down] 'qapi: Use anonymous base in CpuInfo'
016/16:[down] 'qapi: Populate info['name'] for each entity'

v5 notes:
https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg05410.html
see the archives for more

Eric Blake (13):
  qapi: Avoid use of 'data' member of qapi unions
  qapi: Forbid empty unions and useless alternates
  qapi: Drop useless 'data' member of unions
  qapi: Hide tag_name data member of variants
  qapi: Plumb in 'box' to qapi generator lower levels
  qapi: Implement boxed types for commands/events
  net: Use correct type for bool flag
  net: Complete qapi-fication of netdev_add
  qapi: Allow anonymous base for flat union
  qapi: Use anonymous base in SchemaInfo
  qapi: Use anonymous base in Netdev
  qapi: Use anonymous base in CpuInfo
  qapi: Populate info['name'] for each entity

Kővágó, Zoltán (3):
  net: use Netdev instead of NetClientOptions in client init
  qapi: support implicit structs in OptsVisitor
  qapi: Change Netdev into a flat union

 blockdev.c                                         |  31 ++--
 docs/qapi-code-gen.txt                             |  67 ++++++---
 hw/arm/musicpal.c                                  |   2 +-
 hw/core/qdev-properties-system.c                   |   2 +-
 hw/net/allwinner_emac.c                            |   2 +-
 hw/net/cadence_gem.c                               |   2 +-
 hw/net/dp8393x.c                                   |   2 +-
 hw/net/e1000.c                                     |   2 +-
 hw/net/eepro100.c                                  |   2 +-
 hw/net/etraxfs_eth.c                               |   2 +-
 hw/net/fsl_etsec/etsec.c                           |   2 +-
 hw/net/imx_fec.c                                   |   2 +-
 hw/net/lan9118.c                                   |   2 +-
 hw/net/lance.c                                     |   2 +-
 hw/net/mcf_fec.c                                   |   2 +-
 hw/net/milkymist-minimac2.c                        |   2 +-
 hw/net/mipsnet.c                                   |   2 +-
 hw/net/ne2000-isa.c                                |   2 +-
 hw/net/ne2000.c                                    |   2 +-
 hw/net/opencores_eth.c                             |   2 +-
 hw/net/pcnet-pci.c                                 |   2 +-
 hw/net/rocker/rocker_fp.c                          |   2 +-
 hw/net/rtl8139.c                                   |   2 +-
 hw/net/smc91c111.c                                 |   2 +-
 hw/net/spapr_llan.c                                |   2 +-
 hw/net/stellaris_enet.c                            |   2 +-
 hw/net/vhost_net.c                                 |  16 +-
 hw/net/virtio-net.c                                |  10 +-
 hw/net/vmxnet3.c                                   |   2 +-
 hw/net/xen_nic.c                                   |   2 +-
 hw/net/xgmac.c                                     |   2 +-
 hw/net/xilinx_axienet.c                            |   2 +-
 hw/net/xilinx_ethlite.c                            |   2 +-
 hw/usb/dev-network.c                               |   4 +-
 include/net/net.h                                  |   7 +-
 monitor.c                                          |  14 +-
 net/clients.h                                      |  20 +--
 net/dump.c                                         |   8 +-
 net/hub.c                                          |  24 +--
 net/l2tpv3.c                                       |   8 +-
 net/net.c                                          | 161 +++++++++++++--------
 net/netmap.c                                       |   6 +-
 net/slirp.c                                        |   8 +-
 net/socket.c                                       |  10 +-
 net/tap-win32.c                                    |   8 +-
 net/tap.c                                          |  28 ++--
 net/vde.c                                          |   8 +-
 net/vhost-user.c                                   |  20 +--
 qapi-schema.json                                   |  82 ++++++-----
 qapi/introspect.json                               |  12 +-
 qapi/opts-visitor.c                                |  15 ++
 qmp-commands.hx                                    |   2 +-
 scripts/qapi-commands.py                           |  75 ++++++----
 scripts/qapi-event.py                              |  43 ++++--
 scripts/qapi-introspect.py                         |   4 +-
 scripts/qapi-types.py                              |  17 +--
 scripts/qapi-visit.py                              |  10 +-
 scripts/qapi.py                                    | 116 ++++++++++-----
 tests/Makefile                                     |   4 +-
 tests/qapi-schema/alternate-empty.err              |   1 +
 tests/qapi-schema/alternate-empty.exit             |   2 +-
 tests/qapi-schema/alternate-empty.json             |   2 +-
 tests/qapi-schema/alternate-empty.out              |   5 -
 tests/qapi-schema/args-bad-box.err                 |   1 +
 ...{flat-union-bad-base.exit => args-bad-box.exit} |   0
 tests/qapi-schema/args-bad-box.json                |   2 +
 .../{union-clash-data.err => args-bad-box.out}     |   0
 tests/qapi-schema/args-box-anon.err                |   1 +
 tests/qapi-schema/args-box-anon.exit               |   1 +
 tests/qapi-schema/args-box-anon.json               |   2 +
 .../{flat-union-bad-base.out => args-box-anon.out} |   0
 tests/qapi-schema/args-union.err                   |   2 +-
 tests/qapi-schema/args-union.json                  |   3 +-
 tests/qapi-schema/event-case.out                   |   1 +
 tests/qapi-schema/flat-union-bad-base.err          |   1 -
 tests/qapi-schema/flat-union-bad-base.json         |  13 --
 tests/qapi-schema/flat-union-empty.err             |   1 +
 tests/qapi-schema/flat-union-empty.exit            |   2 +-
 tests/qapi-schema/flat-union-empty.json            |   2 +-
 tests/qapi-schema/flat-union-empty.out             |  10 --
 tests/qapi-schema/ident-with-escape.out            |   2 +-
 tests/qapi-schema/indented-expr.out                |   4 +-
 tests/qapi-schema/qapi-schema-test.json            |  10 +-
 tests/qapi-schema/qapi-schema-test.out             |  47 ++++--
 tests/qapi-schema/test-qapi.py                     |  11 +-
 tests/qapi-schema/union-clash-data.exit            |   1 -
 tests/qapi-schema/union-clash-data.json            |   7 -
 tests/qapi-schema/union-clash-data.out             |  11 --
 tests/qapi-schema/union-empty.err                  |   1 +
 tests/qapi-schema/union-empty.exit                 |   2 +-
 tests/qapi-schema/union-empty.json                 |   2 +-
 tests/qapi-schema/union-empty.out                  |   7 -
 tests/test-qmp-commands.c                          |  12 ++
 ui/input.c                                         |   2 +-
 94 files changed, 604 insertions(+), 463 deletions(-)
 create mode 100644 tests/qapi-schema/args-bad-box.err
 rename tests/qapi-schema/{flat-union-bad-base.exit => args-bad-box.exit} (100%)
 create mode 100644 tests/qapi-schema/args-bad-box.json
 rename tests/qapi-schema/{union-clash-data.err => args-bad-box.out} (100%)
 create mode 100644 tests/qapi-schema/args-box-anon.err
 create mode 100644 tests/qapi-schema/args-box-anon.exit
 create mode 100644 tests/qapi-schema/args-box-anon.json
 rename tests/qapi-schema/{flat-union-bad-base.out => args-box-anon.out} (100%)
 delete mode 100644 tests/qapi-schema/flat-union-bad-base.err
 delete mode 100644 tests/qapi-schema/flat-union-bad-base.json
 delete mode 100644 tests/qapi-schema/union-clash-data.exit
 delete mode 100644 tests/qapi-schema/union-clash-data.json
 delete mode 100644 tests/qapi-schema/union-clash-data.out

-- 
2.4.3

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

end of thread, other threads:[~2015-12-23 21:01 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-23 20:55 [Qemu-devel] [PATCH v6 00/16] qapi netdev_add introspection (post-introspection cleanups subset F) Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 01/16] net: use Netdev instead of NetClientOptions in client init Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 02/16] qapi: Avoid use of 'data' member of qapi unions Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 03/16] qapi: Forbid empty unions and useless alternates Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 04/16] qapi: Drop useless 'data' member of unions Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 05/16] qapi: Hide tag_name data member of variants Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 06/16] qapi: Plumb in 'box' to qapi generator lower levels Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 07/16] qapi: Implement boxed types for commands/events Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 08/16] qapi: support implicit structs in OptsVisitor Eric Blake
2015-12-23 20:55 ` [Qemu-arm] [PATCH v6 09/16] qapi: Change Netdev into a flat union Eric Blake
2015-12-23 20:55   ` Eric Blake
2015-12-23 20:55   ` [Qemu-devel] " Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 10/16] net: Use correct type for bool flag Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 11/16] net: Complete qapi-fication of netdev_add Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 12/16] qapi: Allow anonymous base for flat union Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 13/16] qapi: Use anonymous base in SchemaInfo Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 14/16] qapi: Use anonymous base in Netdev Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 15/16] qapi: Use anonymous base in CpuInfo Eric Blake
2015-12-23 20:55 ` [Qemu-devel] [PATCH v6 16/16] qapi: Populate info['name'] for each entity Eric Blake
2015-12-23 20:58 ` [Qemu-devel] [PATCH v6 00/16] qapi netdev_add introspection (post-introspection cleanups subset F) 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.