qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Eduardo Habkost <ehabkost@redhat.com>
To: qemu-devel@nongnu.org
Cc: pbonzini@redhat.com, jasowang@redhat.com, rth@twiddle.net,
	Alexander Graf <agraf@suse.de>,
	"Michael S. Tsirkin" <mst@redhat.com>
Subject: [Qemu-devel] [PATCH 00/10] pc: Don't use QEMUMachine, simplify compat+init code
Date: Fri, 15 May 2015 14:18:51 -0300	[thread overview]
Message-ID: <1431710341-24696-1-git-send-email-ehabkost@redhat.com> (raw)

This series changes the PC code to not need QEMUMachine anymore, removes ~200
lines of code and simplifies the steps needed to new PC machines in the future.

Previously, a new PC machine required:
 * A new PC_COMPAT_* macro;
 * A new pc_compat_*() function[1];
 * A new pc_init_*() function;
 * A new MACHINE_OPTIONS macro;
 * A new QEMUMachine declaration;
 * A new qemu_register_pc_machine() call in the machine_init() function.

Now it just needs:
 * A new PC_COMPAT_* macro;
 * A new pc_compat_*() function[1];
 * A new pc_*_machine_options() function;
 * A DEFINE_*_MACHINE line.

This series depends on the series I submitted recently:
  [PATCH v2 00/13] pc, hw, spapr: Cleanup of {HW, PC, SPAPR}_COMPAT_* macros

A git tree can be seen at:
  git://github.com/ehabkost/qemu-hacks.git work/pc-compat-macros

[1] I still plan to eliminate the need for the pc_compat_*() functions, and
move all compat data inside MachineClass/PCMachineClass, but that will be done
later.

Eduardo Habkost (10):
  pc: Define MACHINE_OPTIONS macros consistently for all machines
  pc: Define machines using a DEFINE_PC_MACHINE macro
  pc: Convert *_MACHINE_OPTIONS macros into functions
  pc: Move compat_props setting inside *_machine_options() functions
  pc: Don't use QEMUMachine anymore
  pc: Remove qemu_register_pc_machine() function
  machine: Remove unused fields from QEMUMachine
  piix: Add kvmclock_enabled, pci_enabled globals
  piix: Eliminate pc_init_pci()
  pc: Generate init functions with a macro

 hw/i386/pc.c         |  45 -----
 hw/i386/pc_piix.c    | 554 +++++++++++++++++++++------------------------------
 hw/i386/pc_q35.c     | 240 ++++++++--------------
 include/hw/boards.h  |  15 +-
 include/hw/i386/pc.h |  45 ++++-
 vl.c                 |  15 --
 6 files changed, 358 insertions(+), 556 deletions(-)

-- 
2.1.0

             reply	other threads:[~2015-05-15 17:19 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-15 17:18 Eduardo Habkost [this message]
2015-05-15 17:18 ` [Qemu-devel] [PATCH 01/10] pc: Define MACHINE_OPTIONS macros consistently for all machines Eduardo Habkost
2015-05-15 17:18 ` [Qemu-devel] [PATCH 02/10] pc: Define machines using a DEFINE_PC_MACHINE macro Eduardo Habkost
2015-05-15 17:18 ` [Qemu-devel] [PATCH 03/10] pc: Convert *_MACHINE_OPTIONS macros into functions Eduardo Habkost
2015-05-15 17:18 ` [Qemu-devel] [PATCH 04/10] pc: Move compat_props setting inside *_machine_options() functions Eduardo Habkost
2015-05-15 17:18 ` [Qemu-devel] [PATCH 05/10] pc: Don't use QEMUMachine anymore Eduardo Habkost
2015-05-15 17:18 ` [Qemu-devel] [PATCH 06/10] pc: Remove qemu_register_pc_machine() function Eduardo Habkost
2015-05-15 17:18 ` [Qemu-devel] [PATCH 07/10] machine: Remove unused fields from QEMUMachine Eduardo Habkost
2015-05-15 17:18 ` [Qemu-devel] [PATCH 08/10] piix: Add kvmclock_enabled, pci_enabled globals Eduardo Habkost
2015-05-15 17:19 ` [Qemu-devel] [PATCH 09/10] piix: Eliminate pc_init_pci() Eduardo Habkost
2015-05-15 17:19 ` [Qemu-devel] [PATCH 10/10] pc: Generate init functions with a macro Eduardo Habkost

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=1431710341-24696-1-git-send-email-ehabkost@redhat.com \
    --to=ehabkost@redhat.com \
    --cc=agraf@suse.de \
    --cc=jasowang@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /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).