[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions |
Date: |
Thu, 18 Apr 2013 11:01:51 +0200 |
On Thu, 18 Apr 2013 10:41:56 +0200
Igor Mammedov <address@hidden> wrote:
[...]
> > - if (!bus) {
> > - bus = sysbus_get_default();
> > - }
> > -
> I've checked all direct childs of TYPE_DEVICE and they all set k->bus_type,
> with only one exception of TYPE_CPU. So it should be safe to remove fallback
> from qdev_device_add POV.
> However TYPE_CPU breaks assumption that device always has parent_bus set
> to not NULL in qdev_unplug() and qdev_print()
Err, qdev_print() is safe since it's called on bus children only, so it has
parent_bus.
>
> It would be better to add something like this:
> // untested
>
> diff --git a/hw/core/qdev.c b/hw/core/qdev.c
> index 4eb0134..45009ba 100644
> --- a/hw/core/qdev.c
> +++ b/hw/core/qdev.c
> @@ -207,7 +207,7 @@ void qdev_unplug(DeviceState *dev, Error **errp)
> {
> DeviceClass *dc = DEVICE_GET_CLASS(dev);
>
> - if (!dev->parent_bus->allow_hotplug) {
> + if (dev->parent_bus && !dev->parent_bus->allow_hotplug) {
> error_set(errp, QERR_BUS_NO_HOTPLUG, dev->parent_bus->name);
> return;
> }
> diff --git a/qdev-monitor.c b/qdev-monitor.c
> index 9a78ccf..2476e4e 100644
> --- a/qdev-monitor.c
> +++ b/qdev-monitor.c
> @@ -557,7 +557,9 @@ static void qdev_print(Monitor *mon, DeviceState *dev,
> int indent) qdev_print_props(mon, dev, DEVICE_CLASS(class)->props, indent);
> class = object_class_get_parent(class);
> } while (class != object_class_by_name(TYPE_DEVICE));
> - bus_print_dev(dev->parent_bus, mon, dev, indent);
> + if (dev->parent_bus) {
> + bus_print_dev(dev->parent_bus, mon, dev, indent);
> + }
> QLIST_FOREACH(child, &dev->child_bus, sibling) {
> qbus_print(mon, child, indent);
> }
>
> > /* create device, set properties */
> > qdev = DEVICE(object_new(driver));
> > - qdev_set_parent_bus(qdev, bus);
> > +
> > + if (bus) {
> > + qdev_set_parent_bus(qdev, bus);
> > + }
> >
> > id = qemu_opts_id(opts);
> > if (id) {
>
>
- [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Andreas Färber, 2013/04/15
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Igor Mammedov, 2013/04/18
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions,
Igor Mammedov <=
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Libaiqing, 2013/04/22
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Andreas Färber, 2013/04/22
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, KONRAD Frédéric, 2013/04/22
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Andreas Färber, 2013/04/22
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, KONRAD Frédéric, 2013/04/22
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Andreas Färber, 2013/04/22
- Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, KONRAD Frédéric, 2013/04/22
Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Andreas Färber, 2013/04/22
Re: [Qemu-devel] [PATCH] qdev: Fix device_add bus assumptions, Anthony Liguori, 2013/04/22