Re: [Xen-devel] [PATCH 2/3] init scripts: xendomains correct order of options for shutdown command

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Thu, 2012-10-11 at 09:25 +0100, linux@xxxxxxxxxxxxxx wrote:
> From: Sander Eikelenboom <linux@xxxxxxxxxxxxxx>
> 
> Options for the shutdown command that are supplied behind the domain id are
> ignored. In case of the default xendomains init script this means that it will
> not wait for the domains to be actually shutdown.
> 
> Signed-off-by: Sander Eikelenboom <linux@xxxxxxxxxxxxxx>

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

and applied, thanks.

> ---
>  tools/hotplug/Linux/init.d/xendomains |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/tools/hotplug/Linux/init.d/xendomains 
> b/tools/hotplug/Linux/init.d/xendomains
> index c363208..00e5944 100644
> --- a/tools/hotplug/Linux/init.d/xendomains
> +++ b/tools/hotplug/Linux/init.d/xendomains
> @@ -438,7 +438,7 @@ stop()
>           echo -n "(shut)"
>           watchdog_xencmd shutdown &
>           WDOG_PID=$!
> -         XMR=`$CMD shutdown $id $XENDOMAINS_SHUTDOWN 2>&1 1>/dev/null`
> +         XMR=`$CMD shutdown $XENDOMAINS_SHUTDOWN $id 2>&1 1>/dev/null`
>           if test $? -ne 0; then
>               echo -e "\nAn error occurred while shutting down 
> domain:\n$XMR\n"
>               rc_failed $?



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel