[Xen-changelog] [xen-unstable] xc: deal with xen/evtchn and xen/gntdev device names

[Xen patchbot-unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User Keir Fraser <keir.fraser@xxxxxxxxxx>
# Date 1275372275 -3600
# Node ID 0610f0de49fe3cf34b72ea6b01064c5c424f5737
# Parent  f41f1ae36b66ab9db88aa35ab3b161af34196ba1
xc: deal with xen/evtchn and xen/gntdev device names

This patch makes xc_linux properly deal with:
 1. discovering and creating device nodes if necessary
 2. the new form of xen/<dev> device names soon to be used by the
 kernel

This changes the logic slightly:
 - If a device node already exists with the proper name, then it uses
 it as-is, assuming it has already been correctly created.
 - If the path doesn't exist, or it exists but isn't a device node,
 and
   it has successfully found the major/minor for the device, then
   (re)create the device node.

Since this logic is identical for gntdev and evtchn, make a common
function to handle both.

Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@xxxxxxxxxx>
---
 tools/libxc/xc_linux.c |  106 +++++++++++++++++++++++++------------------------
 1 files changed, 56 insertions(+), 50 deletions(-)

diff -r f41f1ae36b66 -r 0610f0de49fe tools/libxc/xc_linux.c
--- a/tools/libxc/xc_linux.c    Tue Jun 01 06:55:23 2010 +0100
+++ b/tools/libxc/xc_linux.c    Tue Jun 01 07:04:35 2010 +0100
@@ -378,33 +378,64 @@ int xc_find_device_number(const char *na
     return makedev(major, minor);
 }
 
-#define EVTCHN_DEV_NAME  "/dev/xen/evtchn"
+#define DEVXEN "/dev/xen"
+
+static int make_dev_xen(void)
+{
+    if ( mkdir(DEVXEN, 0755) != 0 )
+    {
+        struct stat st;
+        if ( (stat(DEVXEN, &st) != 0) || !S_ISDIR(st.st_mode) )
+            return -1;
+    }
+
+    return 0;
+}
+
+static int xendev_open(const char *dev)
+{
+    int fd, devnum;
+    struct stat st;
+    char *devname, *devpath;
+
+    devname = devpath = NULL;
+    fd = -1;
+
+    if ( asprintf(&devname, "xen!%s", dev) == 0 )
+        goto fail;
+
+    if ( asprintf(&devpath, "%s/%s", DEVXEN, dev) == 0 )
+        goto fail;
+
+    devnum = xc_find_device_number(dev);
+    if ( devnum == -1 )
+        devnum = xc_find_device_number(devname);
+
+    /*
+     * If we know what the correct device is and the path doesn't exist or 
+     * isn't a device, then remove it so we can create the device.
+     */
+    if ( (devnum != -1) &&
+         ((stat(devpath, &st) != 0) || !S_ISCHR(st.st_mode)) )
+    {
+        unlink(devpath);
+        if ( (make_dev_xen() == -1) ||
+             (mknod(devpath, S_IFCHR|0600, devnum) != 0) )
+            goto fail;
+    }
+
+    fd = open(devpath, O_RDWR);
+
+ fail:
+    free(devname);
+    free(devpath);
+
+    return fd;
+}
 
 int xc_evtchn_open(void)
 {
-    struct stat st;
-    int fd;
-    int devnum;
-
-    devnum = xc_find_device_number("evtchn");
-
-    /* Make sure any existing device file links to correct device. */
-    if ( (lstat(EVTCHN_DEV_NAME, &st) != 0) || !S_ISCHR(st.st_mode) ||
-         (st.st_rdev != devnum) )
-        (void)unlink(EVTCHN_DEV_NAME);
-
- reopen:
-    if ( (fd = open(EVTCHN_DEV_NAME, O_RDWR)) == -1 )
-    {
-        if ( (errno == ENOENT) &&
-             ((mkdir("/dev/xen", 0755) == 0) || (errno == EEXIST)) &&
-             (mknod(EVTCHN_DEV_NAME, S_IFCHR|0600, devnum) == 0) )
-            goto reopen;
-
-        return -1;
-    }
-
-    return fd;
+    return xendev_open("evtchn");
 }
 
 int xc_evtchn_close(int xce_handle)
@@ -518,34 +549,9 @@ void discard_file_cache(xc_interface *xc
     errno = saved_errno;
 }
 
-#define GNTTAB_DEV_NAME "/dev/xen/gntdev"
-
 int xc_gnttab_open(xc_interface *xch)
 {
-    struct stat st;
-    int fd;
-    int devnum;
-
-    devnum = xc_find_device_number("gntdev");
-
-    /* Make sure any existing device file links to correct device. */
-    if ( (lstat(GNTTAB_DEV_NAME, &st) != 0) || !S_ISCHR(st.st_mode) ||
-         (st.st_rdev != devnum) )
-        (void)unlink(GNTTAB_DEV_NAME);
-
-reopen:
-    if ( (fd = open(GNTTAB_DEV_NAME, O_RDWR)) == -1 )
-    {
-        if ( (errno == ENOENT) &&
-             ((mkdir("/dev/xen", 0755) == 0) || (errno == EEXIST)) &&
-             (mknod(GNTTAB_DEV_NAME, S_IFCHR|0600, devnum) == 0) )
-            goto reopen;
-
-        PERROR("Could not open grant table interface");
-        return -1;
-    }
-
-    return fd;
+    return xendev_open("gntdev");
 }
 
 int xc_gnttab_close(xc_interface *xch, int xcg_handle)

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