Re: [libvirt] [PATCH 11/11] libxl: use job functions in libxlDomainSetSchedulerParametersFlags

On 07.02.2014 04:53, Jim Fehlig wrote:

Modify operation that needs to wait in the queue of modify jobs.

Signed-off-by: Jim Fehlig <jfehlig@suse.com> --- src/libxl/libxl_driver.c | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c index 79d64a5..a4b6ecd 100644 --- a/src/libxl/libxl_driver.c +++ b/src/libxl/libxl_driver.c

@@ -4102,11 +4106,14 @@ libxlDomainSetSchedulerParametersFlags(virDomainPtr dom, virReportError(VIR_ERR_INTERNAL_ERROR, _("Failed to set scheduler parameters for domain '%d'" " with libxenlight"), dom->id); - goto cleanup; + goto endjob; }

ret = 0;

+endjob: + libxlDomainObjEndJob(driver, vm); + cleanup: if (vm) virObjectUnlock(vm);

Conditional ACK Michal