[libvirt] [Qemu-devel] [PATCH] qdev: DEVICE_DELETED event

[Laine Stump]

On 03/08/2013 04:25 AM, Jiri Denemark wrote:
> On Fri, Mar 08, 2013 at 09:50:55 +0100, Markus Armbruster wrote:
>> Osier Yang <jyang at redhat.com> writes:
>>
>>> I'm wondering if it could be long time to wait for the device_del
>>> completes (AFAIK from previous bugs, it can be, though it should be
>>> fine for most of the cases). If it's too long, it will be a problem
>>> for management, because it looks like hanging. We can have a timeout
>>> for the device_del in libvirt, but the problem is the device_del
>>> can be still in progress by qemu, which could cause the inconsistency.
>>> Unless qemu has some command to cancel the device_del.
>> I'm afraid cancelling isn't possible, at least not for PCI.
> I don't think we need anything like that. We just need the device
> deletion API to return immediately without actually removing stuff from
> domain definition

I don't think we can do that - it changes the user-visible semantics. I
think we need to continue to remove the device from the XML immediately,
but internally keep track of the fact that this device (and the qemu id
used to refer to it) can't yet be re-used.


The qemu driver currently has activeHostPciDevs and inactiveHostPciDevs.
Maybe we also need a "zombieHostPciDevs" for devices that we've sent the
device_del command for, but haven't yet received notice that they're
actually removed.

(BTW, shouldn't these lists of devices be global to all of libvirt,
rather than qemu-specific?)