From: Anthony Liguori <anthony@codemonkey.ws>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
Paolo Bonzini <pbonzini@redhat.com>,
qemu-devel@nongnu.org, Markus Armbruster <armbru@redhat.com>,
Jan Kiszka <jan.kiszka@siemens.com>
Subject: Re: [Qemu-devel] [PATCH 00/14] qdev: assign unique names to all devices (part 1)
Date: Fri, 16 Sep 2011 11:22:15 -0500 [thread overview]
Message-ID: <4E7377B7.1040109@codemonkey.ws> (raw)
In-Reply-To: <1316188834-13675-1-git-send-email-aliguori@us.ibm.com>
On 09/16/2011 11:00 AM, Anthony Liguori wrote:
> This series introduces an infrastructure to remove anonymous devices from qdev.
> Anonymous devices are one of the big gaps between qdev and QOM so removing is
> a prerequisite to incrementally merging QOM.
>
> Besides the infrastructure, I also converted almost all of the possible PC
> devices to have unique names. Please not that naming is not a property of
> devices but rather of the thing that creates the devices (usually machines).
>
> The names are ugly but this is because of the alternating device/bus hierarchy
> in qdev. For now, the names use '::' as deliminators but I think Jan has
> convinced me that down the road, we should use '/' as a deliminator such that
> the resulting names are actually valid paths (using a canonical path format).
>
>
Here's a diffstat:
apb_pci.c | 2
apic.c | 2
arm11mpcore.c | 4 -
arm_sysctl.c | 2
armv7m.c | 6 +-
axis_dev88.c | 8 +--
bonito.c | 2
collie.c | 2
cs4231a.c | 2
empty_slot.c | 2
escc.c | 4 -
esp.c | 2
etraxfs.h | 2
fdc.c | 4 -
fdc.h | 2
fw_cfg.c | 2
grackle_pci.c | 2
grlib.h | 6 +-
gt64xxx.c | 2
gus.c | 2
i2c.c | 2
ide/isa.c | 2
ide/qdev.c | 3 -
integratorcp.c | 21 +++++-----
intel-hda.c | 2
isa-bus.c | 32 +++++++++++----
isa.h | 9 ++--
kvmclock.c | 2
lan9118.c | 2
lm32.h | 4 -
lm32_boards.c | 14 +++---
m48t59.c | 4 -
mainstone.c | 2
mc146818rtc.c | 2
milkymist-hw.h | 22 +++++-----
mips_fulong2e.c | 2
mips_jazz.c | 4 -
mips_malta.c | 2
mips_mipssim.c | 2
mips_r4k.c | 2
musicpal.c | 21 +++++-----
nand.c | 2
nseries.c | 4 -
omap1.c | 2
omap2.c | 2
pc.c | 15 +++----
pc.h | 12 ++---
pc_piix.c | 8 ++-
pci.c | 35 +++++++++++-----
petalogix_s3adsp1800_mmu.c | 2
piix_pci.c | 2
ppc_prep.c | 2
ppce500_mpc8544ds.c | 4 -
pxa2xx.c | 24 +++++++----
pxa2xx_dma.c | 4 -
pxa2xx_gpio.c | 2
pxa2xx_pic.c | 2
qdev.c | 94 +++++++++++++++++++++++++++++++++++++++------
qdev.h | 11 +++--
r2d.c | 4 -
realview.c | 40 +++++++++----------
s390-virtio-bus.c | 2
s390-virtio.c | 4 -
sb16.c | 2
scsi-bus.c | 2
sm501.c | 2
smbus_eeprom.c | 3 -
smc91c111.c | 2
spapr_llan.c | 2
spapr_vio.c | 2
spapr_vscsi.c | 2
spapr_vty.c | 2
spitz.c | 8 +--
ssi.c | 2
stellaris.c | 14 +++---
strongarm.c | 17 ++++----
sun4m.c | 32 +++++++--------
sun4u.c | 6 +-
syborg.c | 22 +++++-----
sysbus.c | 67 +++++++++++++++++++++++++++++---
sysbus.h | 31 ++++++++------
tosa.c | 4 -
unin_pci.c | 8 +--
usb-bus.c | 2
versatilepb.c | 34 ++++++++--------
vexpress.c | 28 ++++++-------
xilinx.h | 10 ++--
87 files changed, 490 insertions(+), 307 deletions(-)
Regards,
Anthony Liguori
next prev parent reply other threads:[~2011-09-16 16:22 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-16 16:00 [Qemu-devel] [PATCH 00/14] qdev: assign unique names to all devices (part 1) Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 01/14] apic: rename apic.id -> apic.index Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 02/14] qdev: enforce that no devices overload the id property Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 03/14] qdev: push id into qdev_create calls Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 04/14] qdev: take ownership of id pointer Anthony Liguori
2011-09-19 7:34 ` Gerd Hoffmann
2011-09-19 16:27 ` Anthony Liguori
2011-09-20 6:36 ` Gerd Hoffmann
2011-09-20 13:04 ` Anthony Liguori
2011-09-20 13:21 ` Gerd Hoffmann
2011-09-20 13:55 ` Anthony Liguori
2011-09-20 14:11 ` Gerd Hoffmann
2011-09-16 16:00 ` [Qemu-devel] [PATCH 05/14] qdev: remove opts pointer tracking Anthony Liguori
2011-09-19 7:35 ` Gerd Hoffmann
2011-09-16 16:00 ` [Qemu-devel] [PATCH 06/14] qdev: add ability to do QOM-style derived naming Anthony Liguori
2011-09-17 18:39 ` Blue Swirl
2011-09-16 16:00 ` [Qemu-devel] [PATCH 07/14] sysbus: add an id argument to sysbus_create_simple() Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 08/14] sysbus: make create_varargs take an id Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 09/14] fw_cfg: add name to global fw_cfg device Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 10/14] isa: add name parameter to device creation Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 11/14] pci: obtain devfn before initializing the device Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 12/14] pci: give pci devices a default name Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 13/14] ide: give IDE drives a default name in qdev Anthony Liguori
2011-09-16 16:00 ` [Qemu-devel] [PATCH 14/14] pc: assign names to machine created devices Anthony Liguori
2011-09-16 16:22 ` Anthony Liguori [this message]
2011-09-16 16:48 ` [Qemu-devel] [PATCH 00/14] qdev: assign unique names to all devices (part 1) Jan Kiszka
2011-09-16 16:54 ` Anthony Liguori
2011-09-16 17:03 ` Jan Kiszka
2011-09-16 18:06 ` Anthony Liguori
2011-09-16 17:11 ` Kevin Wolf
2011-09-16 18:03 ` Anthony Liguori
2011-09-19 7:26 ` Jan Kiszka
2011-09-19 14:05 ` Anthony Liguori
2011-09-19 14:24 ` Kevin Wolf
2011-09-20 8:32 ` Jan Kiszka
2011-09-19 7:41 ` Kevin Wolf
2011-09-16 18:21 ` Anthony Liguori
2011-09-19 7:34 ` Jan Kiszka
2011-09-17 18:41 ` Blue Swirl
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=4E7377B7.1040109@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=aliguori@us.ibm.com \
--cc=armbru@redhat.com \
--cc=jan.kiszka@siemens.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.