2009-05-15 05:35:06 +08:00
|
|
|
/*
|
|
|
|
* Dynamic device configuration and creation.
|
|
|
|
*
|
|
|
|
* Copyright (c) 2009 CodeSourcery
|
|
|
|
*
|
|
|
|
* This library is free software; you can redistribute it and/or
|
|
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
|
|
* License as published by the Free Software Foundation; either
|
|
|
|
* version 2 of the License, or (at your option) any later version.
|
|
|
|
*
|
|
|
|
* This library is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
|
|
* Lesser General Public License for more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU Lesser General Public
|
2009-07-17 04:47:01 +08:00
|
|
|
* License along with this library; if not, see <http://www.gnu.org/licenses/>.
|
2009-05-15 05:35:06 +08:00
|
|
|
*/
|
|
|
|
|
|
|
|
/* The theory here is that it should be possible to create a machine without
|
|
|
|
knowledge of specific devices. Historically board init routines have
|
|
|
|
passed a bunch of arguments to each device, requiring the board know
|
|
|
|
exactly which device it is dealing with. This file provides an abstract
|
|
|
|
API for device configuration and initialization. Devices will generally
|
|
|
|
inherit from a particular bus (e.g. PCI or I2C) rather than
|
|
|
|
this API directly. */
|
|
|
|
|
2013-02-04 22:40:22 +08:00
|
|
|
#include "hw/qdev.h"
|
2014-03-17 10:40:23 +08:00
|
|
|
#include "hw/fw-path-provider.h"
|
2012-12-18 01:20:04 +08:00
|
|
|
#include "sysemu/sysemu.h"
|
2012-12-18 01:19:43 +08:00
|
|
|
#include "qapi/error.h"
|
2013-02-04 18:37:52 +08:00
|
|
|
#include "qapi/qmp/qerror.h"
|
2012-12-18 01:19:43 +08:00
|
|
|
#include "qapi/visitor.h"
|
2013-03-06 20:58:59 +08:00
|
|
|
#include "qapi/qmp/qjson.h"
|
2014-02-05 23:36:45 +08:00
|
|
|
#include "hw/hotplug.h"
|
2014-06-02 21:25:03 +08:00
|
|
|
#include "hw/boards.h"
|
2014-06-18 14:43:43 +08:00
|
|
|
#include "qapi-event.h"
|
2009-05-15 05:35:06 +08:00
|
|
|
|
2011-12-23 05:24:20 +08:00
|
|
|
int qdev_hotplug = 0;
|
2011-01-05 03:37:50 +08:00
|
|
|
static bool qdev_hot_added = false;
|
|
|
|
static bool qdev_hot_removed = false;
|
2009-09-26 03:42:41 +08:00
|
|
|
|
2011-12-10 00:51:49 +08:00
|
|
|
const VMStateDescription *qdev_get_vmsd(DeviceState *dev)
|
|
|
|
{
|
2011-12-10 01:06:57 +08:00
|
|
|
DeviceClass *dc = DEVICE_GET_CLASS(dev);
|
|
|
|
return dc->vmsd;
|
2011-12-10 00:51:49 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
const char *qdev_fw_name(DeviceState *dev)
|
|
|
|
{
|
2011-12-10 01:06:57 +08:00
|
|
|
DeviceClass *dc = DEVICE_GET_CLASS(dev);
|
2011-12-10 00:51:49 +08:00
|
|
|
|
2011-12-10 01:06:57 +08:00
|
|
|
if (dc->fw_name) {
|
|
|
|
return dc->fw_name;
|
2011-12-10 00:51:49 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return object_get_typename(OBJECT(dev));
|
|
|
|
}
|
|
|
|
|
2011-12-19 00:05:11 +08:00
|
|
|
static void qdev_property_add_legacy(DeviceState *dev, Property *prop,
|
|
|
|
Error **errp);
|
|
|
|
|
2011-12-24 05:34:39 +08:00
|
|
|
static void bus_remove_child(BusState *bus, DeviceState *child)
|
2010-02-20 02:12:18 +08:00
|
|
|
{
|
2011-12-24 05:34:39 +08:00
|
|
|
BusChild *kid;
|
|
|
|
|
|
|
|
QTAILQ_FOREACH(kid, &bus->children, sibling) {
|
|
|
|
if (kid->child == child) {
|
|
|
|
char name[32];
|
|
|
|
|
|
|
|
snprintf(name, sizeof(name), "child[%d]", kid->index);
|
|
|
|
QTAILQ_REMOVE(&bus->children, kid, sibling);
|
2013-01-25 21:12:32 +08:00
|
|
|
|
|
|
|
/* This gives back ownership of kid->child back to us. */
|
2011-12-24 05:34:39 +08:00
|
|
|
object_property_del(OBJECT(bus), name, NULL);
|
2013-01-25 21:12:32 +08:00
|
|
|
object_unref(OBJECT(kid->child));
|
2011-12-24 05:34:39 +08:00
|
|
|
g_free(kid);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static void bus_add_child(BusState *bus, DeviceState *child)
|
|
|
|
{
|
|
|
|
char name[32];
|
|
|
|
BusChild *kid = g_malloc0(sizeof(*kid));
|
2010-02-20 02:12:18 +08:00
|
|
|
|
2011-12-24 05:34:39 +08:00
|
|
|
kid->index = bus->max_index++;
|
|
|
|
kid->child = child;
|
2013-01-25 21:12:32 +08:00
|
|
|
object_ref(OBJECT(kid->child));
|
2011-12-13 04:29:27 +08:00
|
|
|
|
2011-12-24 05:34:39 +08:00
|
|
|
QTAILQ_INSERT_HEAD(&bus->children, kid, sibling);
|
|
|
|
|
2013-01-25 21:12:32 +08:00
|
|
|
/* This transfers ownership of kid->child to the property. */
|
2011-12-24 05:34:39 +08:00
|
|
|
snprintf(name, sizeof(name), "child[%d]", kid->index);
|
|
|
|
object_property_add_link(OBJECT(bus), name,
|
|
|
|
object_get_typename(OBJECT(child)),
|
2014-03-19 15:58:56 +08:00
|
|
|
(Object **)&kid->child,
|
|
|
|
NULL, /* read-only property */
|
|
|
|
0, /* return ownership on prop deletion */
|
|
|
|
NULL);
|
2011-12-24 05:34:39 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
void qdev_set_parent_bus(DeviceState *dev, BusState *bus)
|
|
|
|
{
|
2011-12-23 05:14:27 +08:00
|
|
|
dev->parent_bus = bus;
|
2013-01-25 21:12:35 +08:00
|
|
|
object_ref(OBJECT(bus));
|
2011-12-24 05:34:39 +08:00
|
|
|
bus_add_child(bus, dev);
|
2010-02-20 02:12:18 +08:00
|
|
|
}
|
|
|
|
|
2014-09-26 17:28:22 +08:00
|
|
|
static void qbus_set_hotplug_handler_internal(BusState *bus, Object *handler,
|
|
|
|
Error **errp)
|
|
|
|
{
|
|
|
|
|
|
|
|
object_property_set_link(OBJECT(bus), OBJECT(handler),
|
|
|
|
QDEV_HOTPLUG_HANDLER_PROPERTY, errp);
|
|
|
|
}
|
|
|
|
|
|
|
|
void qbus_set_hotplug_handler(BusState *bus, DeviceState *handler, Error **errp)
|
|
|
|
{
|
|
|
|
qbus_set_hotplug_handler_internal(bus, OBJECT(handler), errp);
|
|
|
|
}
|
|
|
|
|
|
|
|
void qbus_set_bus_hotplug_handler(BusState *bus, Error **errp)
|
|
|
|
{
|
|
|
|
qbus_set_hotplug_handler_internal(bus, OBJECT(bus), errp);
|
|
|
|
}
|
|
|
|
|
2009-05-15 05:35:06 +08:00
|
|
|
/* Create a new device. This only initializes the device state structure
|
|
|
|
and allows properties to be set. qdev_init should be called to
|
|
|
|
initialize the actual device emulation. */
|
2009-05-23 07:05:19 +08:00
|
|
|
DeviceState *qdev_create(BusState *bus, const char *name)
|
2011-02-05 22:34:25 +08:00
|
|
|
{
|
|
|
|
DeviceState *dev;
|
|
|
|
|
|
|
|
dev = qdev_try_create(bus, name);
|
|
|
|
if (!dev) {
|
2011-08-04 06:49:04 +08:00
|
|
|
if (bus) {
|
error: Strip trailing '\n' from error string arguments (again)
Commit 6daf194d and be62a2eb got rid of a bunch, but they keep coming
back. Tracked down with this Coccinelle semantic patch:
@r@
expression err, eno, cls, fmt;
position p;
@@
(
error_report(fmt, ...)@p
|
error_set(err, cls, fmt, ...)@p
|
error_set_errno(err, eno, cls, fmt, ...)@p
|
error_setg(err, fmt, ...)@p
|
error_setg_errno(err, eno, fmt, ...)@p
)
@script:python@
fmt << r.fmt;
p << r.p;
@@
if "\\n" in str(fmt):
print "%s:%s:%s:%s" % (p[0].file, p[0].line, p[0].column, fmt)
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-id: 1360354939-10994-4-git-send-email-armbru@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
2013-02-09 04:22:16 +08:00
|
|
|
error_report("Unknown device '%s' for bus '%s'", name,
|
2012-12-04 21:19:34 +08:00
|
|
|
object_get_typename(OBJECT(bus)));
|
2011-08-04 06:49:04 +08:00
|
|
|
} else {
|
error: Strip trailing '\n' from error string arguments (again)
Commit 6daf194d and be62a2eb got rid of a bunch, but they keep coming
back. Tracked down with this Coccinelle semantic patch:
@r@
expression err, eno, cls, fmt;
position p;
@@
(
error_report(fmt, ...)@p
|
error_set(err, cls, fmt, ...)@p
|
error_set_errno(err, eno, cls, fmt, ...)@p
|
error_setg(err, fmt, ...)@p
|
error_setg_errno(err, eno, fmt, ...)@p
)
@script:python@
fmt << r.fmt;
p << r.p;
@@
if "\\n" in str(fmt):
print "%s:%s:%s:%s" % (p[0].file, p[0].line, p[0].column, fmt)
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-id: 1360354939-10994-4-git-send-email-armbru@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
2013-02-09 04:22:16 +08:00
|
|
|
error_report("Unknown device '%s' for default sysbus", name);
|
2011-08-04 06:49:04 +08:00
|
|
|
}
|
2013-03-22 16:44:14 +08:00
|
|
|
abort();
|
2011-02-05 22:34:25 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return dev;
|
|
|
|
}
|
|
|
|
|
2012-03-28 00:38:47 +08:00
|
|
|
DeviceState *qdev_try_create(BusState *bus, const char *type)
|
2009-05-15 05:35:06 +08:00
|
|
|
{
|
2011-12-23 05:14:27 +08:00
|
|
|
DeviceState *dev;
|
|
|
|
|
2012-03-28 00:38:47 +08:00
|
|
|
if (object_class_by_name(type) == NULL) {
|
2012-02-17 09:47:44 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
2012-03-28 00:38:47 +08:00
|
|
|
dev = DEVICE(object_new(type));
|
2011-12-23 05:14:27 +08:00
|
|
|
if (!dev) {
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2009-06-30 20:12:08 +08:00
|
|
|
if (!bus) {
|
2010-12-17 02:33:22 +08:00
|
|
|
bus = sysbus_get_default();
|
2009-06-30 20:12:08 +08:00
|
|
|
}
|
|
|
|
|
2011-12-23 05:14:27 +08:00
|
|
|
qdev_set_parent_bus(dev, bus);
|
2013-01-25 21:12:37 +08:00
|
|
|
object_unref(OBJECT(dev));
|
2011-12-23 05:14:27 +08:00
|
|
|
return dev;
|
2009-05-15 05:35:06 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/* Initialize a device. Device properties should be set before calling
|
|
|
|
this function. IRQs and MMIO regions should be connected/mapped after
|
2009-10-07 07:15:56 +08:00
|
|
|
calling this function.
|
|
|
|
On failure, destroy the device and return negative value.
|
|
|
|
Return 0 on success. */
|
2009-08-14 16:36:05 +08:00
|
|
|
int qdev_init(DeviceState *dev)
|
2009-05-15 05:35:06 +08:00
|
|
|
{
|
2013-01-09 10:58:11 +08:00
|
|
|
Error *local_err = NULL;
|
2009-09-01 15:56:12 +08:00
|
|
|
|
2013-01-09 10:58:10 +08:00
|
|
|
assert(!dev->realized);
|
2011-12-10 01:06:57 +08:00
|
|
|
|
2013-01-09 10:58:11 +08:00
|
|
|
object_property_set_bool(OBJECT(dev), true, "realized", &local_err);
|
|
|
|
if (local_err != NULL) {
|
2013-04-29 20:35:08 +08:00
|
|
|
qerror_report_err(local_err);
|
2013-01-09 10:58:11 +08:00
|
|
|
error_free(local_err);
|
2013-09-11 20:54:09 +08:00
|
|
|
object_unparent(OBJECT(dev));
|
2013-01-09 10:58:11 +08:00
|
|
|
return -1;
|
2009-10-07 07:15:56 +08:00
|
|
|
}
|
2013-01-09 10:58:11 +08:00
|
|
|
return 0;
|
|
|
|
}
|
2012-03-28 00:38:47 +08:00
|
|
|
|
2014-04-25 18:44:21 +08:00
|
|
|
static void device_realize(DeviceState *dev, Error **errp)
|
2013-01-09 10:58:11 +08:00
|
|
|
{
|
|
|
|
DeviceClass *dc = DEVICE_GET_CLASS(dev);
|
2012-03-28 00:38:47 +08:00
|
|
|
|
2013-01-09 10:58:11 +08:00
|
|
|
if (dc->init) {
|
|
|
|
int rc = dc->init(dev);
|
|
|
|
if (rc < 0) {
|
2014-04-25 18:44:21 +08:00
|
|
|
error_setg(errp, "Device initialization failed.");
|
2013-01-09 10:58:11 +08:00
|
|
|
return;
|
|
|
|
}
|
2011-07-25 01:38:36 +08:00
|
|
|
}
|
2009-05-23 07:05:19 +08:00
|
|
|
}
|
|
|
|
|
2013-04-16 00:34:10 +08:00
|
|
|
static void device_unrealize(DeviceState *dev, Error **errp)
|
|
|
|
{
|
|
|
|
DeviceClass *dc = DEVICE_GET_CLASS(dev);
|
|
|
|
|
|
|
|
if (dc->exit) {
|
|
|
|
int rc = dc->exit(dev);
|
|
|
|
if (rc < 0) {
|
|
|
|
error_setg(errp, "Device exit failed.");
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2010-05-15 19:32:40 +08:00
|
|
|
void qdev_set_legacy_instance_id(DeviceState *dev, int alias_id,
|
|
|
|
int required_for_version)
|
|
|
|
{
|
2013-01-09 10:58:10 +08:00
|
|
|
assert(!dev->realized);
|
2010-05-15 19:32:40 +08:00
|
|
|
dev->instance_id_alias = alias_id;
|
|
|
|
dev->alias_required_for_version = required_for_version;
|
|
|
|
}
|
|
|
|
|
2014-09-26 17:28:41 +08:00
|
|
|
static HotplugHandler *qdev_get_hotplug_handler(DeviceState *dev)
|
|
|
|
{
|
|
|
|
HotplugHandler *hotplug_ctrl = NULL;
|
|
|
|
|
|
|
|
if (dev->parent_bus && dev->parent_bus->hotplug_handler) {
|
|
|
|
hotplug_ctrl = dev->parent_bus->hotplug_handler;
|
|
|
|
} else if (object_dynamic_cast(qdev_get_machine(), TYPE_MACHINE)) {
|
|
|
|
MachineState *machine = MACHINE(qdev_get_machine());
|
|
|
|
MachineClass *mc = MACHINE_GET_CLASS(machine);
|
|
|
|
|
|
|
|
if (mc->get_hotplug_handler) {
|
|
|
|
hotplug_ctrl = mc->get_hotplug_handler(machine, dev);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return hotplug_ctrl;
|
|
|
|
}
|
|
|
|
|
2012-03-15 04:37:38 +08:00
|
|
|
void qdev_unplug(DeviceState *dev, Error **errp)
|
2009-09-26 03:42:41 +08:00
|
|
|
{
|
2011-12-10 01:06:57 +08:00
|
|
|
DeviceClass *dc = DEVICE_GET_CLASS(dev);
|
2014-09-26 17:28:41 +08:00
|
|
|
HotplugHandler *hotplug_ctrl;
|
|
|
|
HotplugHandlerClass *hdc;
|
2011-12-10 01:06:57 +08:00
|
|
|
|
2014-09-26 17:28:17 +08:00
|
|
|
if (dev->parent_bus && !qbus_is_hotpluggable(dev->parent_bus)) {
|
2012-03-15 04:37:38 +08:00
|
|
|
error_set(errp, QERR_BUS_NO_HOTPLUG, dev->parent_bus->name);
|
|
|
|
return;
|
2009-09-26 03:42:41 +08:00
|
|
|
}
|
2009-11-02 17:26:41 +08:00
|
|
|
|
2014-02-05 23:36:46 +08:00
|
|
|
if (!dc->hotpluggable) {
|
|
|
|
error_set(errp, QERR_DEVICE_NO_HOTPLUG,
|
|
|
|
object_get_typename(OBJECT(dev)));
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2011-01-05 03:37:50 +08:00
|
|
|
qdev_hot_removed = true;
|
|
|
|
|
2014-09-26 17:28:41 +08:00
|
|
|
hotplug_ctrl = qdev_get_hotplug_handler(dev);
|
|
|
|
/* hotpluggable device MUST have HotplugHandler, if it doesn't
|
|
|
|
* then something is very wrong with it */
|
|
|
|
g_assert(hotplug_ctrl);
|
|
|
|
|
|
|
|
/* If device supports async unplug just request it to be done,
|
|
|
|
* otherwise just remove it synchronously */
|
|
|
|
hdc = HOTPLUG_HANDLER_GET_CLASS(hotplug_ctrl);
|
|
|
|
if (hdc->unplug_request) {
|
|
|
|
hotplug_handler_unplug_request(hotplug_ctrl, dev, errp);
|
2014-02-05 23:36:52 +08:00
|
|
|
} else {
|
2014-09-26 17:28:41 +08:00
|
|
|
hotplug_handler_unplug(hotplug_ctrl, dev, errp);
|
2012-03-15 04:37:38 +08:00
|
|
|
}
|
2009-09-26 03:42:41 +08:00
|
|
|
}
|
|
|
|
|
2010-11-19 17:55:59 +08:00
|
|
|
static int qdev_reset_one(DeviceState *dev, void *opaque)
|
|
|
|
{
|
2011-12-05 01:36:01 +08:00
|
|
|
device_reset(dev);
|
2010-11-19 17:55:59 +08:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2010-11-19 17:56:00 +08:00
|
|
|
static int qbus_reset_one(BusState *bus, void *opaque)
|
|
|
|
{
|
2012-05-02 15:00:20 +08:00
|
|
|
BusClass *bc = BUS_GET_CLASS(bus);
|
|
|
|
if (bc->reset) {
|
2013-12-07 00:54:27 +08:00
|
|
|
bc->reset(bus);
|
2010-11-19 17:56:00 +08:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2010-11-19 17:56:01 +08:00
|
|
|
void qdev_reset_all(DeviceState *dev)
|
|
|
|
{
|
2013-12-07 00:54:27 +08:00
|
|
|
qdev_walk_children(dev, NULL, NULL, qdev_reset_one, qbus_reset_one, NULL);
|
2010-11-19 17:56:01 +08:00
|
|
|
}
|
|
|
|
|
2013-01-10 22:49:07 +08:00
|
|
|
void qbus_reset_all(BusState *bus)
|
|
|
|
{
|
2013-12-07 00:54:27 +08:00
|
|
|
qbus_walk_children(bus, NULL, NULL, qdev_reset_one, qbus_reset_one, NULL);
|
2013-01-10 22:49:07 +08:00
|
|
|
}
|
|
|
|
|
2010-12-20 13:33:35 +08:00
|
|
|
void qbus_reset_all_fn(void *opaque)
|
|
|
|
{
|
|
|
|
BusState *bus = opaque;
|
2013-01-10 22:49:07 +08:00
|
|
|
qbus_reset_all(bus);
|
2010-12-20 13:33:35 +08:00
|
|
|
}
|
|
|
|
|
2009-09-26 03:42:41 +08:00
|
|
|
/* can be used as ->unplug() callback for the simple cases */
|
2014-09-26 17:28:21 +08:00
|
|
|
void qdev_simple_device_unplug_cb(HotplugHandler *hotplug_dev,
|
|
|
|
DeviceState *dev, Error **errp)
|
|
|
|
{
|
2014-09-26 17:28:40 +08:00
|
|
|
/* just zap it */
|
|
|
|
object_unparent(OBJECT(dev));
|
2014-09-26 17:28:21 +08:00
|
|
|
}
|
2011-04-06 21:51:59 +08:00
|
|
|
|
|
|
|
/* Like qdev_init(), but terminate program via error_report() instead of
|
2009-10-07 07:15:58 +08:00
|
|
|
returning an error value. This is okay during machine creation.
|
|
|
|
Don't use for hotplug, because there callers need to recover from
|
|
|
|
failure. Exception: if you know the device's init() callback can't
|
|
|
|
fail, then qdev_init_nofail() can't fail either, and is therefore
|
|
|
|
usable even then. But relying on the device implementation that
|
|
|
|
way is somewhat unclean, and best avoided. */
|
|
|
|
void qdev_init_nofail(DeviceState *dev)
|
|
|
|
{
|
2012-06-27 20:37:54 +08:00
|
|
|
const char *typename = object_get_typename(OBJECT(dev));
|
|
|
|
|
2010-05-28 03:23:08 +08:00
|
|
|
if (qdev_init(dev) < 0) {
|
2012-06-27 20:37:54 +08:00
|
|
|
error_report("Initialization of device %s failed", typename);
|
2010-05-28 03:23:08 +08:00
|
|
|
exit(1);
|
|
|
|
}
|
2009-10-07 07:15:58 +08:00
|
|
|
}
|
|
|
|
|
2009-09-26 03:42:41 +08:00
|
|
|
void qdev_machine_creation_done(void)
|
|
|
|
{
|
|
|
|
/*
|
|
|
|
* ok, initial machine setup is done, starting from now we can
|
|
|
|
* only create hotpluggable devices
|
|
|
|
*/
|
|
|
|
qdev_hotplug = 1;
|
|
|
|
}
|
|
|
|
|
2011-01-05 03:37:50 +08:00
|
|
|
bool qdev_machine_modified(void)
|
|
|
|
{
|
|
|
|
return qdev_hot_added || qdev_hot_removed;
|
|
|
|
}
|
|
|
|
|
2009-05-23 07:05:19 +08:00
|
|
|
BusState *qdev_get_parent_bus(DeviceState *dev)
|
2009-05-15 05:35:06 +08:00
|
|
|
{
|
2009-05-23 07:05:19 +08:00
|
|
|
return dev->parent_bus;
|
2009-05-15 05:35:06 +08:00
|
|
|
}
|
|
|
|
|
2014-05-20 14:30:58 +08:00
|
|
|
static NamedGPIOList *qdev_get_named_gpio_list(DeviceState *dev,
|
|
|
|
const char *name)
|
|
|
|
{
|
|
|
|
NamedGPIOList *ngl;
|
|
|
|
|
|
|
|
QLIST_FOREACH(ngl, &dev->gpios, node) {
|
|
|
|
/* NULL is a valid and matchable name, otherwise do a normal
|
|
|
|
* strcmp match.
|
|
|
|
*/
|
|
|
|
if ((!ngl->name && !name) ||
|
|
|
|
(name && ngl->name && strcmp(name, ngl->name) == 0)) {
|
|
|
|
return ngl;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
ngl = g_malloc0(sizeof(*ngl));
|
|
|
|
ngl->name = g_strdup(name);
|
|
|
|
QLIST_INSERT_HEAD(&dev->gpios, ngl, node);
|
|
|
|
return ngl;
|
|
|
|
}
|
|
|
|
|
|
|
|
void qdev_init_gpio_in_named(DeviceState *dev, qemu_irq_handler handler,
|
|
|
|
const char *name, int n)
|
|
|
|
{
|
2014-09-26 13:17:41 +08:00
|
|
|
int i;
|
2014-05-20 14:30:58 +08:00
|
|
|
NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name);
|
2014-09-26 13:17:41 +08:00
|
|
|
char *propname = g_strdup_printf("%s[*]", name ? name : "unnamed-gpio-in");
|
2014-05-20 14:30:58 +08:00
|
|
|
|
2014-09-26 13:17:08 +08:00
|
|
|
assert(gpio_list->num_out == 0 || !name);
|
2014-05-20 14:30:58 +08:00
|
|
|
gpio_list->in = qemu_extend_irqs(gpio_list->in, gpio_list->num_in, handler,
|
|
|
|
dev, n);
|
2014-09-26 13:17:41 +08:00
|
|
|
|
|
|
|
for (i = gpio_list->num_in; i < gpio_list->num_in + n; i++) {
|
|
|
|
object_property_add_child(OBJECT(dev), propname,
|
|
|
|
OBJECT(gpio_list->in[i]), &error_abort);
|
|
|
|
}
|
|
|
|
g_free(propname);
|
|
|
|
|
2014-05-20 14:30:58 +08:00
|
|
|
gpio_list->num_in += n;
|
|
|
|
}
|
|
|
|
|
2009-05-15 05:35:06 +08:00
|
|
|
void qdev_init_gpio_in(DeviceState *dev, qemu_irq_handler handler, int n)
|
|
|
|
{
|
2014-05-20 14:30:58 +08:00
|
|
|
qdev_init_gpio_in_named(dev, handler, NULL, n);
|
|
|
|
}
|
|
|
|
|
|
|
|
void qdev_init_gpio_out_named(DeviceState *dev, qemu_irq *pins,
|
|
|
|
const char *name, int n)
|
|
|
|
{
|
2014-09-26 13:18:14 +08:00
|
|
|
int i;
|
2014-05-20 14:30:58 +08:00
|
|
|
NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name);
|
2014-09-26 13:18:14 +08:00
|
|
|
char *propname = g_strdup_printf("%s[*]", name ? name : "unnamed-gpio-out");
|
2014-05-20 14:30:58 +08:00
|
|
|
|
2014-09-26 13:17:08 +08:00
|
|
|
assert(gpio_list->num_in == 0 || !name);
|
2014-05-20 14:30:58 +08:00
|
|
|
assert(gpio_list->num_out == 0);
|
|
|
|
gpio_list->num_out = n;
|
|
|
|
gpio_list->out = pins;
|
2014-09-26 13:18:14 +08:00
|
|
|
|
|
|
|
for (i = 0; i < n; ++i) {
|
|
|
|
memset(&pins[i], 0, sizeof(*pins));
|
|
|
|
object_property_add_link(OBJECT(dev), propname, TYPE_IRQ,
|
|
|
|
(Object **)&pins[i],
|
|
|
|
object_property_allow_set_link,
|
|
|
|
OBJ_PROP_LINK_UNREF_ON_RELEASE,
|
|
|
|
&error_abort);
|
|
|
|
}
|
|
|
|
g_free(propname);
|
2009-05-15 05:35:06 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
void qdev_init_gpio_out(DeviceState *dev, qemu_irq *pins, int n)
|
|
|
|
{
|
2014-05-20 14:30:58 +08:00
|
|
|
qdev_init_gpio_out_named(dev, pins, NULL, n);
|
|
|
|
}
|
|
|
|
|
|
|
|
qemu_irq qdev_get_gpio_in_named(DeviceState *dev, const char *name, int n)
|
|
|
|
{
|
|
|
|
NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name);
|
|
|
|
|
|
|
|
assert(n >= 0 && n < gpio_list->num_in);
|
|
|
|
return gpio_list->in[n];
|
2009-05-15 05:35:06 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
qemu_irq qdev_get_gpio_in(DeviceState *dev, int n)
|
|
|
|
{
|
2014-05-20 14:30:58 +08:00
|
|
|
return qdev_get_gpio_in_named(dev, NULL, n);
|
|
|
|
}
|
|
|
|
|
|
|
|
void qdev_connect_gpio_out_named(DeviceState *dev, const char *name, int n,
|
|
|
|
qemu_irq pin)
|
|
|
|
{
|
|
|
|
NamedGPIOList *gpio_list = qdev_get_named_gpio_list(dev, name);
|
|
|
|
|
|
|
|
assert(n >= 0 && n < gpio_list->num_out);
|
|
|
|
gpio_list->out[n] = pin;
|
2009-05-15 05:35:06 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
void qdev_connect_gpio_out(DeviceState * dev, int n, qemu_irq pin)
|
|
|
|
{
|
2014-05-20 14:30:58 +08:00
|
|
|
qdev_connect_gpio_out_named(dev, NULL, n, pin);
|
2009-05-15 05:35:06 +08:00
|
|
|
}
|
|
|
|
|
2009-05-23 07:05:19 +08:00
|
|
|
BusState *qdev_get_child_bus(DeviceState *dev, const char *name)
|
2009-05-15 05:35:06 +08:00
|
|
|
{
|
2009-05-23 07:05:19 +08:00
|
|
|
BusState *bus;
|
2009-05-15 05:35:06 +08:00
|
|
|
|
2009-09-12 15:36:22 +08:00
|
|
|
QLIST_FOREACH(bus, &dev->child_bus, sibling) {
|
2009-05-15 05:35:06 +08:00
|
|
|
if (strcmp(name, bus->name) == 0) {
|
2009-05-23 07:05:19 +08:00
|
|
|
return bus;
|
2009-05-15 05:35:06 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2013-12-07 00:54:26 +08:00
|
|
|
int qbus_walk_children(BusState *bus,
|
|
|
|
qdev_walkerfn *pre_devfn, qbus_walkerfn *pre_busfn,
|
|
|
|
qdev_walkerfn *post_devfn, qbus_walkerfn *post_busfn,
|
|
|
|
void *opaque)
|
2010-11-19 17:55:58 +08:00
|
|
|
{
|
2011-12-24 05:34:39 +08:00
|
|
|
BusChild *kid;
|
2010-11-19 17:55:58 +08:00
|
|
|
int err;
|
|
|
|
|
2013-12-07 00:54:26 +08:00
|
|
|
if (pre_busfn) {
|
|
|
|
err = pre_busfn(bus, opaque);
|
2010-11-19 17:55:58 +08:00
|
|
|
if (err) {
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-12-24 05:34:39 +08:00
|
|
|
QTAILQ_FOREACH(kid, &bus->children, sibling) {
|
2013-12-07 00:54:26 +08:00
|
|
|
err = qdev_walk_children(kid->child,
|
|
|
|
pre_devfn, pre_busfn,
|
|
|
|
post_devfn, post_busfn, opaque);
|
2010-11-19 17:55:58 +08:00
|
|
|
if (err < 0) {
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2013-12-07 00:54:26 +08:00
|
|
|
if (post_busfn) {
|
|
|
|
err = post_busfn(bus, opaque);
|
|
|
|
if (err) {
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2010-11-19 17:55:58 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2013-12-07 00:54:26 +08:00
|
|
|
int qdev_walk_children(DeviceState *dev,
|
|
|
|
qdev_walkerfn *pre_devfn, qbus_walkerfn *pre_busfn,
|
|
|
|
qdev_walkerfn *post_devfn, qbus_walkerfn *post_busfn,
|
|
|
|
void *opaque)
|
2010-11-19 17:55:58 +08:00
|
|
|
{
|
|
|
|
BusState *bus;
|
|
|
|
int err;
|
|
|
|
|
2013-12-07 00:54:26 +08:00
|
|
|
if (pre_devfn) {
|
|
|
|
err = pre_devfn(dev, opaque);
|
2010-11-19 17:55:58 +08:00
|
|
|
if (err) {
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
QLIST_FOREACH(bus, &dev->child_bus, sibling) {
|
2013-12-07 00:54:26 +08:00
|
|
|
err = qbus_walk_children(bus, pre_devfn, pre_busfn,
|
|
|
|
post_devfn, post_busfn, opaque);
|
2010-11-19 17:55:58 +08:00
|
|
|
if (err < 0) {
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2013-12-07 00:54:26 +08:00
|
|
|
if (post_devfn) {
|
|
|
|
err = post_devfn(dev, opaque);
|
|
|
|
if (err) {
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2010-11-19 17:55:58 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2010-12-24 11:14:12 +08:00
|
|
|
DeviceState *qdev_find_recursive(BusState *bus, const char *id)
|
2009-09-26 03:42:41 +08:00
|
|
|
{
|
2011-12-24 05:34:39 +08:00
|
|
|
BusChild *kid;
|
|
|
|
DeviceState *ret;
|
2009-09-26 03:42:41 +08:00
|
|
|
BusState *child;
|
|
|
|
|
2011-12-24 05:34:39 +08:00
|
|
|
QTAILQ_FOREACH(kid, &bus->children, sibling) {
|
|
|
|
DeviceState *dev = kid->child;
|
|
|
|
|
|
|
|
if (dev->id && strcmp(dev->id, id) == 0) {
|
2009-09-26 03:42:41 +08:00
|
|
|
return dev;
|
2011-12-24 05:34:39 +08:00
|
|
|
}
|
|
|
|
|
2009-09-26 03:42:41 +08:00
|
|
|
QLIST_FOREACH(child, &dev->child_bus, sibling) {
|
|
|
|
ret = qdev_find_recursive(child, id);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2013-01-25 21:12:27 +08:00
|
|
|
static void qbus_realize(BusState *bus, DeviceState *parent, const char *name)
|
2009-05-23 07:05:19 +08:00
|
|
|
{
|
2012-02-04 03:32:19 +08:00
|
|
|
const char *typename = object_get_typename(OBJECT(bus));
|
qdev: Keep global allocation counter per bus
When we have 2 separate qdev devices that both create a qbus of the
same type without specifying a bus name or device name, we end up
with two buses of the same name, such as ide.0 on the Mac machines:
dev: macio-ide, id ""
bus: ide.0
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
If we now spawn a device that connects to a ide.0 the last created
bus gets the device, with the first created bus inaccessible to the
command line.
After some discussion on IRC we concluded that the best quick fix way
forward for this is to make automated bus-class type based allocation
count a global counter. That's what this patch implements. With this
we instead get
dev: macio-ide, id ""
bus: ide.1
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
on the example mentioned above.
This also means that if you did -device ...,bus=ide.0 you got a device
on the first bus (the last created one) before this patch and get that
device on the second one (the first created one) now. Breaks
migration unless you change bus=ide.0 to bus=ide.1 on the destination.
This is intended and makes the bus enumeration work as expected.
As per review request follows a list of otherwise affected boards and
the reasoning for the conclusion that they are ok:
target machine bus id times
------ ------- ------ -----
aarch64 n800 i2c-bus.0 2
aarch64 n810 i2c-bus.0 2
arm n800 i2c-bus.0 2
arm n810 i2c-bus.0 2
-> Devices are only created explicitly on one of the two buses, using
s->mpu->i2c[0], so no change to the guest.
aarch64 vexpress-a15 virtio-mmio-bus.0 4
aarch64 vexpress-a9 virtio-mmio-bus.0 4
aarch64 virt virtio-mmio-bus.0 32
arm vexpress-a15 virtio-mmio-bus.0 4
arm vexpress-a9 virtio-mmio-bus.0 4
arm virt virtio-mmio-bus.0 32
-> Makes -device bus= work for all virtio-mmio buses. Breaks
migration. Workaround for migration from old to new: specify
virtio-mmio-bus.4 or .32 respectively rather than .0 on the
destination.
aarch64 xilinx-zynq-a9 usb-bus.0 2
arm xilinx-zynq-a9 usb-bus.0 2
mips64el fulong2e usb-bus.0 2
-> Normal USB operation not affected. Migration driver needs command
line to use the other bus.
i386 isapc ide.0 2
x86_64 isapc ide.0 2
mips mips ide.0 2
mips64 mips ide.0 2
mips64el mips ide.0 2
mipsel mips ide.0 2
ppc g3beige ide.0 2
ppc mac99 ide.0 2
ppc prep ide.0 2
ppc64 g3beige ide.0 2
ppc64 mac99 ide.0 2
ppc64 prep ide.0 2
-> Makes -device bus= work for all IDE buses. Breaks migration.
Workaround for migration from old to new: specify ide.1 rather than
ide.0 on the destination.
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Andreas Faerber <afaerber@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
2014-02-06 23:08:15 +08:00
|
|
|
BusClass *bc;
|
2009-07-15 19:59:24 +08:00
|
|
|
char *buf;
|
qdev: Keep global allocation counter per bus
When we have 2 separate qdev devices that both create a qbus of the
same type without specifying a bus name or device name, we end up
with two buses of the same name, such as ide.0 on the Mac machines:
dev: macio-ide, id ""
bus: ide.0
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
If we now spawn a device that connects to a ide.0 the last created
bus gets the device, with the first created bus inaccessible to the
command line.
After some discussion on IRC we concluded that the best quick fix way
forward for this is to make automated bus-class type based allocation
count a global counter. That's what this patch implements. With this
we instead get
dev: macio-ide, id ""
bus: ide.1
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
on the example mentioned above.
This also means that if you did -device ...,bus=ide.0 you got a device
on the first bus (the last created one) before this patch and get that
device on the second one (the first created one) now. Breaks
migration unless you change bus=ide.0 to bus=ide.1 on the destination.
This is intended and makes the bus enumeration work as expected.
As per review request follows a list of otherwise affected boards and
the reasoning for the conclusion that they are ok:
target machine bus id times
------ ------- ------ -----
aarch64 n800 i2c-bus.0 2
aarch64 n810 i2c-bus.0 2
arm n800 i2c-bus.0 2
arm n810 i2c-bus.0 2
-> Devices are only created explicitly on one of the two buses, using
s->mpu->i2c[0], so no change to the guest.
aarch64 vexpress-a15 virtio-mmio-bus.0 4
aarch64 vexpress-a9 virtio-mmio-bus.0 4
aarch64 virt virtio-mmio-bus.0 32
arm vexpress-a15 virtio-mmio-bus.0 4
arm vexpress-a9 virtio-mmio-bus.0 4
arm virt virtio-mmio-bus.0 32
-> Makes -device bus= work for all virtio-mmio buses. Breaks
migration. Workaround for migration from old to new: specify
virtio-mmio-bus.4 or .32 respectively rather than .0 on the
destination.
aarch64 xilinx-zynq-a9 usb-bus.0 2
arm xilinx-zynq-a9 usb-bus.0 2
mips64el fulong2e usb-bus.0 2
-> Normal USB operation not affected. Migration driver needs command
line to use the other bus.
i386 isapc ide.0 2
x86_64 isapc ide.0 2
mips mips ide.0 2
mips64 mips ide.0 2
mips64el mips ide.0 2
mipsel mips ide.0 2
ppc g3beige ide.0 2
ppc mac99 ide.0 2
ppc prep ide.0 2
ppc64 g3beige ide.0 2
ppc64 mac99 ide.0 2
ppc64 prep ide.0 2
-> Makes -device bus= work for all IDE buses. Breaks migration.
Workaround for migration from old to new: specify ide.1 rather than
ide.0 on the destination.
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Andreas Faerber <afaerber@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
2014-02-06 23:08:15 +08:00
|
|
|
int i, len, bus_id;
|
2009-05-23 07:05:19 +08:00
|
|
|
|
2013-01-25 21:12:27 +08:00
|
|
|
bus->parent = parent;
|
|
|
|
|
|
|
|
if (name) {
|
|
|
|
bus->name = g_strdup(name);
|
2012-02-04 03:32:19 +08:00
|
|
|
} else if (bus->parent && bus->parent->id) {
|
qdev: Keep global allocation counter per bus
When we have 2 separate qdev devices that both create a qbus of the
same type without specifying a bus name or device name, we end up
with two buses of the same name, such as ide.0 on the Mac machines:
dev: macio-ide, id ""
bus: ide.0
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
If we now spawn a device that connects to a ide.0 the last created
bus gets the device, with the first created bus inaccessible to the
command line.
After some discussion on IRC we concluded that the best quick fix way
forward for this is to make automated bus-class type based allocation
count a global counter. That's what this patch implements. With this
we instead get
dev: macio-ide, id ""
bus: ide.1
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
on the example mentioned above.
This also means that if you did -device ...,bus=ide.0 you got a device
on the first bus (the last created one) before this patch and get that
device on the second one (the first created one) now. Breaks
migration unless you change bus=ide.0 to bus=ide.1 on the destination.
This is intended and makes the bus enumeration work as expected.
As per review request follows a list of otherwise affected boards and
the reasoning for the conclusion that they are ok:
target machine bus id times
------ ------- ------ -----
aarch64 n800 i2c-bus.0 2
aarch64 n810 i2c-bus.0 2
arm n800 i2c-bus.0 2
arm n810 i2c-bus.0 2
-> Devices are only created explicitly on one of the two buses, using
s->mpu->i2c[0], so no change to the guest.
aarch64 vexpress-a15 virtio-mmio-bus.0 4
aarch64 vexpress-a9 virtio-mmio-bus.0 4
aarch64 virt virtio-mmio-bus.0 32
arm vexpress-a15 virtio-mmio-bus.0 4
arm vexpress-a9 virtio-mmio-bus.0 4
arm virt virtio-mmio-bus.0 32
-> Makes -device bus= work for all virtio-mmio buses. Breaks
migration. Workaround for migration from old to new: specify
virtio-mmio-bus.4 or .32 respectively rather than .0 on the
destination.
aarch64 xilinx-zynq-a9 usb-bus.0 2
arm xilinx-zynq-a9 usb-bus.0 2
mips64el fulong2e usb-bus.0 2
-> Normal USB operation not affected. Migration driver needs command
line to use the other bus.
i386 isapc ide.0 2
x86_64 isapc ide.0 2
mips mips ide.0 2
mips64 mips ide.0 2
mips64el mips ide.0 2
mipsel mips ide.0 2
ppc g3beige ide.0 2
ppc mac99 ide.0 2
ppc prep ide.0 2
ppc64 g3beige ide.0 2
ppc64 mac99 ide.0 2
ppc64 prep ide.0 2
-> Makes -device bus= work for all IDE buses. Breaks migration.
Workaround for migration from old to new: specify ide.1 rather than
ide.0 on the destination.
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Andreas Faerber <afaerber@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
2014-02-06 23:08:15 +08:00
|
|
|
/* parent device has id -> use it plus parent-bus-id for bus name */
|
|
|
|
bus_id = bus->parent->num_child_bus;
|
|
|
|
|
2012-02-04 03:32:19 +08:00
|
|
|
len = strlen(bus->parent->id) + 16;
|
2011-08-21 11:09:37 +08:00
|
|
|
buf = g_malloc(len);
|
qdev: Keep global allocation counter per bus
When we have 2 separate qdev devices that both create a qbus of the
same type without specifying a bus name or device name, we end up
with two buses of the same name, such as ide.0 on the Mac machines:
dev: macio-ide, id ""
bus: ide.0
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
If we now spawn a device that connects to a ide.0 the last created
bus gets the device, with the first created bus inaccessible to the
command line.
After some discussion on IRC we concluded that the best quick fix way
forward for this is to make automated bus-class type based allocation
count a global counter. That's what this patch implements. With this
we instead get
dev: macio-ide, id ""
bus: ide.1
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
on the example mentioned above.
This also means that if you did -device ...,bus=ide.0 you got a device
on the first bus (the last created one) before this patch and get that
device on the second one (the first created one) now. Breaks
migration unless you change bus=ide.0 to bus=ide.1 on the destination.
This is intended and makes the bus enumeration work as expected.
As per review request follows a list of otherwise affected boards and
the reasoning for the conclusion that they are ok:
target machine bus id times
------ ------- ------ -----
aarch64 n800 i2c-bus.0 2
aarch64 n810 i2c-bus.0 2
arm n800 i2c-bus.0 2
arm n810 i2c-bus.0 2
-> Devices are only created explicitly on one of the two buses, using
s->mpu->i2c[0], so no change to the guest.
aarch64 vexpress-a15 virtio-mmio-bus.0 4
aarch64 vexpress-a9 virtio-mmio-bus.0 4
aarch64 virt virtio-mmio-bus.0 32
arm vexpress-a15 virtio-mmio-bus.0 4
arm vexpress-a9 virtio-mmio-bus.0 4
arm virt virtio-mmio-bus.0 32
-> Makes -device bus= work for all virtio-mmio buses. Breaks
migration. Workaround for migration from old to new: specify
virtio-mmio-bus.4 or .32 respectively rather than .0 on the
destination.
aarch64 xilinx-zynq-a9 usb-bus.0 2
arm xilinx-zynq-a9 usb-bus.0 2
mips64el fulong2e usb-bus.0 2
-> Normal USB operation not affected. Migration driver needs command
line to use the other bus.
i386 isapc ide.0 2
x86_64 isapc ide.0 2
mips mips ide.0 2
mips64 mips ide.0 2
mips64el mips ide.0 2
mipsel mips ide.0 2
ppc g3beige ide.0 2
ppc mac99 ide.0 2
ppc prep ide.0 2
ppc64 g3beige ide.0 2
ppc64 mac99 ide.0 2
ppc64 prep ide.0 2
-> Makes -device bus= work for all IDE buses. Breaks migration.
Workaround for migration from old to new: specify ide.1 rather than
ide.0 on the destination.
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Andreas Faerber <afaerber@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
2014-02-06 23:08:15 +08:00
|
|
|
snprintf(buf, len, "%s.%d", bus->parent->id, bus_id);
|
2009-07-15 19:59:24 +08:00
|
|
|
bus->name = buf;
|
|
|
|
} else {
|
qdev: Keep global allocation counter per bus
When we have 2 separate qdev devices that both create a qbus of the
same type without specifying a bus name or device name, we end up
with two buses of the same name, such as ide.0 on the Mac machines:
dev: macio-ide, id ""
bus: ide.0
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
If we now spawn a device that connects to a ide.0 the last created
bus gets the device, with the first created bus inaccessible to the
command line.
After some discussion on IRC we concluded that the best quick fix way
forward for this is to make automated bus-class type based allocation
count a global counter. That's what this patch implements. With this
we instead get
dev: macio-ide, id ""
bus: ide.1
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
on the example mentioned above.
This also means that if you did -device ...,bus=ide.0 you got a device
on the first bus (the last created one) before this patch and get that
device on the second one (the first created one) now. Breaks
migration unless you change bus=ide.0 to bus=ide.1 on the destination.
This is intended and makes the bus enumeration work as expected.
As per review request follows a list of otherwise affected boards and
the reasoning for the conclusion that they are ok:
target machine bus id times
------ ------- ------ -----
aarch64 n800 i2c-bus.0 2
aarch64 n810 i2c-bus.0 2
arm n800 i2c-bus.0 2
arm n810 i2c-bus.0 2
-> Devices are only created explicitly on one of the two buses, using
s->mpu->i2c[0], so no change to the guest.
aarch64 vexpress-a15 virtio-mmio-bus.0 4
aarch64 vexpress-a9 virtio-mmio-bus.0 4
aarch64 virt virtio-mmio-bus.0 32
arm vexpress-a15 virtio-mmio-bus.0 4
arm vexpress-a9 virtio-mmio-bus.0 4
arm virt virtio-mmio-bus.0 32
-> Makes -device bus= work for all virtio-mmio buses. Breaks
migration. Workaround for migration from old to new: specify
virtio-mmio-bus.4 or .32 respectively rather than .0 on the
destination.
aarch64 xilinx-zynq-a9 usb-bus.0 2
arm xilinx-zynq-a9 usb-bus.0 2
mips64el fulong2e usb-bus.0 2
-> Normal USB operation not affected. Migration driver needs command
line to use the other bus.
i386 isapc ide.0 2
x86_64 isapc ide.0 2
mips mips ide.0 2
mips64 mips ide.0 2
mips64el mips ide.0 2
mipsel mips ide.0 2
ppc g3beige ide.0 2
ppc mac99 ide.0 2
ppc prep ide.0 2
ppc64 g3beige ide.0 2
ppc64 mac99 ide.0 2
ppc64 prep ide.0 2
-> Makes -device bus= work for all IDE buses. Breaks migration.
Workaround for migration from old to new: specify ide.1 rather than
ide.0 on the destination.
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Andreas Faerber <afaerber@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
2014-02-06 23:08:15 +08:00
|
|
|
/* no id -> use lowercase bus type plus global bus-id for bus name */
|
|
|
|
bc = BUS_GET_CLASS(bus);
|
|
|
|
bus_id = bc->automatic_ids++;
|
|
|
|
|
2012-05-02 15:00:20 +08:00
|
|
|
len = strlen(typename) + 16;
|
2011-08-21 11:09:37 +08:00
|
|
|
buf = g_malloc(len);
|
qdev: Keep global allocation counter per bus
When we have 2 separate qdev devices that both create a qbus of the
same type without specifying a bus name or device name, we end up
with two buses of the same name, such as ide.0 on the Mac machines:
dev: macio-ide, id ""
bus: ide.0
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
If we now spawn a device that connects to a ide.0 the last created
bus gets the device, with the first created bus inaccessible to the
command line.
After some discussion on IRC we concluded that the best quick fix way
forward for this is to make automated bus-class type based allocation
count a global counter. That's what this patch implements. With this
we instead get
dev: macio-ide, id ""
bus: ide.1
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
on the example mentioned above.
This also means that if you did -device ...,bus=ide.0 you got a device
on the first bus (the last created one) before this patch and get that
device on the second one (the first created one) now. Breaks
migration unless you change bus=ide.0 to bus=ide.1 on the destination.
This is intended and makes the bus enumeration work as expected.
As per review request follows a list of otherwise affected boards and
the reasoning for the conclusion that they are ok:
target machine bus id times
------ ------- ------ -----
aarch64 n800 i2c-bus.0 2
aarch64 n810 i2c-bus.0 2
arm n800 i2c-bus.0 2
arm n810 i2c-bus.0 2
-> Devices are only created explicitly on one of the two buses, using
s->mpu->i2c[0], so no change to the guest.
aarch64 vexpress-a15 virtio-mmio-bus.0 4
aarch64 vexpress-a9 virtio-mmio-bus.0 4
aarch64 virt virtio-mmio-bus.0 32
arm vexpress-a15 virtio-mmio-bus.0 4
arm vexpress-a9 virtio-mmio-bus.0 4
arm virt virtio-mmio-bus.0 32
-> Makes -device bus= work for all virtio-mmio buses. Breaks
migration. Workaround for migration from old to new: specify
virtio-mmio-bus.4 or .32 respectively rather than .0 on the
destination.
aarch64 xilinx-zynq-a9 usb-bus.0 2
arm xilinx-zynq-a9 usb-bus.0 2
mips64el fulong2e usb-bus.0 2
-> Normal USB operation not affected. Migration driver needs command
line to use the other bus.
i386 isapc ide.0 2
x86_64 isapc ide.0 2
mips mips ide.0 2
mips64 mips ide.0 2
mips64el mips ide.0 2
mipsel mips ide.0 2
ppc g3beige ide.0 2
ppc mac99 ide.0 2
ppc prep ide.0 2
ppc64 g3beige ide.0 2
ppc64 mac99 ide.0 2
ppc64 prep ide.0 2
-> Makes -device bus= work for all IDE buses. Breaks migration.
Workaround for migration from old to new: specify ide.1 rather than
ide.0 on the destination.
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Andreas Faerber <afaerber@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
2014-02-06 23:08:15 +08:00
|
|
|
len = snprintf(buf, len, "%s.%d", typename, bus_id);
|
|
|
|
for (i = 0; i < len; i++) {
|
2009-07-30 21:28:45 +08:00
|
|
|
buf[i] = qemu_tolower(buf[i]);
|
qdev: Keep global allocation counter per bus
When we have 2 separate qdev devices that both create a qbus of the
same type without specifying a bus name or device name, we end up
with two buses of the same name, such as ide.0 on the Mac machines:
dev: macio-ide, id ""
bus: ide.0
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
If we now spawn a device that connects to a ide.0 the last created
bus gets the device, with the first created bus inaccessible to the
command line.
After some discussion on IRC we concluded that the best quick fix way
forward for this is to make automated bus-class type based allocation
count a global counter. That's what this patch implements. With this
we instead get
dev: macio-ide, id ""
bus: ide.1
type IDE
dev: macio-ide, id ""
bus: ide.0
type IDE
on the example mentioned above.
This also means that if you did -device ...,bus=ide.0 you got a device
on the first bus (the last created one) before this patch and get that
device on the second one (the first created one) now. Breaks
migration unless you change bus=ide.0 to bus=ide.1 on the destination.
This is intended and makes the bus enumeration work as expected.
As per review request follows a list of otherwise affected boards and
the reasoning for the conclusion that they are ok:
target machine bus id times
------ ------- ------ -----
aarch64 n800 i2c-bus.0 2
aarch64 n810 i2c-bus.0 2
arm n800 i2c-bus.0 2
arm n810 i2c-bus.0 2
-> Devices are only created explicitly on one of the two buses, using
s->mpu->i2c[0], so no change to the guest.
aarch64 vexpress-a15 virtio-mmio-bus.0 4
aarch64 vexpress-a9 virtio-mmio-bus.0 4
aarch64 virt virtio-mmio-bus.0 32
arm vexpress-a15 virtio-mmio-bus.0 4
arm vexpress-a9 virtio-mmio-bus.0 4
arm virt virtio-mmio-bus.0 32
-> Makes -device bus= work for all virtio-mmio buses. Breaks
migration. Workaround for migration from old to new: specify
virtio-mmio-bus.4 or .32 respectively rather than .0 on the
destination.
aarch64 xilinx-zynq-a9 usb-bus.0 2
arm xilinx-zynq-a9 usb-bus.0 2
mips64el fulong2e usb-bus.0 2
-> Normal USB operation not affected. Migration driver needs command
line to use the other bus.
i386 isapc ide.0 2
x86_64 isapc ide.0 2
mips mips ide.0 2
mips64 mips ide.0 2
mips64el mips ide.0 2
mipsel mips ide.0 2
ppc g3beige ide.0 2
ppc mac99 ide.0 2
ppc prep ide.0 2
ppc64 g3beige ide.0 2
ppc64 mac99 ide.0 2
ppc64 prep ide.0 2
-> Makes -device bus= work for all IDE buses. Breaks migration.
Workaround for migration from old to new: specify ide.1 rather than
ide.0 on the destination.
Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Andreas Faerber <afaerber@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
2014-02-06 23:08:15 +08:00
|
|
|
}
|
2009-07-15 19:59:24 +08:00
|
|
|
bus->name = buf;
|
|
|
|
}
|
|
|
|
|
2012-02-04 03:32:19 +08:00
|
|
|
if (bus->parent) {
|
|
|
|
QLIST_INSERT_HEAD(&bus->parent->child_bus, bus, sibling);
|
|
|
|
bus->parent->num_child_bus++;
|
|
|
|
object_property_add_child(OBJECT(bus->parent), bus->name, OBJECT(bus), NULL);
|
2013-01-25 21:12:37 +08:00
|
|
|
object_unref(OBJECT(bus));
|
2012-05-02 18:06:55 +08:00
|
|
|
} else if (bus != sysbus_get_default()) {
|
2010-12-20 13:33:35 +08:00
|
|
|
/* TODO: once all bus devices are qdevified,
|
|
|
|
only reset handler for main_system_bus should be registered here. */
|
|
|
|
qemu_register_reset(qbus_reset_all_fn, bus);
|
2009-05-23 07:05:19 +08:00
|
|
|
}
|
2009-09-17 04:25:27 +08:00
|
|
|
}
|
|
|
|
|
2013-01-25 21:12:33 +08:00
|
|
|
static void bus_unparent(Object *obj)
|
|
|
|
{
|
|
|
|
BusState *bus = BUS(obj);
|
|
|
|
BusChild *kid;
|
|
|
|
|
|
|
|
while ((kid = QTAILQ_FIRST(&bus->children)) != NULL) {
|
|
|
|
DeviceState *dev = kid->child;
|
2013-09-11 20:54:09 +08:00
|
|
|
object_unparent(OBJECT(dev));
|
2013-01-25 21:12:33 +08:00
|
|
|
}
|
|
|
|
if (bus->parent) {
|
|
|
|
QLIST_REMOVE(bus, sibling);
|
|
|
|
bus->parent->num_child_bus--;
|
|
|
|
bus->parent = NULL;
|
|
|
|
} else {
|
|
|
|
assert(bus != sysbus_get_default()); /* main_system_bus is never freed */
|
|
|
|
qemu_unregister_reset(qbus_reset_all_fn, bus);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2014-04-25 18:44:21 +08:00
|
|
|
static bool bus_get_realized(Object *obj, Error **errp)
|
2013-11-26 06:48:40 +08:00
|
|
|
{
|
|
|
|
BusState *bus = BUS(obj);
|
|
|
|
|
|
|
|
return bus->realized;
|
|
|
|
}
|
|
|
|
|
2014-04-25 18:44:21 +08:00
|
|
|
static void bus_set_realized(Object *obj, bool value, Error **errp)
|
2013-11-26 06:48:40 +08:00
|
|
|
{
|
|
|
|
BusState *bus = BUS(obj);
|
|
|
|
BusClass *bc = BUS_GET_CLASS(bus);
|
2014-06-11 20:52:09 +08:00
|
|
|
BusChild *kid;
|
2013-11-26 06:48:40 +08:00
|
|
|
Error *local_err = NULL;
|
|
|
|
|
|
|
|
if (value && !bus->realized) {
|
|
|
|
if (bc->realize) {
|
|
|
|
bc->realize(bus, &local_err);
|
|
|
|
}
|
2014-06-11 20:52:09 +08:00
|
|
|
|
|
|
|
/* TODO: recursive realization */
|
2013-11-26 06:48:40 +08:00
|
|
|
} else if (!value && bus->realized) {
|
2014-06-11 20:52:09 +08:00
|
|
|
QTAILQ_FOREACH(kid, &bus->children, sibling) {
|
|
|
|
DeviceState *dev = kid->child;
|
|
|
|
object_property_set_bool(OBJECT(dev), false, "realized",
|
|
|
|
&local_err);
|
|
|
|
if (local_err != NULL) {
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (bc->unrealize && local_err == NULL) {
|
2013-11-26 06:48:40 +08:00
|
|
|
bc->unrealize(bus, &local_err);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2014-06-11 20:52:08 +08:00
|
|
|
if (local_err != NULL) {
|
|
|
|
error_propagate(errp, local_err);
|
|
|
|
return;
|
|
|
|
}
|
2013-11-26 06:48:40 +08:00
|
|
|
|
2014-06-11 20:52:08 +08:00
|
|
|
bus->realized = value;
|
2013-11-26 06:48:40 +08:00
|
|
|
}
|
|
|
|
|
2013-08-24 06:02:27 +08:00
|
|
|
void qbus_create_inplace(void *bus, size_t size, const char *typename,
|
2012-05-02 15:00:20 +08:00
|
|
|
DeviceState *parent, const char *name)
|
2009-09-17 04:25:27 +08:00
|
|
|
{
|
2013-08-24 01:37:12 +08:00
|
|
|
object_initialize(bus, size, typename);
|
2013-01-25 21:12:27 +08:00
|
|
|
qbus_realize(bus, parent, name);
|
2009-05-23 07:05:19 +08:00
|
|
|
}
|
2009-06-05 22:53:17 +08:00
|
|
|
|
2012-05-02 15:00:20 +08:00
|
|
|
BusState *qbus_create(const char *typename, DeviceState *parent, const char *name)
|
2011-08-02 09:59:13 +08:00
|
|
|
{
|
2009-09-17 04:25:27 +08:00
|
|
|
BusState *bus;
|
|
|
|
|
2012-05-02 15:00:20 +08:00
|
|
|
bus = BUS(object_new(typename));
|
2013-01-25 21:12:27 +08:00
|
|
|
qbus_realize(bus, parent, name);
|
2012-02-04 03:32:19 +08:00
|
|
|
|
2009-05-23 07:05:19 +08:00
|
|
|
return bus;
|
2011-08-02 09:59:13 +08:00
|
|
|
}
|
|
|
|
|
2012-05-02 15:00:20 +08:00
|
|
|
static char *bus_get_fw_dev_path(BusState *bus, DeviceState *dev)
|
|
|
|
{
|
|
|
|
BusClass *bc = BUS_GET_CLASS(bus);
|
|
|
|
|
|
|
|
if (bc->get_fw_dev_path) {
|
|
|
|
return bc->get_fw_dev_path(dev);
|
2009-09-26 03:42:34 +08:00
|
|
|
}
|
2012-05-02 15:00:20 +08:00
|
|
|
|
|
|
|
return NULL;
|
2009-09-26 03:42:34 +08:00
|
|
|
}
|
|
|
|
|
2014-03-17 10:40:23 +08:00
|
|
|
static char *qdev_get_fw_dev_path_from_handler(BusState *bus, DeviceState *dev)
|
|
|
|
{
|
|
|
|
Object *obj = OBJECT(dev);
|
|
|
|
char *d = NULL;
|
|
|
|
|
|
|
|
while (!d && obj->parent) {
|
|
|
|
obj = obj->parent;
|
|
|
|
d = fw_path_provider_try_get_dev_path(obj, bus, dev);
|
|
|
|
}
|
|
|
|
return d;
|
|
|
|
}
|
|
|
|
|
2010-12-08 19:35:05 +08:00
|
|
|
static int qdev_get_fw_dev_path_helper(DeviceState *dev, char *p, int size)
|
|
|
|
{
|
|
|
|
int l = 0;
|
|
|
|
|
|
|
|
if (dev && dev->parent_bus) {
|
|
|
|
char *d;
|
|
|
|
l = qdev_get_fw_dev_path_helper(dev->parent_bus->parent, p, size);
|
2014-03-17 10:40:23 +08:00
|
|
|
d = qdev_get_fw_dev_path_from_handler(dev->parent_bus, dev);
|
|
|
|
if (!d) {
|
|
|
|
d = bus_get_fw_dev_path(dev->parent_bus, dev);
|
|
|
|
}
|
2012-05-02 15:00:20 +08:00
|
|
|
if (d) {
|
2010-12-08 19:35:05 +08:00
|
|
|
l += snprintf(p + l, size - l, "%s", d);
|
2011-08-21 11:09:37 +08:00
|
|
|
g_free(d);
|
2010-12-08 19:35:05 +08:00
|
|
|
} else {
|
2013-05-29 15:56:42 +08:00
|
|
|
return l;
|
2010-12-08 19:35:05 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
l += snprintf(p + l , size - l, "/");
|
|
|
|
|
|
|
|
return l;
|
|
|
|
}
|
|
|
|
|
|
|
|
char* qdev_get_fw_dev_path(DeviceState *dev)
|
|
|
|
{
|
|
|
|
char path[128];
|
|
|
|
int l;
|
|
|
|
|
|
|
|
l = qdev_get_fw_dev_path_helper(dev, path, 128);
|
|
|
|
|
|
|
|
path[l-1] = '\0';
|
|
|
|
|
scsi, pci, qdev, isa-bus, sysbus: don't let *_get_fw_dev_path return NULL
Use g_strdup rather than strdup, because the sole caller
(qdev_get_fw_dev_path_helper) assumes it gets non-NULL, and dereferences
it. Besides, in that caller, the allocated buffer is already freed with
g_free, so it's better to allocate with a matching g_strdup.
In one case, (scsi-bus.c) it was trivial, so I replaced an snprintf+
g_strdup combination with an equivalent g_strdup_printf use.
Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
2012-10-04 19:09:44 +08:00
|
|
|
return g_strdup(path);
|
2010-12-08 19:35:05 +08:00
|
|
|
}
|
2011-12-13 04:29:25 +08:00
|
|
|
|
2012-02-04 02:28:43 +08:00
|
|
|
char *qdev_get_dev_path(DeviceState *dev)
|
2011-12-13 04:29:25 +08:00
|
|
|
{
|
2012-05-02 15:00:20 +08:00
|
|
|
BusClass *bc;
|
2012-02-04 02:28:43 +08:00
|
|
|
|
|
|
|
if (!dev || !dev->parent_bus) {
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2012-05-02 15:00:20 +08:00
|
|
|
bc = BUS_GET_CLASS(dev->parent_bus);
|
|
|
|
if (bc->get_dev_path) {
|
|
|
|
return bc->get_dev_path(dev);
|
2012-02-04 02:28:43 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return NULL;
|
2011-12-13 04:29:26 +08:00
|
|
|
}
|
2011-12-13 04:29:27 +08:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Legacy property handling
|
|
|
|
*/
|
|
|
|
|
2012-01-30 22:55:55 +08:00
|
|
|
static void qdev_get_legacy_property(Object *obj, Visitor *v, void *opaque,
|
2011-12-13 04:29:27 +08:00
|
|
|
const char *name, Error **errp)
|
|
|
|
{
|
2012-01-30 22:55:55 +08:00
|
|
|
DeviceState *dev = DEVICE(obj);
|
2011-12-13 04:29:27 +08:00
|
|
|
Property *prop = opaque;
|
|
|
|
|
2011-12-19 00:05:06 +08:00
|
|
|
char buffer[1024];
|
|
|
|
char *ptr = buffer;
|
2011-12-13 04:29:27 +08:00
|
|
|
|
2011-12-19 00:05:06 +08:00
|
|
|
prop->info->print(dev, prop, buffer, sizeof(buffer));
|
|
|
|
visit_type_str(v, &ptr, name, errp);
|
2011-12-13 04:29:27 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* @qdev_add_legacy_property - adds a legacy property
|
|
|
|
*
|
|
|
|
* Do not use this is new code! Properties added through this interface will
|
2011-12-19 00:05:11 +08:00
|
|
|
* be given names and types in the "legacy" namespace.
|
2011-12-13 04:29:27 +08:00
|
|
|
*
|
2012-02-02 17:17:19 +08:00
|
|
|
* Legacy properties are string versions of other OOM properties. The format
|
|
|
|
* of the string depends on the property type.
|
2011-12-13 04:29:27 +08:00
|
|
|
*/
|
2014-05-03 04:22:57 +08:00
|
|
|
static void qdev_property_add_legacy(DeviceState *dev, Property *prop,
|
|
|
|
Error **errp)
|
2011-12-13 04:29:27 +08:00
|
|
|
{
|
2014-02-08 18:01:48 +08:00
|
|
|
gchar *name;
|
2011-12-13 04:29:27 +08:00
|
|
|
|
2012-05-02 19:31:07 +08:00
|
|
|
/* Register pointer properties as legacy properties */
|
2014-02-08 18:01:47 +08:00
|
|
|
if (!prop->info->print && prop->info->get) {
|
2012-02-02 17:17:19 +08:00
|
|
|
return;
|
|
|
|
}
|
2012-05-02 19:31:07 +08:00
|
|
|
|
2011-12-19 00:05:11 +08:00
|
|
|
name = g_strdup_printf("legacy-%s", prop->name);
|
2014-02-08 18:01:48 +08:00
|
|
|
object_property_add(OBJECT(dev), name, "str",
|
2012-02-02 17:17:19 +08:00
|
|
|
prop->info->print ? qdev_get_legacy_property : prop->info->get,
|
2014-02-08 18:01:47 +08:00
|
|
|
NULL,
|
2012-01-30 22:55:55 +08:00
|
|
|
NULL,
|
|
|
|
prop, errp);
|
2011-12-13 04:29:27 +08:00
|
|
|
|
2011-12-19 00:05:11 +08:00
|
|
|
g_free(name);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* @qdev_property_add_static - add a @Property to a device.
|
|
|
|
*
|
|
|
|
* Static properties access data in a struct. The actual type of the
|
|
|
|
* property and the field depends on the property type.
|
|
|
|
*/
|
|
|
|
void qdev_property_add_static(DeviceState *dev, Property *prop,
|
|
|
|
Error **errp)
|
|
|
|
{
|
2012-04-03 04:40:26 +08:00
|
|
|
Error *local_err = NULL;
|
|
|
|
Object *obj = OBJECT(dev);
|
|
|
|
|
2012-02-02 16:47:13 +08:00
|
|
|
/*
|
|
|
|
* TODO qdev_prop_ptr does not have getters or setters. It must
|
|
|
|
* go now that it can be replaced with links. The test should be
|
|
|
|
* removed along with it: all static properties are read/write.
|
|
|
|
*/
|
|
|
|
if (!prop->info->get && !prop->info->set) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2012-04-03 04:40:26 +08:00
|
|
|
object_property_add(obj, prop->name, prop->info->name,
|
2012-01-30 22:55:55 +08:00
|
|
|
prop->info->get, prop->info->set,
|
2012-02-02 20:08:48 +08:00
|
|
|
prop->info->release,
|
2012-04-03 04:40:26 +08:00
|
|
|
prop, &local_err);
|
|
|
|
|
|
|
|
if (local_err) {
|
|
|
|
error_propagate(errp, local_err);
|
|
|
|
return;
|
|
|
|
}
|
2014-10-07 14:33:22 +08:00
|
|
|
|
|
|
|
object_property_set_description(obj, prop->name,
|
|
|
|
prop->info->description,
|
|
|
|
&error_abort);
|
|
|
|
|
2012-04-03 04:40:26 +08:00
|
|
|
if (prop->qtype == QTYPE_NONE) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (prop->qtype == QTYPE_QBOOL) {
|
2014-01-02 10:48:08 +08:00
|
|
|
object_property_set_bool(obj, prop->defval, prop->name, &error_abort);
|
2012-04-03 04:40:26 +08:00
|
|
|
} else if (prop->info->enum_table) {
|
|
|
|
object_property_set_str(obj, prop->info->enum_table[prop->defval],
|
2014-01-02 10:48:08 +08:00
|
|
|
prop->name, &error_abort);
|
2012-04-03 04:40:26 +08:00
|
|
|
} else if (prop->qtype == QTYPE_QINT) {
|
2014-01-02 10:48:08 +08:00
|
|
|
object_property_set_int(obj, prop->defval, prop->name, &error_abort);
|
2012-04-03 04:40:26 +08:00
|
|
|
}
|
2011-12-13 04:29:42 +08:00
|
|
|
}
|
2011-12-20 06:37:46 +08:00
|
|
|
|
2014-06-18 17:58:32 +08:00
|
|
|
/* @qdev_alias_all_properties - Add alias properties to the source object for
|
|
|
|
* all qdev properties on the target DeviceState.
|
|
|
|
*/
|
|
|
|
void qdev_alias_all_properties(DeviceState *target, Object *source)
|
|
|
|
{
|
|
|
|
ObjectClass *class;
|
|
|
|
Property *prop;
|
|
|
|
|
|
|
|
class = object_get_class(OBJECT(target));
|
|
|
|
do {
|
|
|
|
DeviceClass *dc = DEVICE_CLASS(class);
|
|
|
|
|
|
|
|
for (prop = dc->props; prop && prop->name; prop++) {
|
|
|
|
object_property_add_alias(source, prop->name,
|
|
|
|
OBJECT(target), prop->name,
|
|
|
|
&error_abort);
|
|
|
|
}
|
|
|
|
class = object_class_get_parent(class);
|
|
|
|
} while (class != object_class_by_name(TYPE_DEVICE));
|
|
|
|
}
|
|
|
|
|
2014-04-25 18:44:21 +08:00
|
|
|
static bool device_get_realized(Object *obj, Error **errp)
|
2013-01-09 10:58:11 +08:00
|
|
|
{
|
|
|
|
DeviceState *dev = DEVICE(obj);
|
|
|
|
return dev->realized;
|
|
|
|
}
|
|
|
|
|
2014-04-25 18:44:21 +08:00
|
|
|
static void device_set_realized(Object *obj, bool value, Error **errp)
|
2013-01-09 10:58:11 +08:00
|
|
|
{
|
|
|
|
DeviceState *dev = DEVICE(obj);
|
|
|
|
DeviceClass *dc = DEVICE_GET_CLASS(dev);
|
2014-09-26 17:28:41 +08:00
|
|
|
HotplugHandler *hotplug_ctrl;
|
2014-03-13 04:02:12 +08:00
|
|
|
BusState *bus;
|
2013-01-09 10:58:11 +08:00
|
|
|
Error *local_err = NULL;
|
|
|
|
|
2014-02-05 23:36:46 +08:00
|
|
|
if (dev->hotplugged && !dc->hotpluggable) {
|
2014-04-25 18:44:21 +08:00
|
|
|
error_set(errp, QERR_DEVICE_NO_HOTPLUG, object_get_typename(obj));
|
2014-02-05 23:36:46 +08:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2013-01-09 10:58:11 +08:00
|
|
|
if (value && !dev->realized) {
|
2014-09-02 20:03:05 +08:00
|
|
|
if (!obj->parent) {
|
2013-01-09 10:58:11 +08:00
|
|
|
static int unattached_count;
|
|
|
|
gchar *name = g_strdup_printf("device[%d]", unattached_count++);
|
|
|
|
|
|
|
|
object_property_add_child(container_get(qdev_get_machine(),
|
|
|
|
"/unattached"),
|
2014-09-02 20:03:05 +08:00
|
|
|
name, obj, &error_abort);
|
2013-01-09 10:58:11 +08:00
|
|
|
g_free(name);
|
|
|
|
}
|
|
|
|
|
2013-04-11 22:51:56 +08:00
|
|
|
if (dc->realize) {
|
|
|
|
dc->realize(dev, &local_err);
|
|
|
|
}
|
|
|
|
|
2014-09-04 10:18:26 +08:00
|
|
|
if (local_err != NULL) {
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
|
2014-09-26 17:28:41 +08:00
|
|
|
hotplug_ctrl = qdev_get_hotplug_handler(dev);
|
|
|
|
if (hotplug_ctrl) {
|
|
|
|
hotplug_handler_plug(hotplug_ctrl, dev, &local_err);
|
2014-02-05 23:36:52 +08:00
|
|
|
}
|
|
|
|
|
2014-09-04 10:18:26 +08:00
|
|
|
if (local_err != NULL) {
|
|
|
|
goto post_realize_fail;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (qdev_get_vmsd(dev)) {
|
2013-01-09 10:58:11 +08:00
|
|
|
vmstate_register_with_alias_id(dev, -1, qdev_get_vmsd(dev), dev,
|
|
|
|
dev->instance_id_alias,
|
|
|
|
dev->alias_required_for_version);
|
|
|
|
}
|
2014-09-04 10:18:26 +08:00
|
|
|
|
|
|
|
QLIST_FOREACH(bus, &dev->child_bus, sibling) {
|
|
|
|
object_property_set_bool(OBJECT(bus), true, "realized",
|
2014-03-13 04:02:12 +08:00
|
|
|
&local_err);
|
2014-09-04 10:18:26 +08:00
|
|
|
if (local_err != NULL) {
|
|
|
|
goto child_realize_fail;
|
2014-03-13 04:02:12 +08:00
|
|
|
}
|
|
|
|
}
|
2014-09-04 10:18:26 +08:00
|
|
|
if (dev->hotplugged) {
|
2013-01-09 10:58:11 +08:00
|
|
|
device_reset(dev);
|
|
|
|
}
|
2014-06-26 21:10:03 +08:00
|
|
|
dev->pending_deleted_event = false;
|
2013-01-09 10:58:11 +08:00
|
|
|
} else if (!value && dev->realized) {
|
2014-09-04 10:18:25 +08:00
|
|
|
Error **local_errp = NULL;
|
2014-03-13 04:02:12 +08:00
|
|
|
QLIST_FOREACH(bus, &dev->child_bus, sibling) {
|
2014-09-04 10:18:25 +08:00
|
|
|
local_errp = local_err ? NULL : &local_err;
|
2014-03-13 04:02:12 +08:00
|
|
|
object_property_set_bool(OBJECT(bus), false, "realized",
|
2014-09-04 10:18:25 +08:00
|
|
|
local_errp);
|
2014-03-13 04:02:12 +08:00
|
|
|
}
|
2014-09-04 10:18:25 +08:00
|
|
|
if (qdev_get_vmsd(dev)) {
|
2013-04-16 00:34:10 +08:00
|
|
|
vmstate_unregister(dev, qdev_get_vmsd(dev), dev);
|
|
|
|
}
|
2014-09-04 10:18:25 +08:00
|
|
|
if (dc->unrealize) {
|
|
|
|
local_errp = local_err ? NULL : &local_err;
|
|
|
|
dc->unrealize(dev, local_errp);
|
2013-01-09 10:58:11 +08:00
|
|
|
}
|
2014-06-26 21:10:03 +08:00
|
|
|
dev->pending_deleted_event = true;
|
2013-01-09 10:58:11 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
if (local_err != NULL) {
|
2014-09-04 10:18:26 +08:00
|
|
|
goto fail;
|
2013-01-09 10:58:11 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
dev->realized = value;
|
2014-09-04 10:18:26 +08:00
|
|
|
return;
|
|
|
|
|
|
|
|
child_realize_fail:
|
|
|
|
QLIST_FOREACH(bus, &dev->child_bus, sibling) {
|
|
|
|
object_property_set_bool(OBJECT(bus), false, "realized",
|
|
|
|
NULL);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (qdev_get_vmsd(dev)) {
|
|
|
|
vmstate_unregister(dev, qdev_get_vmsd(dev), dev);
|
|
|
|
}
|
|
|
|
|
|
|
|
post_realize_fail:
|
|
|
|
if (dc->unrealize) {
|
|
|
|
dc->unrealize(dev, NULL);
|
|
|
|
}
|
|
|
|
|
|
|
|
fail:
|
|
|
|
error_propagate(errp, local_err);
|
|
|
|
return;
|
2013-01-09 10:58:11 +08:00
|
|
|
}
|
|
|
|
|
2014-04-25 18:44:21 +08:00
|
|
|
static bool device_get_hotpluggable(Object *obj, Error **errp)
|
2014-02-05 23:36:46 +08:00
|
|
|
{
|
|
|
|
DeviceClass *dc = DEVICE_GET_CLASS(obj);
|
|
|
|
DeviceState *dev = DEVICE(obj);
|
|
|
|
|
2014-03-08 02:04:13 +08:00
|
|
|
return dc->hotpluggable && (dev->parent_bus == NULL ||
|
2014-09-26 17:28:17 +08:00
|
|
|
qbus_is_hotpluggable(dev->parent_bus));
|
2014-02-05 23:36:46 +08:00
|
|
|
}
|
|
|
|
|
2014-06-02 21:25:04 +08:00
|
|
|
static bool device_get_hotplugged(Object *obj, Error **err)
|
|
|
|
{
|
|
|
|
DeviceState *dev = DEVICE(obj);
|
|
|
|
|
|
|
|
return dev->hotplugged;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void device_set_hotplugged(Object *obj, bool value, Error **err)
|
|
|
|
{
|
|
|
|
DeviceState *dev = DEVICE(obj);
|
|
|
|
|
|
|
|
dev->hotplugged = value;
|
|
|
|
}
|
|
|
|
|
2011-12-23 05:06:37 +08:00
|
|
|
static void device_initfn(Object *obj)
|
|
|
|
{
|
|
|
|
DeviceState *dev = DEVICE(obj);
|
2012-03-29 00:12:47 +08:00
|
|
|
ObjectClass *class;
|
2011-12-23 05:06:37 +08:00
|
|
|
Property *prop;
|
|
|
|
|
|
|
|
if (qdev_hotplug) {
|
|
|
|
dev->hotplugged = 1;
|
|
|
|
qdev_hot_added = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
dev->instance_id_alias = -1;
|
2013-01-09 10:58:10 +08:00
|
|
|
dev->realized = false;
|
2011-12-23 05:06:37 +08:00
|
|
|
|
2013-01-09 10:58:11 +08:00
|
|
|
object_property_add_bool(obj, "realized",
|
|
|
|
device_get_realized, device_set_realized, NULL);
|
2014-02-05 23:36:46 +08:00
|
|
|
object_property_add_bool(obj, "hotpluggable",
|
|
|
|
device_get_hotpluggable, NULL, NULL);
|
2014-06-02 21:25:04 +08:00
|
|
|
object_property_add_bool(obj, "hotplugged",
|
|
|
|
device_get_hotplugged, device_set_hotplugged,
|
|
|
|
&error_abort);
|
2013-01-09 10:58:11 +08:00
|
|
|
|
2012-03-29 00:12:47 +08:00
|
|
|
class = object_get_class(OBJECT(dev));
|
|
|
|
do {
|
|
|
|
for (prop = DEVICE_CLASS(class)->props; prop && prop->name; prop++) {
|
2014-01-02 10:48:08 +08:00
|
|
|
qdev_property_add_legacy(dev, prop, &error_abort);
|
|
|
|
qdev_property_add_static(dev, prop, &error_abort);
|
2012-03-29 00:12:47 +08:00
|
|
|
}
|
|
|
|
class = object_class_get_parent(class);
|
|
|
|
} while (class != object_class_by_name(TYPE_DEVICE));
|
2011-12-23 05:06:37 +08:00
|
|
|
|
2012-05-02 16:39:01 +08:00
|
|
|
object_property_add_link(OBJECT(dev), "parent_bus", TYPE_BUS,
|
2014-03-19 15:58:56 +08:00
|
|
|
(Object **)&dev->parent_bus, NULL, 0,
|
qom: Make QOM link property unref optional
Some object_property_add_link() callers expect property deletion to
unref the link property object. Other callers expect to manage the
refcount themselves. The former are currently broken and therefore leak
the link property object.
This patch adds a flags argument to object_property_add_link() so the
caller can specify which refcount behavior they require. The new
OBJ_PROP_LINK_UNREF_ON_RELEASE flag causes the link pointer to be
unreferenced when the property is deleted.
This fixes refcount leaks in qdev.c, xilinx_axidma.c, xilinx_axienet.c,
s390-virtio-bus.c, virtio-pci.c, virtio-rng.c, and ui/console.c.
Rationale for refcount behavior:
* hw/core/qdev.c
- bus children are explicitly unreferenced, don't interfere
- parent_bus is essentially a read-only property that doesn't hold a
refcount, don't unref
- hotplug_handler is leaked, do unref
* hw/dma/xilinx_axidma.c
- rx stream "dma" links are set using set_link, therefore they
need unref
- tx streams are set using set_link, therefore they need unref
* hw/net/xilinx_axienet.c
- same reasoning as hw/dma/xilinx_axidma.c
* hw/pcmcia/pxa2xx.c
- pxa2xx bypasses set_link and therefore does not use refcounts
* hw/s390x/s390-virtio-bus.c
* hw/virtio/virtio-pci.c
* hw/virtio/virtio-rng.c
* ui/console.c
- set_link is used and there is no explicit unref, do unref
Cc: Peter Crosthwaite <peter.crosthwaite@petalogix.com>
Cc: Alexander Graf <agraf@suse.de>
Cc: Anthony Liguori <aliguori@amazon.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
2014-03-19 15:58:55 +08:00
|
|
|
&error_abort);
|
2014-05-20 14:30:58 +08:00
|
|
|
QLIST_INIT(&dev->gpios);
|
2011-12-23 05:06:37 +08:00
|
|
|
}
|
|
|
|
|
2013-07-11 04:08:42 +08:00
|
|
|
static void device_post_init(Object *obj)
|
|
|
|
{
|
2014-07-04 03:45:34 +08:00
|
|
|
Error *err = NULL;
|
|
|
|
qdev_prop_set_globals(DEVICE(obj), &err);
|
|
|
|
if (err) {
|
|
|
|
qerror_report_err(err);
|
|
|
|
error_free(err);
|
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
2013-07-11 04:08:42 +08:00
|
|
|
}
|
|
|
|
|
2011-12-23 22:38:56 +08:00
|
|
|
/* Unlink device from bus and free the structure. */
|
|
|
|
static void device_finalize(Object *obj)
|
|
|
|
{
|
2014-05-20 14:30:58 +08:00
|
|
|
NamedGPIOList *ngl, *next;
|
|
|
|
|
2011-12-23 22:38:56 +08:00
|
|
|
DeviceState *dev = DEVICE(obj);
|
2013-01-25 21:12:34 +08:00
|
|
|
if (dev->opts) {
|
|
|
|
qemu_opts_del(dev->opts);
|
2011-12-23 22:38:56 +08:00
|
|
|
}
|
2014-05-20 14:30:58 +08:00
|
|
|
|
|
|
|
QLIST_FOREACH_SAFE(ngl, &dev->gpios, node, next) {
|
|
|
|
QLIST_REMOVE(ngl, node);
|
2014-06-18 15:56:31 +08:00
|
|
|
qemu_free_irqs(ngl->in, ngl->num_in);
|
2014-05-20 14:30:58 +08:00
|
|
|
g_free(ngl->name);
|
|
|
|
g_free(ngl);
|
|
|
|
/* ngl->out irqs are owned by the other end and should not be freed
|
|
|
|
* here
|
|
|
|
*/
|
|
|
|
}
|
2011-12-23 22:38:56 +08:00
|
|
|
}
|
|
|
|
|
2012-03-29 00:12:47 +08:00
|
|
|
static void device_class_base_init(ObjectClass *class, void *data)
|
|
|
|
{
|
|
|
|
DeviceClass *klass = DEVICE_CLASS(class);
|
|
|
|
|
|
|
|
/* We explicitly look up properties in the superclasses,
|
|
|
|
* so do not propagate them to the subclasses.
|
|
|
|
*/
|
|
|
|
klass->props = NULL;
|
2011-12-23 22:38:56 +08:00
|
|
|
}
|
|
|
|
|
2013-01-05 01:13:00 +08:00
|
|
|
static void device_unparent(Object *obj)
|
2012-11-23 16:47:13 +08:00
|
|
|
{
|
|
|
|
DeviceState *dev = DEVICE(obj);
|
2013-01-25 21:12:34 +08:00
|
|
|
BusState *bus;
|
2012-11-23 16:47:13 +08:00
|
|
|
|
2014-03-13 04:02:12 +08:00
|
|
|
if (dev->realized) {
|
|
|
|
object_property_set_bool(obj, false, "realized", NULL);
|
|
|
|
}
|
2013-01-25 21:12:34 +08:00
|
|
|
while (dev->num_child_bus) {
|
|
|
|
bus = QLIST_FIRST(&dev->child_bus);
|
2013-12-19 00:15:51 +08:00
|
|
|
object_unparent(OBJECT(bus));
|
2013-01-25 21:12:34 +08:00
|
|
|
}
|
|
|
|
if (dev->parent_bus) {
|
2013-01-05 01:13:00 +08:00
|
|
|
bus_remove_child(dev->parent_bus, dev);
|
2013-01-25 21:12:35 +08:00
|
|
|
object_unref(OBJECT(dev->parent_bus));
|
|
|
|
dev->parent_bus = NULL;
|
2013-01-05 01:13:00 +08:00
|
|
|
}
|
2013-03-06 20:58:59 +08:00
|
|
|
|
2013-03-28 00:36:14 +08:00
|
|
|
/* Only send event if the device had been completely realized */
|
2014-06-26 21:10:03 +08:00
|
|
|
if (dev->pending_deleted_event) {
|
2013-03-28 00:36:14 +08:00
|
|
|
gchar *path = object_get_canonical_path(OBJECT(dev));
|
|
|
|
|
2014-06-18 14:43:43 +08:00
|
|
|
qapi_event_send_device_deleted(!!dev->id, dev->id, path, &error_abort);
|
2013-03-28 00:36:14 +08:00
|
|
|
g_free(path);
|
2013-03-06 20:58:59 +08:00
|
|
|
}
|
2012-11-23 16:47:13 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static void device_class_init(ObjectClass *class, void *data)
|
|
|
|
{
|
2013-01-09 10:58:11 +08:00
|
|
|
DeviceClass *dc = DEVICE_CLASS(class);
|
|
|
|
|
2013-01-05 01:13:00 +08:00
|
|
|
class->unparent = device_unparent;
|
2013-01-09 10:58:11 +08:00
|
|
|
dc->realize = device_realize;
|
2013-04-16 00:34:10 +08:00
|
|
|
dc->unrealize = device_unrealize;
|
2014-02-19 00:56:53 +08:00
|
|
|
|
|
|
|
/* by default all devices were considered as hotpluggable,
|
|
|
|
* so with intent to check it in generic qdev_unplug() /
|
|
|
|
* device_set_realized() functions make every device
|
|
|
|
* hotpluggable. Devices that shouldn't be hotpluggable,
|
|
|
|
* should override it in their class_init()
|
|
|
|
*/
|
|
|
|
dc->hotpluggable = true;
|
2012-11-23 16:47:13 +08:00
|
|
|
}
|
|
|
|
|
2011-12-05 01:36:01 +08:00
|
|
|
void device_reset(DeviceState *dev)
|
|
|
|
{
|
|
|
|
DeviceClass *klass = DEVICE_GET_CLASS(dev);
|
|
|
|
|
|
|
|
if (klass->reset) {
|
|
|
|
klass->reset(dev);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-03-28 22:34:12 +08:00
|
|
|
Object *qdev_get_machine(void)
|
|
|
|
{
|
|
|
|
static Object *dev;
|
|
|
|
|
|
|
|
if (dev == NULL) {
|
2012-04-05 19:21:46 +08:00
|
|
|
dev = container_get(object_get_root(), "/machine");
|
2012-03-28 22:34:12 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return dev;
|
|
|
|
}
|
|
|
|
|
2013-01-10 23:19:07 +08:00
|
|
|
static const TypeInfo device_type_info = {
|
2011-12-17 04:34:46 +08:00
|
|
|
.name = TYPE_DEVICE,
|
|
|
|
.parent = TYPE_OBJECT,
|
|
|
|
.instance_size = sizeof(DeviceState),
|
2011-12-23 05:06:37 +08:00
|
|
|
.instance_init = device_initfn,
|
2013-07-11 04:08:42 +08:00
|
|
|
.instance_post_init = device_post_init,
|
2011-12-23 22:38:56 +08:00
|
|
|
.instance_finalize = device_finalize,
|
2012-03-29 00:12:47 +08:00
|
|
|
.class_base_init = device_class_base_init,
|
2012-11-23 16:47:13 +08:00
|
|
|
.class_init = device_class_init,
|
2011-12-17 04:34:46 +08:00
|
|
|
.abstract = true,
|
|
|
|
.class_size = sizeof(DeviceClass),
|
|
|
|
};
|
|
|
|
|
2012-02-04 03:32:19 +08:00
|
|
|
static void qbus_initfn(Object *obj)
|
|
|
|
{
|
|
|
|
BusState *bus = BUS(obj);
|
|
|
|
|
|
|
|
QTAILQ_INIT(&bus->children);
|
2014-02-05 23:36:45 +08:00
|
|
|
object_property_add_link(obj, QDEV_HOTPLUG_HANDLER_PROPERTY,
|
|
|
|
TYPE_HOTPLUG_HANDLER,
|
qom: Make QOM link property unref optional
Some object_property_add_link() callers expect property deletion to
unref the link property object. Other callers expect to manage the
refcount themselves. The former are currently broken and therefore leak
the link property object.
This patch adds a flags argument to object_property_add_link() so the
caller can specify which refcount behavior they require. The new
OBJ_PROP_LINK_UNREF_ON_RELEASE flag causes the link pointer to be
unreferenced when the property is deleted.
This fixes refcount leaks in qdev.c, xilinx_axidma.c, xilinx_axienet.c,
s390-virtio-bus.c, virtio-pci.c, virtio-rng.c, and ui/console.c.
Rationale for refcount behavior:
* hw/core/qdev.c
- bus children are explicitly unreferenced, don't interfere
- parent_bus is essentially a read-only property that doesn't hold a
refcount, don't unref
- hotplug_handler is leaked, do unref
* hw/dma/xilinx_axidma.c
- rx stream "dma" links are set using set_link, therefore they
need unref
- tx streams are set using set_link, therefore they need unref
* hw/net/xilinx_axienet.c
- same reasoning as hw/dma/xilinx_axidma.c
* hw/pcmcia/pxa2xx.c
- pxa2xx bypasses set_link and therefore does not use refcounts
* hw/s390x/s390-virtio-bus.c
* hw/virtio/virtio-pci.c
* hw/virtio/virtio-rng.c
* ui/console.c
- set_link is used and there is no explicit unref, do unref
Cc: Peter Crosthwaite <peter.crosthwaite@petalogix.com>
Cc: Alexander Graf <agraf@suse.de>
Cc: Anthony Liguori <aliguori@amazon.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
2014-03-19 15:58:55 +08:00
|
|
|
(Object **)&bus->hotplug_handler,
|
2014-03-19 15:58:56 +08:00
|
|
|
object_property_allow_set_link,
|
qom: Make QOM link property unref optional
Some object_property_add_link() callers expect property deletion to
unref the link property object. Other callers expect to manage the
refcount themselves. The former are currently broken and therefore leak
the link property object.
This patch adds a flags argument to object_property_add_link() so the
caller can specify which refcount behavior they require. The new
OBJ_PROP_LINK_UNREF_ON_RELEASE flag causes the link pointer to be
unreferenced when the property is deleted.
This fixes refcount leaks in qdev.c, xilinx_axidma.c, xilinx_axienet.c,
s390-virtio-bus.c, virtio-pci.c, virtio-rng.c, and ui/console.c.
Rationale for refcount behavior:
* hw/core/qdev.c
- bus children are explicitly unreferenced, don't interfere
- parent_bus is essentially a read-only property that doesn't hold a
refcount, don't unref
- hotplug_handler is leaked, do unref
* hw/dma/xilinx_axidma.c
- rx stream "dma" links are set using set_link, therefore they
need unref
- tx streams are set using set_link, therefore they need unref
* hw/net/xilinx_axienet.c
- same reasoning as hw/dma/xilinx_axidma.c
* hw/pcmcia/pxa2xx.c
- pxa2xx bypasses set_link and therefore does not use refcounts
* hw/s390x/s390-virtio-bus.c
* hw/virtio/virtio-pci.c
* hw/virtio/virtio-rng.c
* ui/console.c
- set_link is used and there is no explicit unref, do unref
Cc: Peter Crosthwaite <peter.crosthwaite@petalogix.com>
Cc: Alexander Graf <agraf@suse.de>
Cc: Anthony Liguori <aliguori@amazon.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
2014-03-19 15:58:55 +08:00
|
|
|
OBJ_PROP_LINK_UNREF_ON_RELEASE,
|
|
|
|
NULL);
|
2013-11-26 06:48:40 +08:00
|
|
|
object_property_add_bool(obj, "realized",
|
|
|
|
bus_get_realized, bus_set_realized, NULL);
|
2012-02-04 03:32:19 +08:00
|
|
|
}
|
|
|
|
|
2013-05-29 15:56:42 +08:00
|
|
|
static char *default_bus_get_fw_dev_path(DeviceState *dev)
|
|
|
|
{
|
|
|
|
return g_strdup(object_get_typename(OBJECT(dev)));
|
|
|
|
}
|
|
|
|
|
2013-01-25 21:12:33 +08:00
|
|
|
static void bus_class_init(ObjectClass *class, void *data)
|
|
|
|
{
|
2013-05-29 15:56:42 +08:00
|
|
|
BusClass *bc = BUS_CLASS(class);
|
|
|
|
|
2013-01-25 21:12:33 +08:00
|
|
|
class->unparent = bus_unparent;
|
2013-05-29 15:56:42 +08:00
|
|
|
bc->get_fw_dev_path = default_bus_get_fw_dev_path;
|
2013-01-25 21:12:33 +08:00
|
|
|
}
|
|
|
|
|
2012-02-04 03:32:19 +08:00
|
|
|
static void qbus_finalize(Object *obj)
|
|
|
|
{
|
|
|
|
BusState *bus = BUS(obj);
|
|
|
|
|
|
|
|
g_free((char *)bus->name);
|
|
|
|
}
|
|
|
|
|
2012-05-02 15:00:20 +08:00
|
|
|
static const TypeInfo bus_info = {
|
|
|
|
.name = TYPE_BUS,
|
|
|
|
.parent = TYPE_OBJECT,
|
|
|
|
.instance_size = sizeof(BusState),
|
|
|
|
.abstract = true,
|
|
|
|
.class_size = sizeof(BusClass),
|
2012-02-04 03:32:19 +08:00
|
|
|
.instance_init = qbus_initfn,
|
|
|
|
.instance_finalize = qbus_finalize,
|
2013-01-25 21:12:33 +08:00
|
|
|
.class_init = bus_class_init,
|
2012-05-02 15:00:20 +08:00
|
|
|
};
|
|
|
|
|
2012-02-09 22:20:55 +08:00
|
|
|
static void qdev_register_types(void)
|
2011-12-17 04:34:46 +08:00
|
|
|
{
|
2012-05-02 15:00:20 +08:00
|
|
|
type_register_static(&bus_info);
|
2011-12-17 04:34:46 +08:00
|
|
|
type_register_static(&device_type_info);
|
|
|
|
}
|
|
|
|
|
2012-02-09 22:20:55 +08:00
|
|
|
type_init(qdev_register_types)
|