qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: qemu-devel@nongnu.org, lersek@redhat.com
Subject: [Qemu-devel] [PATCH 00/16] introduce OptsVisitor, rebase -net/-netdev parsing
Date: Tue, 22 May 2012 12:45:39 +0200	[thread overview]
Message-ID: <1337683555-13301-1-git-send-email-lersek@redhat.com> (raw)

Inspired by [1], the first half of this series attempts to implement a new
visitor that should clean up defining and processing command line options.
For a more detailed description, please see "[PATCH 04/16] qapi: introduce
OptsVisitor".

The second half converts -net/-netdev parsing to the new visitor.

The series depends on "qapi: convert netdev_add & netdev_del" v5, see [2].

Comments highly appreciated; please keep me CC'd. Thank you.

[1] http://lists.nongnu.org/archive/html/qemu-devel/2012-04/msg02512.html
[2] http://lists.nongnu.org/archive/html/qemu-devel/2012-05/msg02841.html

Laszlo Ersek (15):
  qapi: introduce "size" type
  expose QemuOpt and QemuOpts struct definitions to interested parties
  qapi: introduce OptsVisitor
  qapi schema: remove trailing whitespace
  qapi schema: add Netdev types
  hw, net: "net_client_type" -> "NetClientOptionsKind" (qapi-generated)
  convert net_client_init() to OptsVisitor
  convert net_init_nic() to NetClientOptions
  convert net_init_dump() to NetClientOptions
  convert net_init_slirp() to NetClientOptions
  convert net_init_socket() to NetClientOptions
  convert net_init_vde() to NetClientOptions
  convert net_init_tap() to NetClientOptions
  convert net_init_bridge() to NetClientOptions
  remove unused QemuOpts parameter from net init functions

Paolo Bonzini (1):
  qapi: fix error propagation

 error.h                        |    4 +-
 net.h                          |   16 +--
 net/dump.h                     |    5 +-
 net/slirp.h                    |    5 +-
 net/socket.h                   |    5 +-
 net/tap.h                      |   10 +-
 net/vde.h                      |    5 +-
 qapi/opts-visitor.h            |   31 +++
 qapi/qapi-visit-core.h         |    4 +
 qemu-option-internal.h         |   53 +++++
 error.c                        |    4 +-
 hw/cadence_gem.c               |    2 +-
 hw/dp8393x.c                   |    2 +-
 hw/e1000.c                     |    2 +-
 hw/eepro100.c                  |    2 +-
 hw/etraxfs_eth.c               |    2 +-
 hw/lan9118.c                   |    2 +-
 hw/lance.c                     |    2 +-
 hw/mcf_fec.c                   |    2 +-
 hw/milkymist-minimac2.c        |    2 +-
 hw/mipsnet.c                   |    2 +-
 hw/musicpal.c                  |    2 +-
 hw/ne2000-isa.c                |    2 +-
 hw/ne2000.c                    |    2 +-
 hw/opencores_eth.c             |    2 +-
 hw/pcnet-pci.c                 |    2 +-
 hw/rtl8139.c                   |    2 +-
 hw/smc91c111.c                 |    2 +-
 hw/spapr_llan.c                |    2 +-
 hw/stellaris_enet.c            |    2 +-
 hw/usb/dev-network.c           |    2 +-
 hw/vhost_net.c                 |    2 +-
 hw/virtio-net.c                |   10 +-
 hw/xen_nic.c                   |    2 +-
 hw/xgmac.c                     |    2 +-
 hw/xilinx_axienet.c            |    2 +-
 hw/xilinx_ethlite.c            |    2 +-
 net.c                          |  493 +++++++++++-----------------------------
 net/dump.c                     |   24 ++-
 net/slirp.c                    |   96 +++------
 net/socket.c                   |  124 ++++-------
 net/tap-aix.c                  |    2 +-
 net/tap-bsd.c                  |    2 +-
 net/tap-haiku.c                |    2 +-
 net/tap-linux.c                |    9 +-
 net/tap-solaris.c              |    2 +-
 net/tap-win32.c                |   14 +-
 net/tap.c                      |  152 ++++++------
 net/vde.c                      |   40 +++-
 qapi/opts-visitor.c            |  363 +++++++++++++++++++++++++++++
 qapi/qapi-visit-core.c         |   17 +-
 qemu-option.c                  |   24 +--
 tests/test-qmp-input-visitor.c |   24 ++-
 Makefile.objs                  |    2 +-
 docs/qapi-code-gen.txt         |    2 +
 qapi-schema.json               |  285 +++++++++++++++++++++++-
 scripts/qapi-visit.py          |  129 ++++++-----
 scripts/qapi.py                |    2 +-
 58 files changed, 1239 insertions(+), 771 deletions(-)
 create mode 100644 qapi/opts-visitor.h
 create mode 100644 qemu-option-internal.h
 create mode 100644 qapi/opts-visitor.c

             reply	other threads:[~2012-05-22 10:45 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-22 10:45 Laszlo Ersek [this message]
2012-05-22 10:45 ` [Qemu-devel] [PATCH 01/16] qapi: fix error propagation Laszlo Ersek
2012-05-22 10:45 ` [Qemu-devel] [PATCH 02/16] qapi: introduce "size" type Laszlo Ersek
2012-06-05 20:39   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 03/16] expose QemuOpt and QemuOpts struct definitions to interested parties Laszlo Ersek
2012-06-05 20:40   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 04/16] qapi: introduce OptsVisitor Laszlo Ersek
2012-06-05 21:12   ` Paolo Bonzini
2012-06-06 11:12     ` Laszlo Ersek
2012-06-06 12:02       ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 05/16] qapi schema: remove trailing whitespace Laszlo Ersek
2012-06-05 20:40   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 06/16] qapi schema: add Netdev types Laszlo Ersek
2012-06-05 21:08   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 07/16] hw, net: "net_client_type" -> "NetClientOptionsKind" (qapi-generated) Laszlo Ersek
2012-06-05 20:41   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 08/16] convert net_client_init() to OptsVisitor Laszlo Ersek
2012-06-05 20:46   ` Paolo Bonzini
2012-06-05 21:07     ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 09/16] convert net_init_nic() to NetClientOptions Laszlo Ersek
2012-06-05 20:50   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 10/16] convert net_init_dump() " Laszlo Ersek
2012-06-05 20:51   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 11/16] convert net_init_slirp() " Laszlo Ersek
2012-06-05 20:53   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 12/16] convert net_init_socket() " Laszlo Ersek
2012-06-05 21:02   ` Paolo Bonzini
2012-06-05 21:14     ` Eric Blake
2012-06-05 21:27       ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 13/16] convert net_init_vde() " Laszlo Ersek
2012-06-05 21:04   ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 14/16] convert net_init_tap() " Laszlo Ersek
2012-05-22 10:45 ` [Qemu-devel] [PATCH 15/16] convert net_init_bridge() " Laszlo Ersek
2012-06-05 21:05   ` Paolo Bonzini
2012-06-06 12:16     ` Laszlo Ersek
2012-06-06 14:13       ` Paolo Bonzini
2012-05-22 10:45 ` [Qemu-devel] [PATCH 16/16] remove unused QemuOpts parameter from net init functions Laszlo Ersek
2012-06-05 21:06   ` Paolo Bonzini
2012-06-05 21:13 ` [Qemu-devel] [PATCH 00/16] introduce OptsVisitor, rebase -net/-netdev parsing Paolo Bonzini
2012-06-06 13:03   ` Laszlo Ersek
2012-06-06 13:31     ` Andreas Färber
2012-06-06 14:10       ` Paolo Bonzini
2012-06-06 14:34         ` Andreas Färber
2012-06-06 14:43           ` Paolo Bonzini
2012-06-06 15:16         ` Michael Roth
2012-06-06 15:30           ` Laszlo Ersek
2012-06-06 15:58             ` Michael Roth
2012-06-06 16:14             ` Michael Roth
2012-06-06 16:47               ` Paolo Bonzini
2012-06-06 16:49               ` Laszlo Ersek
2012-06-06 17:05                 ` Laszlo Ersek
2012-06-06 20:09                 ` Michael Roth
2012-06-06 20:59                   ` Andreas Färber
2012-06-07 11:32                     ` Laszlo Ersek
2012-06-07 12:17                       ` Andreas Färber
2012-06-07 11:29                   ` Laszlo Ersek
2012-06-07 15:29                     ` Michael Roth
2012-06-07 15:46                       ` Paolo Bonzini
2012-06-09 11:21                         ` Laszlo Ersek
2012-06-06 15:31           ` Michael Roth
2012-06-06 14:09     ` Paolo Bonzini
2012-06-09 15:30     ` Laszlo Ersek
2012-06-11  7:06       ` Paolo Bonzini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1337683555-13301-1-git-send-email-lersek@redhat.com \
    --to=lersek@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).