Re: [Xen-devel] [PATCH][RESEND] Fix xm reboot --wait

To:	Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject:	Re: [Xen-devel] [PATCH][RESEND] Fix xm reboot --wait
From:	Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date:	Thu, 31 Aug 2006 00:51:52 +0100
Delivery-date:	Wed, 30 Aug 2006 16:52:08 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxx
In-reply-to:	<BFC6CAA28BC390kanno.masaki@xxxxxxxxxxxxxx>
List-help:	<mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id:	Xen developer discussion <xen-devel.lists.xensource.com>
List-post:	<mailto:xen-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender:	xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index:	AcbMj0QBgphAlDiCEdummAAKle7CWA==
Thread-topic:	[Xen-devel] [PATCH][RESEND] Fix xm reboot --wait
User-agent:	Microsoft-Entourage/11.2.5.060620

I think it'd be neater to add a 'synchronous' flag to the shutdown RPC, than
to expose a restart counter to xm via a special new server command. Would it
be hard to delay the RPC response in xend (and either have xend return a
failure response on a timeout, or have xm wait for the RPC response on a
timeout)?

 -- Keir

On 28/8/06 2:04 pm, "Masaki Kanno" <kanno.masaki@xxxxxxxxxxxxxx> wrote:

> Hi,
> 
> Could you possibly comment about this patch?
> 
> http://lists.xensource.com/archives/html/xen-devel/2006-08/msg01125.html
> 
> Best regards,
>  Kan
> 
>> Hi,
>> 
>> This patch fixed 'xm reboot' to work as you expected with the wait
>> option. This patch adds a restart sequence counter to xenstore.
>> The completion of a VM reboot is detected by observing that
>> its restart sequence counter changes.
>> 
>> 
>> Signed-off-by: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>
>> 
>> Best regards,
>> Kan
>> 
>> 
>> -------------------------------text/plain-------------------------------
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxxxxxxxx
>> http://lists.xensource.com/xen-devel
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel