WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-changelog

[Xen-changelog] [xen-unstable] libxl: coding styles cleanup

To: xen-changelog@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-changelog] [xen-unstable] libxl: coding styles cleanup
From: Xen patchbot-unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>
Date: Wed, 30 Dec 2009 05:40:12 -0800
Delivery-date: Wed, 30 Dec 2009 05:40:52 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-changelog-request@lists.xensource.com?subject=help>
List-id: BK change log <xen-changelog.lists.xensource.com>
List-post: <mailto:xen-changelog@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=unsubscribe>
Reply-to: xen-devel@xxxxxxxxxxxxxxxxxxx
Sender: xen-changelog-bounces@xxxxxxxxxxxxxxxxxxx
# HG changeset patch
# User Keir Fraser <keir.fraser@xxxxxxxxxx>
# Date 1262176882 0
# Node ID 9417aa2c73800e32d1d025761e0294a44eb72bc2
# Parent  042f50e7353f741bf3a22adb2616f83a6bd7651f
libxl: coding styles cleanup

simplify some lines, and keep the xl style consistant with itself.
use libxl_sprintf instead of snprintf/sprintf

Signed-off-by: Vincent Hanquez <vincent.hanquez@xxxxxxxxxxxxx>
---
 tools/libxl/libxl.c          |   94 +++++++++++++++++++------------------------
 tools/libxl/libxl_device.c   |    6 +-
 tools/libxl/libxl_dom.c      |   25 ++++++++++-
 tools/libxl/libxl_exec.c     |   47 +++++++++++----------
 tools/libxl/libxl_internal.h |    2 
 5 files changed, 95 insertions(+), 79 deletions(-)

diff -r 042f50e7353f -r 9417aa2c7380 tools/libxl/libxl.c
--- a/tools/libxl/libxl.c       Wed Dec 30 12:40:44 2009 +0000
+++ b/tools/libxl/libxl.c       Wed Dec 30 12:41:22 2009 +0000
@@ -45,7 +45,7 @@ int libxl_ctx_init(struct libxl_ctx *ctx
     ctx->xch = xc_interface_open();
     ctx->xsh = xs_daemon_open();
 
-    ctx->waitpid_instead= libxl_waitpid_instead_default;
+    ctx->waitpid_instead = libxl_waitpid_instead_default;
     return 0;
 }
 
@@ -332,26 +332,6 @@ xc_dominfo_t * libxl_domain_infolist(str
     return info;
 }
 
