[Xen-changelog] [xen-unstable] libxl: convert libxl_device_vfb_add to an async operation

[Xen patchbot-unstable <patchbot@xxxxxxx>]

# HG changeset patch
# User Roger Pau Monne <roger.pau@xxxxxxxxxx>
# Date 1343317657 -3600
# Node ID c29feacecfc4b0564b364adf0fd9c8bf19e250d8
# Parent  21ffb640c61ccd53784c8d4c3fbf2a453db4bb60
libxl: convert libxl_device_vfb_add to an async operation

Split libxl_device_vfb_add into libxl__device_vfb_add (to be used
inside already running ao's), and make libxl_device_vfb_add a stub
to call libxl__device_vfb_add.

Signed-off-by: Roger Pau Monne <roger.pau@xxxxxxxxxx>
Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Committed-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---


diff -r 21ffb640c61c -r c29feacecfc4 tools/libxl/libxl.c
--- a/tools/libxl/libxl.c       Thu Jul 26 16:47:36 2012 +0100
+++ b/tools/libxl/libxl.c       Thu Jul 26 16:47:37 2012 +0100
@@ -3019,9 +3019,25 @@ static int libxl__device_from_vfb(libxl_
     return 0;
 }
 
-int libxl_device_vfb_add(libxl_ctx *ctx, uint32_t domid, libxl_device_vfb *vfb)
+int libxl_device_vfb_add(libxl_ctx *ctx, uint32_t domid, libxl_device_vfb *vfb,
+                         const libxl_asyncop_how *ao_how)
 {
-    GC_INIT(ctx);
+    AO_CREATE(ctx, domid, ao_how);
+    int rc;
+
+    rc = libxl__device_vfb_add(gc, domid, vfb);
+    if (rc) {
+        LOG(ERROR, "unable to add vfb device");
+        goto out;
+    }
+
+out:
+    libxl__ao_complete(egc, ao, rc);
+    return AO_INPROGRESS;
+}
+
+int libxl__device_vfb_add(libxl__gc *gc, uint32_t domid, libxl_device_vfb *vfb)
+{
     flexarray_t *front;
     flexarray_t *back;
     libxl__device device;
@@ -3079,7 +3095,6 @@ out_free:
     flexarray_free(front);
     flexarray_free(back);
 out:
-    GC_FREE;
     return rc;
 }
 
diff -r 21ffb640c61c -r c29feacecfc4 tools/libxl/libxl.h
--- a/tools/libxl/libxl.h       Thu Jul 26 16:47:36 2012 +0100
+++ b/tools/libxl/libxl.h       Thu Jul 26 16:47:37 2012 +0100
@@ -721,7 +721,8 @@ int libxl_device_vkb_destroy(libxl_ctx *
                              const libxl_asyncop_how *ao_how);
 
 /* Framebuffer */
-int libxl_device_vfb_add(libxl_ctx *ctx, uint32_t domid, libxl_device_vfb 
*vfb);
+int libxl_device_vfb_add(libxl_ctx *ctx, uint32_t domid, libxl_device_vfb *vfb,
+                         const libxl_asyncop_how *ao_how);
 int libxl_device_vfb_remove(libxl_ctx *ctx, uint32_t domid,
                             libxl_device_vfb *vfb,
                             const libxl_asyncop_how *ao_how);
diff -r 21ffb640c61c -r c29feacecfc4 tools/libxl/libxl_create.c
--- a/tools/libxl/libxl_create.c        Thu Jul 26 16:47:36 2012 +0100
+++ b/tools/libxl/libxl_create.c        Thu Jul 26 16:47:37 2012 +0100
@@ -932,7 +932,6 @@ static void domcreate_launch_dm(libxl__e
     const uint32_t domid = dcs->guest_domid;
     libxl_domain_config *const d_config = dcs->guest_config;
     libxl__domain_build_state *const state = &dcs->build_state;
-    libxl_ctx *const ctx = CTX;
 
     if (ret) {
         LOG(ERROR, "unable to add disk devices");
@@ -976,7 +975,7 @@ static void domcreate_launch_dm(libxl__e
         libxl__device_console console;
 
         for (i = 0; i < d_config->num_vfbs; i++) {
-            libxl_device_vfb_add(ctx, domid, &d_config->vfbs[i]);
+            libxl__device_vfb_add(gc, domid, &d_config->vfbs[i]);
             libxl__device_vkb_add(gc, domid, &d_config->vkbs[i]);
         }
 
diff -r 21ffb640c61c -r c29feacecfc4 tools/libxl/libxl_dm.c
--- a/tools/libxl/libxl_dm.c    Thu Jul 26 16:47:36 2012 +0100
+++ b/tools/libxl/libxl_dm.c    Thu Jul 26 16:47:37 2012 +0100
@@ -902,7 +902,7 @@ static void spawn_stub_launch_dm(libxl__
         if (ret)
             goto out;
     }
-    ret = libxl_device_vfb_add(ctx, dm_domid, &dm_config->vfbs[0]);
+    ret = libxl__device_vfb_add(gc, dm_domid, &dm_config->vfbs[0]);
     if (ret)
         goto out;
     ret = libxl__device_vkb_add(gc, dm_domid, &dm_config->vkbs[0]);
diff -r 21ffb640c61c -r c29feacecfc4 tools/libxl/libxl_internal.h
--- a/tools/libxl/libxl_internal.h      Thu Jul 26 16:47:36 2012 +0100
+++ b/tools/libxl/libxl_internal.h      Thu Jul 26 16:47:37 2012 +0100
@@ -1953,6 +1953,10 @@ _hidden void libxl__device_nic_add(libxl
 _hidden int libxl__device_vkb_add(libxl__gc *gc, uint32_t domid,
                                   libxl_device_vkb *vkb);
 
+/* Internal function to connect a vfb device */
+_hidden int libxl__device_vfb_add(libxl__gc *gc, uint32_t domid,
+                                  libxl_device_vfb *vfb);
+
 /* Waits for the passed device to reach state XenbusStateInitWait.
  * This is not really useful by itself, but is important when executing
  * hotplug scripts, since we need to be sure the device is in the correct
diff -r 21ffb640c61c -r c29feacecfc4 tools/ocaml/libs/xl/xenlight_stubs.c
--- a/tools/ocaml/libs/xl/xenlight_stubs.c      Thu Jul 26 16:47:36 2012 +0100
+++ b/tools/ocaml/libs/xl/xenlight_stubs.c      Thu Jul 26 16:47:37 2012 +0100
@@ -369,7 +369,7 @@ value stub_xl_device_vfb_add(value info,
        device_vfb_val(&gc, &lg, &c_info, info);
 
        INIT_CTX();
-       ret = libxl_device_vfb_add(ctx, Int_val(domid), &c_info);
+       ret = libxl_device_vfb_add(ctx, Int_val(domid), &c_info, 0);
        if (ret != 0)
                failwith_xl("vfb_add", &lg);
        FREE_CTX();

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog