init/sysconfig scripts: Remove --halt/-H option for shutdown command.
authorSander Eikelenboom <linux@eikelenboom.it>
Thu, 11 Oct 2012 10:52:10 +0000 (11:52 +0100)
committerSander Eikelenboom <linux@eikelenboom.it>
Thu, 11 Oct 2012 10:52:10 +0000 (11:52 +0100)
The --halt/-H option for the shutdown command is now pointless,
since linux in a guest treats "halt" and "poweroff" identically.
The option is not implemented in xl / libxl and if supplied causes the command
to fail , so remove it from the init and sysconfig scripts.

Signed-off-by: Sander Eikelenboom <linux@eikelenboom.it>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
Committed-by: Ian Campbell <ian.campbell@citrix.com>
tools/hotplug/Linux/init.d/sysconfig.xendomains
tools/hotplug/Linux/init.d/xendomains
tools/hotplug/NetBSD/rc.d/xendomains

index e590d3fd06e714514c7b652aae32c6b2fc51a061..4775277611a4e90624e15f062a483d61568af1a9 100644 (file)
@@ -56,29 +56,29 @@ XENDOMAINS_MIGRATE=""
 XENDOMAINS_SAVE=/var/lib/xen/save
 
 ## Type: string
-## Default: "--halt --wait"
+## Default: "--wait"
 #
 # If neither MIGRATE nor SAVE were enabled or if they failed, you can
 # try to shut down a domain by sending it a shutdown request. To do this,
-# set this to "--halt --wait". Omit the "--wait" flag to avoid waiting
+# set this to "--wait". Omit the "--wait" flag to avoid waiting
 # for the domain to be really down. Leave empty to skip domain shutdown.
 #
-XENDOMAINS_SHUTDOWN="--halt --wait"
+XENDOMAINS_SHUTDOWN="--wait"
 
 ## Type: string
-## Default: "--all --halt --wait"
+## Default: "--all --wait"
 #
 # After we have gone over all virtual machines (resp. all automatically
 # started ones, see XENDOMAINS_AUTO_ONLY below) in a loop and sent SysRq,
 # migrated, saved and/or shutdown according to the settings above, we
 # might want to shutdown the virtual machines that are still running
 # for some reason or another. To do this, set this variable to
-# "--all --halt --wait", it will be passed to xm shutdown.
+# "--all --wait", it will be passed to xm shutdown.
 # Leave it empty not to do anything special here.
 # (Note: This will hit all virtual machines, even if XENDOMAINS_AUTO_ONLY
 # is set.)
 # 
-XENDOMAINS_SHUTDOWN_ALL="--all --halt --wait"
+XENDOMAINS_SHUTDOWN_ALL="--all --wait"
 
 ## Type: boolean
 ## Default: true
index 90949ed43e45f112b95dec0bebe1576055c808c9..00e5944dfdbe925e0f16305fbed0fb1e1d9294f6 100644 (file)
@@ -434,7 +434,7 @@ stop()
            fi
        fi
        if test -n "$XENDOMAINS_SHUTDOWN"; then
-           # XENDOMAINS_SHUTDOWN should be "--halt --wait"
+           # XENDOMAINS_SHUTDOWN should be "--wait"
            echo -n "(shut)"
            watchdog_xencmd shutdown &
            WDOG_PID=$!
@@ -453,7 +453,7 @@ stop()
     # This is because it's easier to do ;-) but arguably if this script is run
     # on system shutdown then it's also the right thing to do.
     if ! all_zombies && test -n "$XENDOMAINS_SHUTDOWN_ALL"; then
-       # XENDOMAINS_SHUTDOWN_ALL should be "--all --halt --wait"
+       # XENDOMAINS_SHUTDOWN_ALL should be "--all --wait"
        echo -n " SHUTDOWN_ALL "
        watchdog_xencmd shutdown 1 false &
        WDOG_PID=$!
index c368c582299fe29106cd6d031beacc919b6d7182..3e62038c249fad08451cad100eeb6e0e97b4b199 100644 (file)
@@ -94,7 +94,7 @@ xendomains_stop()
        #
        echo "Stopping xen domains."
        for domain in $(xendomains_list); do
-               ${ctl_command} shutdown --halt $domain
+               ${ctl_command} shutdown $domain
        done
        while [ $timeout -gt 0 ]; do
                livedomains=$(xendomains_list)