[libvirt] [Xen-devel] [PATCH 2/3] libxl: acquire a job when destroying a domain

Martin Kletzander mkletzan at redhat.com
Wed Apr 1 10:12:05 UTC 2015


On Thu, Mar 26, 2015 at 03:29:51PM -0600, Jim Fehlig wrote:
>Konrad Rzeszutek Wilk wrote:
>> On Wed, Mar 25, 2015 at 02:08:35PM -0600, Jim Fehlig wrote:
>>
>>> A job should be acquired at the beginning of a domain destroy operation,
>>> not at the end when cleaning up the domain.  Fix two occurances of this

And s/occurances/occurrences/ here.

It looks fine, though, with the squash-in.

Also, if you want to have a look at some other things that might be
fixed here, plus some speed-up gained, have a look at my commit
540c339a, that does some similar things in the QEMU driver.

>>> late job acquisition in the libxl driver.  Doing so renders
>>> libxlDomainCleanup unused, so it is removed.
>>>
>
>Just noticed this should be libxlDomainCleanupJob.
>

Yes :)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150401/8d6b895f/attachment-0001.sig>


More information about the libvir-list mailing list