On Thu, Jan 22, 2015 at 15:23:03 +0100, Ján Tomko wrote: > On 01/22/2015 10:20 AM, Peter Krempa wrote: > > While reviewing Martin's reference counting series I've noticed a few qemu API > > impls that don't properly handle jobs. > > > > Peter Krempa (7): > > qemu: Fix job handling in qemuDomainPinVcpuFlags > > qemu: Fix job handling in qemuDomainPinEmulator > > qemu: Fix job handling in qemuDomainSetAutostart > > qemu: Fix job handling in qemuDomainSetMemoryParameters > > qemu: Fix job handling in qemuDomainSetSchedulerParametersFlags > > qemu: Fix job type in qemuDomainGetBlockIoTune > > qemu: Fix job handling in qemuDomainSetMetadata > > > > src/qemu/qemu_driver.c | 135 +++++++++++++++++++++++++++++++------------------ > > 1 file changed, 87 insertions(+), 48 deletions(-) > > > > In patches 1-3,5 it's necessary to check if the domain is still alive after > getting the job. > > I suggest moving the BeginJob between the ACL check and > virDomainLiveConfigHelperMethod, which does check for domain liveness. > > ACK with that fix. > I've moved the job acquisition to the place you've suggested and pushed the series now that the release is out. Thanks Peter
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list