From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1L9hWe-0001LF-LM for qemu-devel@nongnu.org; Mon, 08 Dec 2008 09:55:56 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1L9hWc-0001Ks-8T for qemu-devel@nongnu.org; Mon, 08 Dec 2008 09:55:56 -0500 Received: from [199.232.76.173] (port=58605 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1L9hWc-0001Kp-2b for qemu-devel@nongnu.org; Mon, 08 Dec 2008 09:55:54 -0500 Received: from ns.suse.de ([195.135.220.2]:37477 helo=mx1.suse.de) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1L9hWb-0005PH-G2 for qemu-devel@nongnu.org; Mon, 08 Dec 2008 09:55:53 -0500 Received: from Relay2.suse.de (mail2.suse.de [195.135.221.8]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.suse.de (Postfix) with ESMTP id 3302442E9E for ; Mon, 8 Dec 2008 15:55:50 +0100 (CET) Message-ID: <493D3678.1040406@suse.de> Date: Mon, 08 Dec 2008 16:00:08 +0100 From: Kevin Wolf MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="------------080101010106060301060909" Subject: [Qemu-devel] [PATCH] Fix handling of disk-only snapshots Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org This is a multi-part message in MIME format. --------------080101010106060301060909 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit When creating a snapshot with multiple qcow2 disks attached, the current behaviour is that qemu creates a disk snapshot on all of them and chooses one to write the VM state to. Despite having the state only in one image, loadvm tries to restore the VM state from the middle of nowhere if you run qemu a second time with only one of the other images attached. In the lucky case it will fail because there simply is no state, but it also can happen that it loads the state of a different snapshot (the one this new one is based upon). The fix is to write a zero VM state size to the images which don't contain the state, and check this in loadvm. I agree that you probably have to provoke such things intentionally to get in a state like this with qemu itself. However, with my second patch that adds snapshot support to qemu-img it could become a reasonable use case to have snapshots with and without VM states on the same image. Signed-off-by: Kevin Wolf --------------080101010106060301060909 Content-Type: text/x-patch; name="snapshot-without-vmstate.patch" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="snapshot-without-vmstate.patch" Index: qemu-svn/savevm.c =================================================================== --- qemu-svn.orig/savevm.c +++ qemu-svn/savevm.c @@ -1020,6 +1020,7 @@ void do_savevm(const char *name) BlockDriverInfo bdi1, *bdi = &bdi1; QEMUFile *f; int saved_vm_running; + uint32_t vm_state_size; #ifdef _WIN32 struct _timeb tb; #else @@ -1079,7 +1080,7 @@ void do_savevm(const char *name) goto the_end; } ret = qemu_savevm_state(f); - sn->vm_state_size = qemu_ftell(f); + vm_state_size = qemu_ftell(f); qemu_fclose(f); if (ret < 0) { term_printf("Error %d while writing VM\n", ret); @@ -1098,6 +1099,8 @@ void do_savevm(const char *name) bdrv_get_device_name(bs1)); } } + /* Write VM state size only to the image that contains the state */ + sn->vm_state_size = (bs == bs1 ? vm_state_size : 0); ret = bdrv_snapshot_create(bs1, sn); if (ret < 0) { term_printf("Error while creating snapshot on '%s'\n", @@ -1115,6 +1118,7 @@ void do_loadvm(const char *name) { BlockDriverState *bs, *bs1; BlockDriverInfo bdi1, *bdi = &bdi1; + QEMUSnapshotInfo sn; QEMUFile *f; int i, ret; int saved_vm_running; @@ -1165,6 +1169,11 @@ void do_loadvm(const char *name) return; } + /* Don't even try to load empty VM states */ + ret = bdrv_snapshot_find(bs, &sn, name); + if ((ret >= 0) && (sn.vm_state_size == 0)) + goto the_end; + /* restore the VM state */ f = qemu_fopen_bdrv(bs, bdi->vm_state_offset, 0); if (!f) { --------------080101010106060301060909--