From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42573) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X2Oqr-00017Y-8K for qemu-devel@nongnu.org; Wed, 02 Jul 2014 14:01:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X2Oqd-0004HB-M9 for qemu-devel@nongnu.org; Wed, 02 Jul 2014 14:01:49 -0400 From: Alexander Graf Date: Wed, 2 Jul 2014 20:01:30 +0200 Message-Id: <1404324093-19225-7-git-send-email-agraf@suse.de> In-Reply-To: <1404324093-19225-1-git-send-email-agraf@suse.de> References: <1404324093-19225-1-git-send-email-agraf@suse.de> Subject: [Qemu-devel] [PATCH v2 6/9] sysbus: Make devices spawnable via -device List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-ppc@nongnu.org Cc: peter.maydell@linaro.org, peter.crosthwaite@xilinx.com, eric.auger@linaro.org, qemu-devel@nongnu.org, sean.stalley@intel.com, pbonzini@redhat.com, afaerber@suse.de Now that we can properly map sysbus devices that haven't been connected to something forcefully by C code, we can allow the -device command line option to spawn them. For machines that don't implement dynamic sysbus assignment in their board files we add a new bool "has_dynamic_sysbus" to the machine class. When that property is false (default), we bail out when we see dynamically spawned sysbus devices, like we did before. Signed-off-by: Alexander Graf --- v1 -> v2: - use bool in MachineClass rather than property --- hw/core/machine.c | 43 +++++++++++++++++++++++++++++++++++++++++++ hw/core/sysbus.c | 7 ------- include/hw/boards.h | 8 ++++++-- vl.c | 1 + 4 files changed, 50 insertions(+), 9 deletions(-) diff --git a/hw/core/machine.c b/hw/core/machine.c index c25cc07..713c9d8 100644 --- a/hw/core/machine.c +++ b/hw/core/machine.c @@ -13,6 +13,9 @@ #include "hw/boards.h" #include "qapi/visitor.h" #include "qom/property.h" +#include "hw/sysbus.h" +#include "sysemu/sysemu.h" +#include "qemu/error-report.h" static char *machine_get_accel(Object *obj, Error **errp) { @@ -236,8 +239,44 @@ static void machine_set_firmware(Object *obj, const char *value, Error **errp) ms->firmware = g_strdup(value); } +static int search_for_sysbus_device(Object *obj, void *opaque) +{ + if (!object_dynamic_cast(obj, TYPE_SYS_BUS_DEVICE)) { + /* Container or different device, traverse it for children */ + return object_child_foreach(obj, search_for_sysbus_device, opaque); + } + + error_report("Device '%s' can not be handled by this machine", + qdev_fw_name(DEVICE(obj))); + exit(1); +} + +static void machine_init_notify(Notifier *notifier, void *data) +{ + Object *machine = qdev_get_machine(); + ObjectClass *oc = object_get_class(machine); + MachineClass *mc = MACHINE_CLASS(oc); + Object *container; + + if (mc->has_dynamic_sysbus) { + /* Our machine can handle dynamic sysbus devices, we're all good */ + return; + } + + /* + * Loop through all dynamically created devices and check whether there + * are sysbus devices among them. If there are, error out. + */ + container = container_get(machine, "/peripheral"); + search_for_sysbus_device(container, NULL); + container = container_get(machine, "/peripheral-anon"); + search_for_sysbus_device(container, NULL); +} + static void machine_initfn(Object *obj) { + MachineState *ms = MACHINE(obj); + object_property_add_str(obj, "accel", machine_get_accel, machine_set_accel, NULL); object_property_add_bool(obj, "kernel_irqchip", @@ -275,6 +314,10 @@ static void machine_initfn(Object *obj) object_property_add_bool(obj, "usb", machine_get_usb, machine_set_usb, NULL); object_property_add_str(obj, "firmware", machine_get_firmware, machine_set_firmware, NULL); + + /* Register notifier when init is done for sysbus sanity checks */ + ms->sysbus_notifier.notify = machine_init_notify; + qemu_add_machine_init_done_notifier(&ms->sysbus_notifier); } static void machine_finalize(Object *obj) diff --git a/hw/core/sysbus.c b/hw/core/sysbus.c index e551e16..aacc446 100644 --- a/hw/core/sysbus.c +++ b/hw/core/sysbus.c @@ -294,13 +294,6 @@ static void sysbus_device_class_init(ObjectClass *klass, void *data) DeviceClass *k = DEVICE_CLASS(klass); k->init = sysbus_device_init; k->bus_type = TYPE_SYSTEM_BUS; - /* - * device_add plugs devices into suitable bus. For "real" buses, - * that actually connects the device. For sysbus, the connections - * need to be made separately, and device_add can't do that. The - * device would be left unconnected, and could not possibly work. - */ - k->cannot_instantiate_with_device_add_yet = true; } static const TypeInfo sysbus_device_type_info = { diff --git a/include/hw/boards.h b/include/hw/boards.h index 605a970..9514c62 100644 --- a/include/hw/boards.h +++ b/include/hw/boards.h @@ -35,7 +35,8 @@ struct QEMUMachine { use_sclp:1, no_floppy:1, no_cdrom:1, - no_sdcard:1; + no_sdcard:1, + has_dynamic_sysbus:1; int is_default; const char *default_machine_opts; const char *default_boot_order; @@ -93,7 +94,8 @@ struct MachineClass { use_sclp:1, no_floppy:1, no_cdrom:1, - no_sdcard:1; + no_sdcard:1, + has_dynamic_sysbus:1; int is_default; const char *default_machine_opts; const char *default_boot_order; @@ -110,6 +112,8 @@ struct MachineClass { struct MachineState { /*< private >*/ Object parent_obj; + Notifier sysbus_notifier; + /*< public >*/ char *accel; diff --git a/vl.c b/vl.c index 6e084c2..1df8d4e 100644 --- a/vl.c +++ b/vl.c @@ -1569,6 +1569,7 @@ static void machine_class_init(ObjectClass *oc, void *data) mc->no_floppy = qm->no_floppy; mc->no_cdrom = qm->no_cdrom; mc->no_sdcard = qm->no_sdcard; + mc->has_dynamic_sysbus = qm->has_dynamic_sysbus; mc->is_default = qm->is_default; mc->default_machine_opts = qm->default_machine_opts; mc->default_boot_order = qm->default_boot_order; -- 1.8.1.4