From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MRMnt-0004P3-Tl
	for qemu-devel@nongnu.org; Thu, 16 Jul 2009 04:59:02 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MRMnn-0004OB-Uw
	for qemu-devel@nongnu.org; Thu, 16 Jul 2009 04:59:00 -0400
Received: from [199.232.76.173] (port=50703 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MRMnn-0004O8-QW
	for qemu-devel@nongnu.org; Thu, 16 Jul 2009 04:58:55 -0400
Received: from mx2.redhat.com ([66.187.237.31]:34314)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <amit.shah@redhat.com>) id 1MRMnn-0007Sb-9k
	for qemu-devel@nongnu.org; Thu, 16 Jul 2009 04:58:55 -0400
Date: Thu, 16 Jul 2009 14:28:43 +0530
From: Amit Shah <amit.shah@redhat.com>
Subject: Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot
Message-ID: <20090716085843.GA14243@amit-x200.redhat.com>
References: <20090707192631.GQ11590@us.ibm.com>
	<20090708075601.GJ28046@redhat.com> <4A549990.2060205@us.ibm.com>
	<20090708132623.GN26640@redhat.com> <4A54A367.3010603@us.ibm.com>
	<20090708135607.GO26640@redhat.com>
	<20090708154349.GT11590@us.ibm.com> <4A54C308.8060307@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4A54C308.8060307@redhat.com>
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Avi Kivity <avi@redhat.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>, Ryan Harper <ryanh@us.ibm.com>, qemu-devel@nongnu.org, Gleb Natapov <gleb@redhat.com>

On (Wed) Jul 08 2009 [19:02:16], Avi Kivity wrote:
> On 07/08/2009 06:43 PM, Ryan Harper wrote:
>>> That suffers from the same problem as a system_reboot command in that
>>> the guest may never start the shutdown. If that's an acceptable
>>> limitation, then we might as well implement it directly in QEMU as
>>> system_reboot so its easily available to everyone. Make 'system_reboot'
>>> take an optional timeout arg (default to 60 seconds) after which it
>>> cancels its plans.
>>>      
>>
>> This seems the most reasonable thing to me.  As already mentioned,
>> system_powerdown already is non-deterministic since the guest could
>> ignore the ACPI event.
>>    
>
> It could also send the guest to sleep or initiate the self destruct  
> sequence.  I don't think we should press the power button unless the  
> user explicitly asked us to.  This means we'll need a guest agent like 
> Xen.

This is another use-case for vmchannel (or virtio-serial) where  the user
exists within qemu.

		Amit