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
next 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).