-static int libxl_save_device_model(struct libxl_ctx *ctx, uint32_t domid, int 
fd)
-{
-    int fd2, c;
-    char buf[1024];
-    char *filename = libxl_sprintf(ctx, "/var/lib/xen/qemu-save.%d", domid);
-
-    XL_LOG(ctx, XL_LOG_DEBUG, "Saving device model state to %s", filename);
-    libxl_xs_write(ctx, XBT_NULL, libxl_sprintf(ctx, 
"/local/domain/0/device-model/%d/command", domid), "save", strlen("save"));
-    libxl_wait_for_device_model(ctx, domid, "paused", NULL, NULL);
-
-    write(fd, QEMU_SIGNATURE, strlen(QEMU_SIGNATURE));
-    fd2 = open(filename, O_RDONLY);
-    while ((c = read(fd2, buf, sizeof(buf))) != 0) {
-        write(fd, buf, c);
-    }
-    close(fd2);
-    unlink(filename);
-    return 0;
-}
-
 xc_dominfo_t *libxl_domain_info(struct libxl_ctx *ctx, uint32_t domid)
 {
     xc_dominfo_t *info;
@@ -377,6 +357,7 @@ xc_dominfo_t *libxl_domain_info(struct l
 
     return info;
 }
+
 int libxl_domain_suspend(struct libxl_ctx *ctx, libxl_domain_suspend_info 
*info,
                          uint32_t domid, int fd)
 {
@@ -384,11 +365,9 @@ int libxl_domain_suspend(struct libxl_ct
     int live = info != NULL && info->flags & XL_SUSPEND_LIVE;
     int debug = info != NULL && info->flags & XL_SUSPEND_LIVE;
 
-
     core_suspend(ctx, domid, fd, hvm, live, debug);
     if (hvm)
-        libxl_save_device_model(ctx, domid, fd);
-
+        save_device_model(ctx, domid, fd);
     return 0;
 }
 
@@ -400,11 +379,11 @@ int libxl_domain_pause(struct libxl_ctx 
 
 int libxl_domain_unpause(struct libxl_ctx *ctx, uint32_t domid)
 {
-    char path[50];
+    char *path;
     char *state;
 
     if (is_hvm(ctx, domid)) {
-        snprintf(path, sizeof(path), "/local/domain/0/device-model/%d/state", 
domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/state", 
domid);
         state = libxl_xs_read(ctx, XBT_NULL, path);
         if (state != NULL && !strcmp(state, "paused")) {
             libxl_xs_write(ctx, XBT_NULL, libxl_sprintf(ctx, 
"/local/domain/0/device-model/%d/command", domid), "continue", 
strlen("continue"));
@@ -607,7 +586,7 @@ int libxl_domain_destroy(struct libxl_ct
 int libxl_domain_destroy(struct libxl_ctx *ctx, uint32_t domid, int force)
 {
     char *dom_path, *uuid_str;
-    char vm_path[UUID_LEN_STR + 5], vss_path[UUID_LEN_STR + 6], xapi_path[20];
+    char *vm_path, *vss_path, *xapi_path;
     xen_uuid_t *uuid;
     int rc, dm_present;
 
@@ -648,17 +627,21 @@ int libxl_domain_destroy(struct libxl_ct
         XL_LOG(ctx, XL_LOG_ERROR, "libxl_destroy_devices failed for %d", 
domid);
     if (!xs_rm(ctx->xsh, XBT_NULL, dom_path))
         XL_LOG_ERRNO(ctx, XL_LOG_ERROR, "xs_rm failed for %s", dom_path);
+
     uuid_str = libxl_uuid_to_string(ctx, uuid);
-    snprintf(vm_path, sizeof(vm_path), "/vm/%s", uuid_str);
+
+    vm_path = libxl_sprintf(ctx, "/vm/%s", uuid_str);
     if (!xs_rm(ctx->xsh, XBT_NULL, vm_path))
         XL_LOG_ERRNO(ctx, XL_LOG_ERROR, "xs_rm failed for %s", vm_path);
-    snprintf(vss_path, sizeof(vss_path), "/vss/%s", uuid_str);
+
+    vss_path = libxl_sprintf(ctx, "/vss/%s", uuid_str);
     if (!xs_rm(ctx->xsh, XBT_NULL, vss_path))
         XL_LOG_ERRNO(ctx, XL_LOG_ERROR, "xs_rm failed for %s", vss_path);
-    libxl_free(ctx, uuid_str);
-    snprintf(xapi_path, sizeof(xapi_path), "/xapi/%u", domid);
+
+    xapi_path = libxl_sprintf(ctx, "/xapi/%u", domid);
     if (!xs_rm(ctx->xsh, XBT_NULL, xapi_path))
         XL_LOG_ERRNO(ctx, XL_LOG_ERROR, "xs_rm failed for %s", xapi_path);
+
     rc = xc_domain_destroy(ctx->xch, domid);
     if (rc < 0) {
         XL_LOG_ERRNOVAL(ctx, XL_LOG_ERROR, rc, "xc_domain_destroy failed for 
%d", domid);
@@ -1068,7 +1051,8 @@ int libxl_create_device_model(struct lib
 }
 
 int libxl_detach_device_model(struct libxl_ctx *ctx,
-                              libxl_device_model_starting *starting) {
+                              libxl_device_model_starting *starting)
+{
     int rc;
     rc = libxl_spawn_detach(ctx, starting->for_spawn);
     if (starting->for_spawn) libxl_free(ctx, starting->for_spawn);
@@ -1078,7 +1062,8 @@ int libxl_detach_device_model(struct lib
 
 
 int libxl_confirm_device_model_startup(struct libxl_ctx *ctx,
-                                       libxl_device_model_starting *starting) {
+                                       libxl_device_model_starting *starting)
+{
     int problem = libxl_wait_for_device_model(ctx, starting->domid, "running",
                                               libxl_spawn_check,
                                               starting->for_spawn);
@@ -1176,7 +1161,8 @@ int libxl_device_disk_add(struct libxl_c
             if (is_blktap2_supported()) {
                 int rc, c, p[2], tot;
                 char buf[1024], *dev;
-                if ((dev = get_blktap2_device(ctx, disk->physpath, 
device_disk_string_of_phystype(disk->phystype))) == NULL) {
+                dev = get_blktap2_device(ctx, disk->physpath, 
device_disk_string_of_phystype(disk->phystype));
+                if (dev == NULL) {
                     if (pipe(p) < 0) {
                         XL_LOG(ctx, XL_LOG_ERROR, "Failed to create a pipe");
                         return -1;
@@ -1959,7 +1945,7 @@ retry_transaction2:
 
 int libxl_device_pci_add(struct libxl_ctx *ctx, uint32_t domid, 
libxl_device_pci *pcidev)
 {
-    char path[50];
+    char *path;
     char *state, *vdevfn;
     int rc, hvm;
     int stubdomid = 0;
@@ -1968,7 +1954,8 @@ int libxl_device_pci_add(struct libxl_ct
 
     libxl_device_pci_flr(ctx, pcidev->domain, pcidev->bus, pcidev->dev, 
pcidev->func);
 
-    if ((stubdomid = libxl_get_stubdom_id(ctx, domid)) != 0) {
+    stubdomid = libxl_get_stubdom_id(ctx, domid);
+    if (stubdomid != 0) {
         libxl_device_pci pcidev_s = *pcidev;
         libxl_device_pci_add(ctx, stubdomid, &pcidev_s);
     }
@@ -1978,23 +1965,23 @@ int libxl_device_pci_add(struct libxl_ct
         if (libxl_wait_for_device_model(ctx, domid, "running", 0,0) < 0) {
             return -1;
         }
-        snprintf(path, sizeof(path), "/local/domain/0/device-model/%d/state", 
domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/state", 
domid);
         state = libxl_xs_read(ctx, XBT_NULL, path);
-        snprintf(path, sizeof(path), 
"/local/domain/0/device-model/%d/parameter", domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/parameter", 
domid);
         if (pcidev->vdevfn)
             libxl_xs_write(ctx, XBT_NULL, path, PCI_BDF_VDEVFN, pcidev->domain,
                            pcidev->bus, pcidev->dev, pcidev->func, 
pcidev->vdevfn);
         else
             libxl_xs_write(ctx, XBT_NULL, path, PCI_BDF, pcidev->domain,
                            pcidev->bus, pcidev->dev, pcidev->func);
-        snprintf(path, sizeof(path), 
"/local/domain/0/device-model/%d/command", domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/command", 
domid);
         xs_write(ctx->xsh, XBT_NULL, path, "pci-ins", strlen("pci-ins"));
         if (libxl_wait_for_device_model(ctx, domid, "pci-inserted", 0,0) < 0)
             XL_LOG(ctx, XL_LOG_ERROR, "Device Model didn't respond in time");
-        snprintf(path, sizeof(path), 
"/local/domain/0/device-model/%d/parameter", domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/parameter", 
domid);
         vdevfn = libxl_xs_read(ctx, XBT_NULL, path);
         sscanf(vdevfn + 2, "%x", &pcidev->vdevfn);
-        snprintf(path, sizeof(path), "/local/domain/0/device-model/%d/state", 
domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/state", 
domid);
         xs_write(ctx->xsh, XBT_NULL, path, state, strlen(state));
     } else {
         char *sysfs_path = libxl_sprintf(ctx, 
SYSFS_PCI_DEV"/"PCI_BDF"/resource", pcidev->domain,
@@ -2047,7 +2034,8 @@ int libxl_device_pci_add(struct libxl_ct
     }
 out:
     if (!libxl_is_stubdom(ctx, domid)) {
-        if ((rc = xc_assign_device(ctx->xch, domid, pcidev->value)) < 0)
+        rc = xc_assign_device(ctx->xch, domid, pcidev->value);
+        if (rc < 0)
             XL_LOG_ERRNOVAL(ctx, XL_LOG_ERROR, rc, "xc_assign_device failed");
     }
 
@@ -2057,7 +2045,7 @@ out:
 
 int libxl_device_pci_remove(struct libxl_ctx *ctx, uint32_t domid, 
libxl_device_pci *pcidev)
 {
-    char path[50];
+    char *path;
     char *state;
     int hvm, rc;
     int stubdomid = 0;
@@ -2070,18 +2058,18 @@ int libxl_device_pci_remove(struct libxl
         if (libxl_wait_for_device_model(ctx, domid, "running", 0,0) < 0) {
             return -1;
         }
-        snprintf(path, sizeof(path), "/local/domain/0/device-model/%d/state", 
domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/state", 
domid);
         state = libxl_xs_read(ctx, XBT_NULL, path);
-        snprintf(path, sizeof(path), 
"/local/domain/0/device-model/%d/parameter", domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/parameter", 
domid);
         libxl_xs_write(ctx, XBT_NULL, path, PCI_BDF, pcidev->domain,
                        pcidev->bus, pcidev->dev, pcidev->func);
-        snprintf(path, sizeof(path), 
"/local/domain/0/device-model/%d/command", domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/command", 
domid);
         xs_write(ctx->xsh, XBT_NULL, path, "pci-rem", strlen("pci-rem"));
         if (libxl_wait_for_device_model(ctx, domid, "pci-removed", 0,0) < 0) {
             XL_LOG(ctx, XL_LOG_ERROR, "Device Model didn't respond in time");
             return -1;
         }
-        snprintf(path, sizeof(path), "/local/domain/0/device-model/%d/state", 
domid);
+        path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/state", 
domid);
         xs_write(ctx->xsh, XBT_NULL, path, state, strlen(state));
     } else {
         char *sysfs_path = libxl_sprintf(ctx, 
SYSFS_PCI_DEV"/"PCI_BDF"/resource", pcidev->domain,
@@ -2137,11 +2125,13 @@ out:
     libxl_device_pci_flr(ctx, pcidev->domain, pcidev->bus, pcidev->dev, 
pcidev->func);
 
     if (!libxl_is_stubdom(ctx, domid)) {
-        if ((rc = xc_deassign_device(ctx->xch, domid, pcidev->value)) < 0)
+        rc = xc_deassign_device(ctx->xch, domid, pcidev->value);
+        if (rc < 0)
             XL_LOG_ERRNOVAL(ctx, XL_LOG_ERROR, rc, "xc_deassign_device 
failed");
     }
 
-    if ((stubdomid = libxl_get_stubdom_id(ctx, domid)) != 0) {
+    stubdomid = libxl_get_stubdom_id(ctx, domid);
+    if (stubdomid != 0) {
         libxl_device_pci pcidev_s = *pcidev;
         libxl_device_pci_remove(ctx, stubdomid, &pcidev_s);
     }
@@ -2163,7 +2153,7 @@ libxl_device_pci *libxl_device_pci_list(
         return NULL;
     }
     n = atoi(num_devs);
-    pcidevs = (libxl_device_pci *) calloc(n, sizeof(libxl_device_pci));
+    pcidevs = calloc(n, sizeof(libxl_device_pci));
     *num = n;
 
     for (i = 0; i < n; i++) {
@@ -2361,7 +2351,9 @@ int libxl_set_memory_target(struct libxl
     videoram = atoi(videoram_s);
 
     libxl_xs_write(ctx, XBT_NULL, libxl_sprintf(ctx, "%s/memory/target", 
dompath), "%lu", target_memkb);
-    if ((rc = xc_domain_setmaxmem(ctx->xch, domid, target_memkb + 
LIBXL_MAXMEM_CONSTANT)) != 0) return rc;
+    rc = xc_domain_setmaxmem(ctx->xch, domid, target_memkb + 
LIBXL_MAXMEM_CONSTANT);
+    if (rc != 0)
+        return rc;
     rc = xc_domain_memory_set_pod_target(ctx->xch, domid, (target_memkb - 
videoram) / 4, NULL, NULL, NULL);
     return rc;
 }
diff -r 042f50e7353f -r 9417aa2c7380 tools/libxl/libxl_device.c
--- a/tools/libxl/libxl_device.c        Wed Dec 30 12:40:44 2009 +0000
+++ b/tools/libxl/libxl_device.c        Wed Dec 30 12:41:22 2009 +0000
@@ -344,7 +344,7 @@ int libxl_device_pci_flr(struct libxl_ct
 int libxl_device_pci_flr(struct libxl_ctx *ctx, unsigned int domain, unsigned 
int bus,
                          unsigned int dev, unsigned int func)
 {
-    char *do_flr= "/sys/bus/pci/drivers/pciback/do_flr";
+    char *do_flr = "/sys/bus/pci/drivers/pciback/do_flr";
     FILE *fd;
 
     fd = fopen(do_flr, "w");
@@ -367,7 +367,7 @@ int libxl_wait_for_device_model(struct l
                                                       void *userdata),
                                 void *check_callback_userdata)
 {
-    char path[50];
+    char *path;
     char *p;
     unsigned int len;
     int rc;
@@ -379,7 +379,7 @@ int libxl_wait_for_device_model(struct l
     char **l = NULL;
 
     xsh = xs_daemon_open();
-    snprintf(path, sizeof(path), "/local/domain/0/device-model/%d/state", 
domid);
+    path = libxl_sprintf(ctx, "/local/domain/0/device-model/%d/state", domid);
     xs_watch(xsh, path, path);
     tv.tv_sec = LIBXL_DEVICE_MODEL_START_TIMEOUT;
     tv.tv_usec = 0;
diff -r 042f50e7353f -r 9417aa2c7380 tools/libxl/libxl_dom.c
--- a/tools/libxl/libxl_dom.c   Wed Dec 30 12:40:44 2009 +0000
+++ b/tools/libxl/libxl_dom.c   Wed Dec 30 12:41:22 2009 +0000
@@ -24,6 +24,7 @@
 #include <xenctrl.h>
 #include <xc_dom.h>
 #include <xenguest.h>
+#include <fcntl.h>
 
 #include "libxl.h"
 #include "libxl_internal.h"
@@ -130,10 +131,11 @@ int build_pv(struct libxl_ctx *ctx, uint
         XL_LOG_ERRNO(ctx, XL_LOG_ERROR, "xc_dom_allocate failed");
         return -1;
     }
-    if ((ret = xc_dom_linux_build(ctx->xch, dom, domid, info->target_memkb / 
1024,
+    ret = xc_dom_linux_build(ctx->xch, dom, domid, info->target_memkb / 1024,
                                   info->kernel, info->u.pv.ramdisk, flags,
                                   state->store_port, &state->store_mfn,
-                                  state->console_port, &state->console_mfn)) 
!= 0) {
+                                  state->console_port, &state->console_mfn);
+    if (ret != 0) {
         xc_dom_release(dom);
         XL_LOG_ERRNOVAL(ctx, XL_LOG_ERROR, ret, "xc_dom_linux_build failed");
         return -2;
@@ -317,3 +319,22 @@ int core_suspend(struct libxl_ctx *ctx, 
     return 0;
 }
 
+int save_device_model(struct libxl_ctx *ctx, uint32_t domid, int fd)
+{
+    int fd2, c;
+    char buf[1024];
+    char *filename = libxl_sprintf(ctx, "/var/lib/xen/qemu-save.%d", domid);
+
+    XL_LOG(ctx, XL_LOG_DEBUG, "Saving device model state to %s", filename);
+    libxl_xs_write(ctx, XBT_NULL, libxl_sprintf(ctx, 
"/local/domain/0/device-model/%d/command", domid), "save", strlen("save"));
+    libxl_wait_for_device_model(ctx, domid, "paused", NULL, NULL);
+
+    write(fd, QEMU_SIGNATURE, strlen(QEMU_SIGNATURE));
+    fd2 = open(filename, O_RDONLY);
+    while ((c = read(fd2, buf, sizeof(buf))) != 0) {
+        write(fd, buf, c);
+    }
+    close(fd2);
+    unlink(filename);
+    return 0;
+}
diff -r 042f50e7353f -r 9417aa2c7380 tools/libxl/libxl_exec.c
--- a/tools/libxl/libxl_exec.c  Wed Dec 30 12:40:44 2009 +0000
+++ b/tools/libxl/libxl_exec.c  Wed Dec 30 12:41:22 2009 +0000
@@ -30,7 +30,7 @@
 #include "libxl.h"
 #include "libxl_internal.h"
 
-pid_t libxl_fork(struct libxl_ctx *ctx)
+static pid_t libxl_fork(struct libxl_ctx *ctx)
 {
     pid_t pid;
 
@@ -63,12 +63,13 @@ void libxl_exec(struct libxl_ctx *ctx, i
 }
 
 void libxl_report_child_exitstatus(struct libxl_ctx *ctx,
-                                   const char *what, pid_t pid, int status) {
+                                   const char *what, pid_t pid, int status)
+{
     /* treats all exit statuses as errors; if that's not what you want,
      * check status yourself first */
 
     if (WIFEXITED(status)) {
-        int st= WEXITSTATUS(status);
+        int st = WEXITSTATUS(status);
         if (st)
             XL_LOG(ctx, XL_LOG_ERROR, "%s [%ld] exited"
                    " with error status %d", what, (unsigned long)pid, st);
@@ -76,9 +77,9 @@ void libxl_report_child_exitstatus(struc
             XL_LOG(ctx, XL_LOG_ERROR, "%s [%ld] unexpectedly"
                    " exited status zero", what, (unsigned long)pid);
     } else if (WIFSIGNALED(status)) {
-        int sig= WTERMSIG(status);
-        const char *str= strsignal(sig);
-        const char *coredump= WCOREDUMP(status) ? " (core dumped)" : "";
+        int sig = WTERMSIG(status);
+        const char *str = strsignal(sig);
+        const char *coredump = WCOREDUMP(status) ? " (core dumped)" : "";
         if (str)
             XL_LOG(ctx, XL_LOG_ERROR, "%s [%ld] died due to"
                    " fatal signal %s%s", what, (unsigned long)pid,
@@ -93,36 +94,36 @@ void libxl_report_child_exitstatus(struc
     }
 }
 
-pid_t libxl_waitpid_instead_default(pid_t pid, int *status, int flags) {
+pid_t libxl_waitpid_instead_default(pid_t pid, int *status, int flags)
+{
     return waitpid(pid,status,flags);
 }
-
-
 
 int libxl_spawn_spawn(struct libxl_ctx *ctx,
                       libxl_device_model_starting *starting,
                       const char *what,
                       void (*intermediate_hook)(struct libxl_ctx *ctx,
                                                 void *for_spawn,
-                                                pid_t innerchild)) {
+                                                pid_t innerchild))
+{
     pid_t child, got;
     int status;
     pid_t intermediate;
     struct libxl_spawn_starting *for_spawn = starting->for_spawn;
 
     if (for_spawn) {
-        for_spawn->what= libxl_sprintf(ctx, "%s", what);
+        for_spawn->what = libxl_sprintf(ctx, "%s", what);
         if (!for_spawn->what) return ERROR_NOMEM;
     }
 
     intermediate = libxl_fork(ctx);
-    if (intermediate==-1) {
+    if (intermediate ==-1) {
         if (for_spawn) free(for_spawn->what);
         return ERROR_FAIL;
     }
     if (intermediate) {
         /* parent */
-        if (for_spawn) for_spawn->intermediate= intermediate;
+        if (for_spawn) for_spawn->intermediate = intermediate;
         return 1;
     }
 
@@ -130,7 +131,7 @@ int libxl_spawn_spawn(struct libxl_ctx *
 
     child = libxl_fork(ctx);
     if (!child) return 0; /* caller runs child code */
-    if (child<0) exit(255);
+    if (child < 0) exit(255);
 
     intermediate_hook(ctx, starting, child);
 
@@ -141,17 +142,17 @@ int libxl_spawn_spawn(struct libxl_ctx *
 
     libxl_report_child_exitstatus(ctx, what, child, status);
     _exit(WIFEXITED(status) ? WEXITSTATUS(status) :
-          WIFSIGNALED(status) && WTERMSIG(status)<127
+          WIFSIGNALED(status) && WTERMSIG(status) < 127
           ? WTERMSIG(status)+128 : -1);
 }
 
 static void report_spawn_intermediate_status(struct libxl_ctx *ctx,
                                  struct libxl_spawn_starting *for_spawn,
-                                 int status) {
+                                 int status)
+{
     if (!WIFEXITED(status)) {
         /* intermediate process did the logging itself if it exited */
-        char *intermediate_what=
-            libxl_sprintf(ctx,
+        char *intermediate_what = libxl_sprintf(ctx,
                           "%s intermediate process (startup monitor)",
                           for_spawn->what);
         libxl_report_child_exitstatus(ctx, intermediate_what,
@@ -160,7 +161,8 @@ static void report_spawn_intermediate_st
 }
 
 int libxl_spawn_detach(struct libxl_ctx *ctx,
-                       struct libxl_spawn_starting *for_spawn) {
+                       struct libxl_spawn_starting *for_spawn)
+{
     int r, status;
     pid_t got;
     int rc = 0;
@@ -178,7 +180,7 @@ int libxl_spawn_detach(struct libxl_ctx 
         }
         got = ctx->waitpid_instead(for_spawn->intermediate, &status, 0);
         assert(got == for_spawn->intermediate);
-        if (!(WIFSIGNALED(status) && WTERMSIG(status)==SIGKILL)) {
+        if (!(WIFSIGNALED(status) && WTERMSIG(status) == SIGKILL)) {
             report_spawn_intermediate_status(ctx, for_spawn, status);
             rc = ERROR_FAIL;
         }
@@ -191,7 +193,8 @@ int libxl_spawn_detach(struct libxl_ctx 
     return rc;
 }
 
-int libxl_spawn_check(struct libxl_ctx *ctx, void *for_spawn_void) {
+int libxl_spawn_check(struct libxl_ctx *ctx, void *for_spawn_void)
+{
     struct libxl_spawn_starting *for_spawn = for_spawn_void;
     pid_t got;
     int status;
@@ -205,6 +208,6 @@ int libxl_spawn_check(struct libxl_ctx *
     assert(got == for_spawn->intermediate);
     report_spawn_intermediate_status(ctx, for_spawn, status);
 
-    for_spawn->intermediate= 0;
+    for_spawn->intermediate = 0;
     return ERROR_FAIL;
 }
diff -r 042f50e7353f -r 9417aa2c7380 tools/libxl/libxl_internal.h
--- a/tools/libxl/libxl_internal.h      Wed Dec 30 12:40:44 2009 +0000
+++ b/tools/libxl/libxl_internal.h      Wed Dec 30 12:41:22 2009 +0000
@@ -116,6 +116,7 @@ int restore_common(struct libxl_ctx *ctx
 int restore_common(struct libxl_ctx *ctx, uint32_t domid,
                    libxl_domain_build_info *info, libxl_domain_build_state 
*state, int fd);
 int core_suspend(struct libxl_ctx *ctx, uint32_t domid, int fd, int hvm, int 
live, int debug);
+int save_device_model(struct libxl_ctx *ctx, uint32_t domid, int fd);
 
 /* from xl_device */
 char *device_disk_backend_type_of_phystype(libxl_disk_phystype phystype);
@@ -186,7 +187,6 @@ int libxl_spawn_check(struct libxl_ctx *
 
  /* low-level stuff, for synchronous subprocesses etc. */
 
-pid_t libxl_fork(struct libxl_ctx *ctx); // logs errors
 void libxl_exec(struct libxl_ctx *ctx, int stdinfd, int stdoutfd, int stderrfd,
                 char *arg0, char **args); // logs errors, never returns
 void libxl_log_child_exitstatus(struct libxl_ctx *ctx,

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

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-changelog] [xen-unstable] libxl: coding styles cleanup, Xen patchbot-unstable <=