[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus.
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus. |
Date: |
Fri, 23 Nov 2012 13:08:26 +0100 |
On Thu, 22 Nov 2012 15:50:50 +0100
address@hidden wrote:
> +/* Create a virtio bus. */
> +VirtioBus *virtio_bus_new(DeviceState *host, const VirtioBusInfo *info)
> +{
> + /*
> + * This is needed, as we want to have different names for each
> virtio-bus.
> + * If we don't do that, we can't add more than one VirtIODevice.
> + */
> + static int next_virtio_bus;
> + char *bus_name = g_strdup_printf("virtio-bus.%d", next_virtio_bus++);
This still has the overflow/id-reuse problem, hasn't it?
> +
> + BusState *qbus = qbus_create(TYPE_VIRTIO_BUS, host, bus_name);
> + VirtioBus *bus = VIRTIO_BUS(qbus);
> + bus->info = info;
> + qbus->allow_hotplug = 0;
> + bus->bus_in_use = false;
> + DPRINTF("%s bus created\n", bus_name);
> + return bus;
> +}
Don't you need a way to destroy the bus again when the proxy device is
hotunplugged?
Re: [Qemu-devel] [RFC PATCH v2 2/3] virtio-pci : add a virtio-bus interface, Anthony Liguori, 2012/11/26
[Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus., fred . konrad, 2012/11/22
Re: [Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus., Stefan Hajnoczi, 2012/11/23
Re: [Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus., Andreas Färber, 2012/11/24
Re: [Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus., Anthony Liguori, 2012/11/26