From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:34273) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UU6yd-0006Bd-2s for qemu-devel@nongnu.org; Sun, 21 Apr 2013 22:59:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UU6yb-0004v1-Tw for qemu-devel@nongnu.org; Sun, 21 Apr 2013 22:59:35 -0400 Received: from e28smtp09.in.ibm.com ([122.248.162.9]:43983) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UU6yb-0004nr-B9 for qemu-devel@nongnu.org; Sun, 21 Apr 2013 22:59:33 -0400 Received: from /spool/local by e28smtp09.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 22 Apr 2013 08:26:02 +0530 Received: from d28relay05.in.ibm.com (d28relay05.in.ibm.com [9.184.220.62]) by d28dlp02.in.ibm.com (Postfix) with ESMTP id 275053940058 for ; Mon, 22 Apr 2013 08:29:14 +0530 (IST) Received: from d28av03.in.ibm.com (d28av03.in.ibm.com [9.184.220.65]) by d28relay05.in.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r3M2x9jZ11993524 for ; Mon, 22 Apr 2013 08:29:10 +0530 Received: from d28av03.in.ibm.com (loopback [127.0.0.1]) by d28av03.in.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id r3M2xCuC018206 for ; Mon, 22 Apr 2013 12:59:12 +1000 Message-ID: <5174A748.3060205@linux.vnet.ibm.com> Date: Mon, 22 Apr 2013 10:58:16 +0800 From: Wenchao Xia MIME-Version: 1.0 References: <1365843407-16504-1-git-send-email-xiawenc@linux.vnet.ibm.com> In-Reply-To: <1365843407-16504-1-git-send-email-xiawenc@linux.vnet.ibm.com> Content-Type: text/plain; charset=GB2312 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V12 00/18] qmp/hmp interfaces for internal snapshot info List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Wenchao Xia Cc: kwolf@redhat.com, phrdina@redhat.com, stefanha@gmail.com, qemu-devel@nongnu.org, armbru@redhat.com, pbonzini@redhat.com, lcapitulino@redhat.com Hi, Any other comments for it, especially HMP part? > > V12: > Address Markus's comments: > 02/18: better incode comments for bdrv_snapshot_find(), add tip about the > logic change in snapshot create/delete/load/info if some snapshot's id is mixed > with name in commit message. > 05/18: better incode comments for bdrv_query_snapshot_info_list(), do not > check return value in caller qemu-img. > 07/18: new patch changing the vm snapshot filter logic, to be exactly the > same with load_vmstate(). To tip it clearly this patch is made a seperate one, > and can be droped if original logic want to be kepted. > 08/18: squash -ENOMEDIUM and -ENOTSUP in switch statement in > bdrv_query_image_info(). > 11/18: better incode comments for bdrv_query_image_info(). > 12/18: change disk name suffix from .img to .qcow2 in example, to tip better. > 14/18: new funtion message_printf(), which automatically dump to monitor if > it present, discard buffer or GString. > 15/18: use message_printf() instead of buffer. > 17/18: use message_printf() instead of buffer. > > Address Eric's comments: > 05/18: better incode comments for bdrv_query_snapshot_info_list(), do not > check return value in caller qemu-img. > > Address Stefan's comments: > 10/18: added doc about the meaning of "consistent snapshot" in > qmp-commands.hx. > > Address Kevin's comments: > 12/18: use local_error to detect error in qmp_query_snapshots(), in case > caller set errp = NULL. > > Wenchao Xia (18): > 1 block: move bdrv_snapshot_find() to block/snapshot.c > 2 block: distinguish id and name in bdrv_find_snapshot() > 3 qemu-img: remove unused parameter in collect_image_info() > 4 block: move collect_snapshots() and collect_image_info() to block/qapi.c > 5 block: add snapshot info query function bdrv_query_snapshot_info_list() > 6 block: add check for VM snapshot in bdrv_query_snapshot_info_list() > 7 block: change VM snapshot checking logic > 8 block: add image info query function bdrv_query_image_info() > 9 block: move qmp_query_block() and bdrv_query_info() to block/qapi.c > 10 qmp: add interface query-snapshots > 11 qmp: add recursive member in ImageInfo > 12 qmp: add ImageInfo in BlockDeviceInfo used by query-block > 13 block: move bdrv_snapshot_dump() and dump_human_image_info() to block/qapi.c > 14 block: dump to monitor for bdrv_snapshot_dump() and bdrv_image_info_dump() > 15 hmp: add function hmp_info_snapshots() > 16 hmp: switch snapshot info function to qmp based one > 17 hmp: show ImageInfo in 'info block' > 18 hmp: add parameters device and -v for info block > > block.c | 109 ----------- > block/Makefile.objs | 1 + > block/qapi.c | 452 +++++++++++++++++++++++++++++++++++++++++++ > block/snapshot.c | 78 ++++++++ > hmp.c | 63 ++++++ > hmp.h | 1 + > include/block/block.h | 2 - > include/block/qapi.h | 43 ++++ > include/block/snapshot.h | 37 ++++ > include/qemu/error-report.h | 1 + > monitor.c | 9 +- > qapi-schema.json | 24 ++- > qemu-img.c | 165 +--------------- > qmp-commands.hx | 127 ++++++++++++- > savevm.c | 96 +--------- > util/qemu-error.c | 18 ++ > 16 files changed, 861 insertions(+), 365 deletions(-) > create mode 100644 block/qapi.c > create mode 100644 block/snapshot.c > create mode 100644 include/block/qapi.h > create mode 100644 include/block/snapshot.h > > -- Best Regards Wenchao Xia