qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Apfelbaum <marcel.a@redhat.com>
To: qemu-devel@nongnu.org
Cc: mst@redhat.com, aik@ozlabs.ru, lcapitulino@redhat.com,
	blauwirbel@gmail.com, jcmvbkbc@gmail.com,
	edgar.iglesias@gmail.com, gxt@mprc.pku.edu.cn, proljc@gmail.com,
	agraf@suse.de, lersek@redhat.com, scottwood@freescale.com,
	borntraeger@de.ibm.com, hpoussin@reactos.org,
	aliguori@amazon.com, mdroth@linux.vnet.ibm.com,
	chouteau@adacore.com, jan.kiszka@web.de, stefanha@redhat.com,
	pbonzini@redhat.com, cornelia.huck@de.ibm.com,
	peter.crosthwaite@xilinx.com, mark.langsdorf@calxeda.com,
	armbru@redhat.com, michael@walle.cc, qemu-ppc@nongnu.org,
	peter.chubb@nicta.com.au, afaerber@suse.de, aurelien@aurel32.net
Subject: [Qemu-devel] [Qemu-detvel] [PATCH 0/4] machine: QemuOpts per machine
Date: Wed,  7 May 2014 17:42:56 +0300	[thread overview]
Message-ID: <1399473780-20374-1-git-send-email-marcel.a@redhat.com> (raw)

This series continues 'machine as QOM' targeting two related issues:
 - Patch 1/4 completly removes QEMUMachineInitArgs from the code
   with a big mechanical patch. It removes QEMUMachineInitArgs
   from MachineState and gets rid of duplicated fields.
   - I am perfectly aware that patches touching a lot of files
     are not desirable, but this one is a very simple replacement
     patch:
       QEMUMachineInitArgs -> MachineState
       args -> ms
   - This is the simplest way to get rid of QEMUMachineInitArgs fast.
 - Patch 4/4 aims to enable QemuOpts per machine by automatically 
   filling in the machine opts into the current machine. Subclassing
   MachineClass and adding options mapped into properties is a
   better way than using global options.

Patches 2/4 and 3/4 fix little bugs needed by 4/4.

Marcel Apfelbaum (4):
  machine: conversion of QEMUMachineInitArgs to MachineState
  qapi: output visitor crashes qemu if it encounters a NULL value
  vl.c: do not set 'type' property in obj_set_property
  hw/machine: qemu machine opts as properties to QemuMachineState

 hw/alpha/dp264.c                         |  12 +-
 hw/arm/collie.c                          |  10 +-
 hw/arm/cubieboard.c                      |  10 +-
 hw/arm/digic_boards.c                    |   2 +-
 hw/arm/exynos4_boards.c                  |  16 +-
 hw/arm/gumstix.c                         |   6 +-
 hw/arm/highbank.c                        |  20 +--
 hw/arm/integratorcp.c                    |  12 +-
 hw/arm/kzm.c                             |  12 +-
 hw/arm/mainstone.c                       |  14 +-
 hw/arm/musicpal.c                        |  10 +-
 hw/arm/nseries.c                         |  22 +--
 hw/arm/omap_sx1.c                        |  20 +--
 hw/arm/palm.c                            |  10 +-
 hw/arm/realview.c                        |  44 +++---
 hw/arm/spitz.c                           |  26 ++--
 hw/arm/stellaris.c                       |  12 +-
 hw/arm/tosa.c                            |  10 +-
 hw/arm/versatilepb.c                     |  26 ++--
 hw/arm/vexpress.c                        |  20 +--
 hw/arm/virt.c                            |  16 +-
 hw/arm/xilinx_zynq.c                     |  12 +-
 hw/arm/z2.c                              |  10 +-
 hw/core/machine.c                        | 256 +++++++++++++++++++++++++++++++
 hw/core/null-machine.c                   |   2 +-
 hw/cris/axis_dev88.c                     |  10 +-
 hw/i386/pc_piix.c                        | 108 ++++++-------
 hw/i386/pc_q35.c                         |  66 ++++----
 hw/i386/xen_machine_pv.c                 |   8 +-
 hw/lm32/lm32_boards.c                    |  16 +-
 hw/lm32/milkymist.c                      |  10 +-
 hw/m68k/an5206.c                         |   8 +-
 hw/m68k/dummy_m68k.c                     |   8 +-
 hw/m68k/mcf5208.c                        |   8 +-
 hw/microblaze/petalogix_ml605_mmu.c      |   6 +-
 hw/microblaze/petalogix_s3adsp1800_mmu.c |   8 +-
 hw/mips/mips_fulong2e.c                  |  12 +-
 hw/mips/mips_jazz.c                      |  12 +-
 hw/mips/mips_malta.c                     |  12 +-
 hw/mips/mips_mipssim.c                   |  12 +-
 hw/mips/mips_r4k.c                       |  12 +-
 hw/moxie/moxiesim.c                      |  12 +-
 hw/openrisc/openrisc_sim.c               |   8 +-
 hw/ppc/e500.c                            |  44 +++---
 hw/ppc/e500.h                            |   2 +-
 hw/ppc/e500plat.c                        |   4 +-
 hw/ppc/mac_newworld.c                    |  14 +-
 hw/ppc/mac_oldworld.c                    |  14 +-
 hw/ppc/mpc8544ds.c                       |   4 +-
 hw/ppc/ppc405_boards.c                   |  18 +--
 hw/ppc/ppc440_bamboo.c                   |  12 +-
 hw/ppc/prep.c                            |  14 +-
 hw/ppc/spapr.c                           |  14 +-
 hw/ppc/virtex_ml507.c                    |  16 +-
 hw/s390x/s390-virtio-ccw.c               |  10 +-
 hw/s390x/s390-virtio.c                   |  10 +-
 hw/sh4/r2d.c                             |  10 +-
 hw/sh4/shix.c                            |   4 +-
 hw/sparc/leon3.c                         |   8 +-
 hw/sparc/sun4m.c                         |  64 ++++----
 hw/sparc64/sun4u.c                       |  36 ++---
 hw/unicore32/puv3.c                      |  10 +-
 hw/xtensa/xtensa_lx60.c                  |  26 ++--
 hw/xtensa/xtensa_sim.c                   |   8 +-
 include/hw/boards.h                      |  28 ++--
 qapi/qmp-output-visitor.c                |   5 +
 vl.c                                     |  23 +--
 67 files changed, 801 insertions(+), 543 deletions(-)

