From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53463) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuoKi-00014d-7g for qemu-devel@nongnu.org; Fri, 06 Nov 2015 16:14:05 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZuoKe-0006Kt-Rp for qemu-devel@nongnu.org; Fri, 06 Nov 2015 16:14:04 -0500 Received: from relay.parallels.com ([195.214.232.42]:44476) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZuoKe-0006Hy-KO for qemu-devel@nongnu.org; Fri, 06 Nov 2015 16:14:00 -0500 References: <1446657582-21619-1-git-send-email-den@openvz.org> <20151106155455.GS12285@stefanha-x1.localdomain> <563CCFC8.2000206@redhat.com> From: "Denis V. Lunev" Message-ID: <563D1806.40602@openvz.org> Date: Sat, 7 Nov 2015 00:13:42 +0300 MIME-Version: 1.0 In-Reply-To: <563CCFC8.2000206@redhat.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 2.5 v5 0/11] dataplane snapshot fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake , Stefan Hajnoczi Cc: Kevin Wolf , qemu-devel@nongnu.org, Stefan Hajnoczi , Juan Quintela On 11/06/2015 07:05 PM, Eric Blake wrote: > On 11/06/2015 08:54 AM, Stefan Hajnoczi wrote: >> On Wed, Nov 04, 2015 at 08:19:31PM +0300, Denis V. Lunev wrote: >>> with test >>> while /bin/true ; do >>> virsh snapshot-create rhel7 >>> sleep 10 >>> virsh snapshot-delete rhel7 --current >>> done >>> with enabled iothreads on a running VM leads to a lot of troubles: hangs, >>> asserts, errors. > That is a case of using libvirt to trigger internal snapshots... > >> The HMP monitor is legacy and also not used by modern libvirt. > ...and libvirt is forced to use HMP for internal snapshots, since we > _still_ haven't exposed internal snapshots as a QMP command. Eric, by the way, there is a user visible bug with this too :)))))) EFI based VM with pflash storage for NVRAM could not be snapshoted as libvirt configures storage as 'raw'. OK, this is a libvirt bug. The patch will be sent next week or in a week by my colleague switching storage type from 'raw' to 'qcow2'. For a QEMU this results in the following: - QEMU receives command via HMP to make a snapshot - it fails, but QEMU does not see that fact (error code is not delivered to libvirt in HMP AFAIK) - on request to switch to snapshot the commands just do nothing and from the point of libvirt the command was successful We should have these commands even in the simple rudimentary current non-ideal form even just as wrappers around HMP functions. Do you have an opinion about importance of the last issue? Should it be considered for 2.6? Den