Xen: Fixes for core handling configuration

An improper string comparison operator was used to check whether
FORCE_COREFILES was enabled.  Further, the check to enable core files
was only down when vswitch was started, and not when restarted.

Thanks to Ben for help debugging the issue.

diff --git a/xenserver/etc_init.d_vswitch b/xenserver/etc_init.d_vswitch
index 25aca61f9e0347cf269699eba530b19f17ddf5b6..821dd2377624574e3283baafd3aca46af5b01ffc 100755 (executable)
--- a/xenserver/etc_init.d_vswitch
+++ b/xenserver/etc_init.d_vswitch
@@ -242,6 +242,10 @@ EOF
 }
 
 function start {
+    if [ "$FORCE_COREFILES" = "y" ]; then
+        turn_on_corefiles
+    fi
+
     insert_modules_if_required
     start_vswitchd
     start_brcompatd
@@ -262,9 +266,6 @@ function restart {
 
 case "$1" in
     start)
-        if [ "$FORCE_COREFILES" == "y" ]; then
-            turn_on_corefiles
-        fi
         start
         ;;
     stop)