Revert "libxl, hotplug/Linux: default to phy backend for raw format file"
authorIan Campbell <ian.campbell@citrix.com>
Mon, 16 Sep 2013 13:29:47 +0000 (14:29 +0100)
committerIan Campbell <ian.campbell@citrix.com>
Mon, 16 Sep 2013 13:29:47 +0000 (14:29 +0100)
This reverts commit 11a63a1660453269ddd1893c15d0a98133b33ca7. The change causes
issues with live migration.

tools/hotplug/Linux/block
tools/libxl/libxl_linux.c

index 8d2ee9d213c58c90f4ff42b96897d3138b1c20ce..da26e221dc83f7a380d894dea41225a7aedf854d 100644 (file)
@@ -206,13 +206,6 @@ and so cannot be mounted ${m2}${when}."
 
 
 t=$(xenstore_read_default "$XENBUS_PATH/type" 'MISSING')
-p=$(xenstore_read "$XENBUS_PATH/params")
-mode=$(xenstore_read "$XENBUS_PATH/mode")
-if [ -b "$p" ]; then
-    truetype="phy"
-elif [ -f "$p" ]; then
-    truetype="file"
-fi
 
 case "$command" in
   add)
@@ -224,11 +217,16 @@ case "$command" in
       exit 0
     fi
 
+    if [ -n "$t" ]
+    then
+      p=$(xenstore_read "$XENBUS_PATH/params")
+      mode=$(xenstore_read "$XENBUS_PATH/mode")
+    fi
     FRONTEND_ID=$(xenstore_read "$XENBUS_PATH/frontend-id")
     FRONTEND_UUID=$(xenstore_read_default \
             "/local/domain/$FRONTEND_ID/vm" 'unknown')
 
-    case $truetype in
+    case $t in 
       phy)
         dev=$(expand_dev $p)
 
@@ -321,7 +319,7 @@ mount it read-write in a guest domain."
     ;;
 
   remove)
-    case $truetype in
+    case $t in 
       phy)
        exit 0
        ;;
index 3cfa6dacc67e91f6e6a9bda10b4b5163eaad4100..37815eb650701ba8e5592781002ed160d94df1aa 100644 (file)
  
 int libxl__try_phy_backend(mode_t st_mode)
 {
-    if (S_ISBLK(st_mode) || S_ISREG(st_mode)) {
-        return 1;
+    if (!S_ISBLK(st_mode)) {
+        return 0;
     }
 
-    return 0;
+    return 1;
 }
 
 #define EXT_SHIFT 28