Re: [PATCH 2 of 2] xencommons: Wait for xenstored to start before setting dom0 name

[Jeremy Fitzhardinge <jeremy@goop.org>]

On 06/29/2010 03:56 PM, George Dunlap wrote:
> On one of my boxes, the xenstore-write setting dom0's name starts
> before xenstored is actually ready to handle the connection properly,
> resulting in the name set failing.  Wait for xenstored to be up and
> responding to reads before continuing, timing out after 30 seconds.
>
> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
>
> diff -r ad0f9cc72415 -r dc49f02fc463 tools/hotplug/Linux/init.d/xencommons
> --- a/tools/hotplug/Linux/init.d/xencommons	Tue Jun 29 14:56:28 2010 +0100
> +++ b/tools/hotplug/Linux/init.d/xencommons	Tue Jun 29 14:56:28 2010 +0100
> @@ -37,14 +37,32 @@
>  fi
>  
>  do_start () {
> +        local time=0
> +	local timeout=30
> +
>  	if ! `xenstore-read -s / >/dev/null 2>&1`
>  	then
>  		test -z "$XENSTORED_ROOTDIR" || XENSTORED_ROOTDIR="/var/lib/xenstored"
>  		rm -f "$XENSTORED_ROOTDIR"/tdb* &>/dev/null
>  		test -z "$XENSTORED_TRACE" || XENSTORED_ARGS=" -T /var/log/xen/xenstored-trace.log"
>  
> -		echo Starting xenstored...
> +		echo -n Starting xenstored...
>  		xenstored --pid-file=/var/run/xenstore.pid $XENSTORED_ARGS
>   

Why isn't xenstored ready by the time the main process exits?  Seems
like a bug in xenstored. Does oxenstored get this right?

    J


> +
> +		# Wait for xenstored to actually come up, timing out after 30 seconds
> +                while [ $time -lt $timeout ] && ! `xenstore-read -s / >/dev/null 2>&1` ; do
> +                    echo -n .
> +		    time=$(($time+1))
> +                    sleep 1
> +                done
> +		echo
> +
> +		# Exit if we timed out
> +		if ! [ $time -lt $timeout ] ; then
> +		    echo Could not start xenstored
> +		    exit 1
> +		fi
> +
>  		echo Setting domain 0 name...
>  		xenstore-write "/local/domain/0/name" "Domain-0"
>  	fi
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>
>