-- 
1.8.3.1

             reply	other threads:[~2014-05-07 14:46 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-07 14:42 Marcel Apfelbaum [this message]
2014-05-07 14:42 ` [Qemu-devel] [PATCH 1/4] machine: conversion of QEMUMachineInitArgs to MachineState Marcel Apfelbaum
2014-05-12 16:00   ` Laszlo Ersek
2014-05-13 13:25   ` Cornelia Huck
2014-05-13 15:44   ` Michael S. Tsirkin
2014-05-13 17:34   ` Andreas Färber
2014-05-15 15:04   ` Markus Armbruster
2014-05-18  8:37     ` Marcel Apfelbaum
2014-05-16 14:39   ` Igor Mammedov
2014-05-16 18:33     ` Andreas Färber
2014-05-18  8:51       ` Marcel Apfelbaum
2014-05-16 16:20   ` Igor Mammedov
2014-05-16 18:38     ` Andreas Färber
2014-05-18  8:48       ` Marcel Apfelbaum
2014-05-07 14:42 ` [Qemu-devel] [PATCH 2/4] qapi: output visitor crashes qemu if it encounters a NULL value Marcel Apfelbaum
2014-05-13 17:36   ` Andreas Färber
2014-05-13 19:08     ` Eric Blake
2014-05-14 17:00       ` Andreas Färber
2014-05-14 17:29         ` Marcel Apfelbaum
2014-05-14 18:25           ` Luiz Capitulino
2014-05-14 19:51             ` Markus Armbruster
2014-05-14 20:38             ` Michael Roth
2014-05-18  8:42               ` Marcel Apfelbaum
2014-05-14 20:26           ` Andreas Färber
2014-05-15 16:13   ` Markus Armbruster
2014-05-15 16:27     ` Michael Roth
2014-05-15 17:19       ` Markus Armbruster
2014-05-15 17:55         ` Michael Roth
2014-05-07 14:42 ` [Qemu-devel] [PATCH 3/4] vl.c: do not set 'type' property in obj_set_property Marcel Apfelbaum
2014-05-13 17:39   ` Andreas Färber
2014-05-15 16:15   ` Markus Armbruster
2014-05-15 16:38     ` Andreas Färber
2014-05-15 17:13       ` Paolo Bonzini
2014-05-07 14:43 ` [Qemu-devel] [PATCH 4/4] hw/machine: qemu machine opts as properties to QemuMachineState Marcel Apfelbaum
2014-05-13 17:54   ` Andreas Färber
2014-05-13 13:13 ` [Qemu-devel] [Qemu-detvel] [PATCH 0/4] machine: QemuOpts per machine Marcel Apfelbaum

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=1399473780-20374-1-git-send-email-marcel.a@redhat.com \
    --to=marcel.a@redhat.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=aik@ozlabs.ru \
    --cc=aliguori@amazon.com \
    --cc=armbru@redhat.com \
    --cc=aurelien@aurel32.net \
    --cc=blauwirbel@gmail.com \
    --cc=borntraeger@de.ibm.com \
    --cc=chouteau@adacore.com \
    --cc=cornelia.huck@de.ibm.com \
    --cc=edgar.iglesias@gmail.com \
    --cc=gxt@mprc.pku.edu.cn \
    --cc=hpoussin@reactos.org \
    --cc=jan.kiszka@web.de \
    --cc=jcmvbkbc@gmail.com \
    --cc=lcapitulino@redhat.com \
    --cc=lersek@redhat.com \
    --cc=mark.langsdorf@calxeda.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=michael@walle.cc \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.chubb@nicta.com.au \
    --cc=peter.crosthwaite@xilinx.com \
    --cc=proljc@gmail.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=scottwood@freescale.com \
    --cc=stefanha@redhat.com \
    /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).