Skip to content

qapi flattening

Kővágó, Zoltán qemu-devel 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest6 patches 0 repliesNew
  2. v15 patches 13 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6qapi: support implicit structs in OptsVisitorqapi/opts-visitor.c+15 -00 replies
2/6qapi: convert NumaOptions into a flat unionnuma.c qapi-schema.json+37 -120 replies
3/6net: remove NetLegacy structnet/net.c qapi-schema.json+30 -420 replies
4/6net: use Netdev instead of NetClientOptions in client initnet/clients.h net/dump.c net/hub.c +9 more+56 -570 replies
5/6qapi: change Netdev into a flat unionhw/arm/musicpal.c hw/core/qdev-properties-system.c hw/net/allwinner_emac.c +42 more+166 -1540 replies
6/6qapi: reorder NetdevBase and Netdevqapi-schema.json+23 -230 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [Qemu-devel] [PATCH v2 0/6] qapi flattening

This patch series flattens the structures that are used by OptsVisitor, namely NumaOptions and Netdev (NetLegacy is removed in the process).

Patch 1 is needed, otherwise qemu segfaults when OptsVisitor visits a flat struct. Patch 1 and 2 are identical to v1, while the networking related patches are completely rewritten.

The NetLegacy and Netdev structs are merged, this way we won't have to work-around the type system. This requires a few extra checks in net_client_init1, but allows us to simply use Netdev everywhere, without using void *, union, or the NetClientOpts union.

Please review.

Kővágó, Zoltán (6):
qapi: support implicit structs in OptsVisitor
qapi: convert NumaOptions into a flat union
net: remove NetLegacy struct
net: use Netdev instead of NetClientOptions in client init
qapi: change Netdev into a flat union
qapi: reorder NetdevBase and Netdev

 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/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               |  18 +++---
 hw/net/virtio-net.c              |   6 +-
 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             |   2 +-
 include/net/net.h                |   4 +-
 monitor.c                        |  14 ++---
 net/clients.h                    |  20 +++---
 net/dump.c                       |   8 +--
 net/hub.c                        |  24 ++++----
 net/l2tpv3.c                     |   8 +--
 net/net.c                        | 129 +++++++++++++++++++--------------------
 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                 |  14 ++---
 numa.c                           |   2 +-
 qapi-schema.json                 | 129 +++++++++++++++++++++++----------------
 qapi/opts-visitor.c              |  15 +++++
 48 files changed, 278 insertions(+), 239 deletions(-)
Signature · 1 line
2.4.5