From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:47205) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TjAu7-00012S-Rf for qemu-devel@nongnu.org; Thu, 13 Dec 2012 10:41:00 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TjAu6-0005qc-La for qemu-devel@nongnu.org; Thu, 13 Dec 2012 10:40:55 -0500 Received: from mx1.redhat.com ([209.132.183.28]:2702) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TjAu6-0005qW-E0 for qemu-devel@nongnu.org; Thu, 13 Dec 2012 10:40:54 -0500 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id qBDFerf8023152 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 13 Dec 2012 10:40:53 -0500 From: Pavel Hrdina Date: Thu, 13 Dec 2012 16:40:34 +0100 Message-Id: Subject: [Qemu-devel] [PATCH v2 00/17] qapi: Convert savevm, loadvm, delvm and info snapshots List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: phrdina@redhat.com This patch series convert these commands into qapi and introduce QMP commands vm-snapshot-save, vm-snapshot-load, vm-snapshot-delete and query-vm-snapshots. It also rewrite error report for functions used by these commands. Last patch introduce new functionality of savevm and vm-snapshot-save that you cannot override existing snapshot without using 'force' parameter and for QMP you have to always provide name parameter. We should also merge functionality of migrations and vm-snapshots and make some clean-ups of this code. I could do it as another patch. With this rewrite we could make vm-snapshots asynchronous. Pavel Hrdina (17): error: introduce handle_error block: add error parameter to bdrv_snapshot_create() and related functions block: add error parameter to bdrv_snapshot_goto() and related functions block: add error parameter to bdrv_snapshot_delete() and related functions block: add error parameter to bdrv_snapshot_list() and related functions block: add error parameter to bdrv_snapshot_find() block: add error parameter to del_existing_snapshots() savevm: add error parameter to qemu_savevm_state_begin() savevm: add error parameter to qemu_savevm_state_iterate() savevm: add error parameter to qemu_savevm_state_complete() savevm: add error parameter to qemu_savevm_state() savevm: add error parameter to qemu_loadvm_state() qapi: Convert savevm qapi: Convert loadvm qapi: Convert delvm qapi: Convert info snapshots vm-snapshot-save: add force parameter block.c | 106 ++++++++++++------ block.h | 13 ++- block/qcow2-snapshot.c | 41 ++++++- block/qcow2.h | 16 ++- block/rbd.c | 33 ++++-- block/sheepdog.c | 45 +++++--- block_int.h | 13 ++- error.c | 8 ++ error.h | 6 + hmp-commands.hx | 24 ++-- hmp.c | 84 ++++++++++++++ hmp.h | 4 + migration.c | 14 +-- monitor.c | 14 +-- qapi-schema.json | 64 +++++++++++ qemu-img.c | 28 ++--- qmp-commands.hx | 115 +++++++++++++++++++ savevm.c | 297 +++++++++++++++++++++++++------------------------ sysemu.h | 17 ++- vl.c | 6 +- 20 files changed, 663 insertions(+), 285 deletions(-) -- 1.8.0.2