[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]