From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:39308) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RyR2E-0002Iv-SA for qemu-devel@nongnu.org; Fri, 17 Feb 2012 11:51:52 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RyR2A-00033V-Mv for qemu-devel@nongnu.org; Fri, 17 Feb 2012 11:51:50 -0500 Received: from thoth.sbs.de ([192.35.17.2]:28552) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RyR2A-000338-6T for qemu-devel@nongnu.org; Fri, 17 Feb 2012 11:51:46 -0500 Message-ID: <4F3E8590.1060505@siemens.com> Date: Fri, 17 Feb 2012 17:51:28 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <4F333AAA.1070601@cn.fujitsu.com> <4F333D4B.6090300@cn.fujitsu.com> <4F3AA11C.50408@siemens.com> <4F3E1546.9090303@cn.fujitsu.com> <4F3E8130.10908@redhat.com> In-Reply-To: <4F3E8130.10908@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC][PATCH 09/16 v6] introduce a new monitor command 'dump' to dump guest's memory List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: HATAYAMA Daisuke , Dave Anderson , qemu-devel , Luiz Capitulino On 2012-02-17 17:32, Eric Blake wrote: > There are other APIs where qemu has ended up pausing the domain and not > restoring things back to running when done, and where libvirt has had to > track existing state prior to starting actions in order to manually fix > things after the fact (see libvirt's qemudDomainCoreDump as a wrapper > around migration to file, for an example). If we do things right in > this new DumpState API, we may want to decide to fix other monitor > commands to use the same mechanism (it won't offload any of the burden > from libvirt, which must still correctly interact with older qemu, but > would make life nicer for clients that can assume the saner semantics). I think there is no need for a new API. Everything you need is there: check current state, prevent transitions or invoked handlers on unexpected transitions. If other commands do not make use of this, they should probably be fixed. What command or series of commands do you have in mind? Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux