Re: [Qemu-devel] [PATCH V4 0/5] export internal snapshot by qemu-nbd

[Wenchao Xia <xiawenc@linux.vnet.ibm.com>]

于 2013/10/11 10:33, Wenchao Xia 写道:
> This series allow user to read internal snapshot's contents without qemu-img
> convert.
> 
> V2:
>    Address Stefan's comments:
>    02: add 'fall through' comments in the case statement.
>    03: add doc about the difference of internal snapshot and backing chain
> snapshot, which is used in previous '--snapshot' parameter.
>    Other:
>    01,04: rebased on upstream with conflict resolved.
> 
> v3:
>    Address Paolo's comments:
>    02: add parameter "-l snapshot_id_or_name", rename options
> snapshot-load to load-snapshot, use QemuOpts.
>    03: rename snapshot-load to load-snapshot.
>    04: related change to test both -l and -L case.
>    05-07: add similar parameter for qemu-img convert.
>    Other:
>    01: foldered original snapshot logic into function
> bdrv_snapshot_load_tmp_by_id_or_name(), since multiple
> caller present in this version. Refined error message from
> ", reason: %s" to ": %s".
>    02: Refined error message from ", reason: %s" to ": %s".
>    03: Rename PARAM to SNAPSHOT_PARAM.
> 
> v4:
>    Address Eric's comments:
>    01: typo fix.
>    02: squashed patch. Use only one option -l and distiguish
> the cases by the starting string.
>    03: remove 'eval', remove '_supported_os Linux', remove the
> comments that have typo.
>    04: squashed patch. Add only one option -l and distiguish
> the cases by the starting string.
>    05: remove indentation.
>    Address Paolo's comments:
>    02: Use only one option -l and distiguish the cases by
> the starting string.
>    04: Use only one option -l and distiguish the cases by
> the starting string, mark old -s as deprecated, added doc
> for them.
> 
> Wenchao Xia (5):
>    1 snapshot: distinguish id and name in load_tmp
>    2 qemu-nbd: support internal snapshot export
>    3 qemu-iotests: add 058 internal snapshot export with qemu-nbd case
>    4 qemu-img: add -l for snapshot in convert
>    5 qemu-iotests: add test for snapshot in qemu-img convert
> 
>   block/qcow2-snapshot.c     |   16 +++++-
>   block/qcow2.h              |    5 ++-
>   block/snapshot.c           |   78 +++++++++++++++++++++++++++-
>   include/block/block_int.h  |    4 +-
>   include/block/snapshot.h   |   15 +++++-
>   qemu-img-cmds.hx           |    2 +-
>   qemu-img.c                 |   45 ++++++++++++++---
>   qemu-img.texi              |   12 +++--
>   qemu-nbd.c                 |   47 +++++++++++++++++-
>   qemu-nbd.texi              |    8 +++-
>   tests/qemu-iotests/058     |  119 ++++++++++++++++++++++++++++++++++++++++++++
>   tests/qemu-iotests/058.out |   44 ++++++++++++++++
>   tests/qemu-iotests/check   |    1 +
>   tests/qemu-iotests/group   |    1 +
>   14 files changed, 374 insertions(+), 23 deletions(-)
>   create mode 100755 tests/qemu-iotests/058
>   create mode 100644 tests/qemu-iotests/058.out
> 
 ping?