From: Justin Pettit <jpettit@nicira.com>
Date: Tue, 9 Jun 2009 22:46:50 +0000 (-0700)
Subject: Xen: Fixes for core handling configuration
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ae825abbe74bce987fdeb814f95c0ae99472b5e8;p=openvswitch

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 25aca61f..821dd237 100755
--- 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)