[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v10 0/4] Add subcommand compare for qemu-img
From: |
Miroslav Rezanina |
Subject: |
Re: [Qemu-devel] [PATCH v10 0/4] Add subcommand compare for qemu-img |
Date: |
Wed, 13 Feb 2013 03:08:27 -0500 (EST) |
Please ignore this serie. It's resended v10 instead of v11.
Sorry for spam.
----- Original Message -----
> From: "Miroslav Rezanina" <address@hidden>
> To: address@hidden, address@hidden
> Cc: address@hidden, address@hidden, address@hidden, address@hidden, "Miroslav
> Rezanina"
> <address@hidden>
> Sent: Wednesday, February 13, 2013 9:05:53 AM
> Subject: [PATCH v10 0/4] Add subcommand compare for qemu-img
>
> This is 10th version of patch adding compare subcommand that
> compares two images. Compare has following criteria:
> - only data part is compared
> - unallocated sectors are not read
> to be zeroed/unallocated to compare rest
> - qemu-img returns:
> - 0 if images are identical
> - 1 if images differ
> - >2 error on execution
>
> v10:
> - rebased to kwolf's block-next branch due to conflict patch (patch
> 2)
> - exit status updates (patch 3)
> - minor non-functional fixes
>
> v9:
> - Merged patch 3 and 4 (subcommmand implementation and
> documentation) of v8
> - Added basic test for qemu-img compare
> - Fixed incorrect indentation
> - add bdrv_is_allocated_above() return value check
> - Add description of exit codes into the documentation
> - minor non-functional changes
>
> v8:
> - minor grammar fixes (no behavior change)
>
> v7:
> - split patch into pieces
> - Quiet mode added for all relevant subcommands
> - check non-shared part of disk after shared one
> - minor docummentation and naming fixes
>
> v6:
> - added handling -?, -h options for compare subcommand
>
> v5 (only minor changes):
> - removed redundant comment
> - removed dead code (goto after help())
> - set final total_sectors on first assignment
>
> v4:
> - Fixed various typos
> - Added functions for empty sector check and sector-to-bytes offset
> conversion
> - Fixed command-line parameters processing
>
> v3:
> - options -f/-F are orthogonal
> - documentation updated to new syntax and behavior
> - used byte offset instead of sector number for output
>
> v2:
> - changed option for second image format to -F
> - changed handling of -f and -F [1]
> - added strict mode (-s)
> - added quiet mode (-q)
> - improved output messages [2]
> - rename variables for larger image handling
> - added man page content
>
> Signed-off-by: Miroslav Rezanina <address@hidden>
>
> Miroslav Rezanina (4):
> block: Add synchronous wrapper for bdrv_co_is_allocated_above
> qemu-img: Add "Quiet mode" option
> This patch adds new qemu-img subcommand that compares content of
> two
> disk images.
> qemu-iotests: Add qemu-img compare test
>
> block.c | 51 +++++-
> blockdev.c | 6 +-
> include/block/block.h | 5 +-
> qemu-img-cmds.hx | 34 ++--
> qemu-img.c | 441
> +++++++++++++++++++++++++++++++++++++++-----
> qemu-img.texi | 56 ++++++
> tests/qemu-iotests/048 | 77 ++++++++
> tests/qemu-iotests/048.out | 25 +++
> tests/qemu-iotests/group | 1 +
> 9 files changed, 626 insertions(+), 70 deletions(-)
> create mode 100755 tests/qemu-iotests/048
> create mode 100644 tests/qemu-iotests/048.out
>
>
--
Miroslav Rezanina
Software Engineer - Virtualization Team
- [Qemu-devel] [PATCH v10 1/4] block: Add synchronous wrapper for bdrv_co_is_allocated_above, (continued)
- [Qemu-devel] [PATCH v10 1/4] block: Add synchronous wrapper for bdrv_co_is_allocated_above, Miroslav Rezanina, 2013/02/12
- [Qemu-devel] [PATCH v10 3/4] This patch adds new qemu-img subcommand that compares content of two disk images., Miroslav Rezanina, 2013/02/12
- [Qemu-devel] [PATCH v10 2/4] qemu-img: Add "Quiet mode" option, Miroslav Rezanina, 2013/02/12
- [Qemu-devel] [PATCH v10 4/4] qemu-iotests: Add qemu-img compare test, Miroslav Rezanina, 2013/02/12
- Re: [Qemu-devel] [PATCH v10 0/4] Add subcommand compare for qemu-img, Kevin Wolf, 2013/02/12
- Re: [Qemu-devel] [PATCH v10 0/4] Add subcommand compare for qemu-img, Stefan Hajnoczi, 2013/02/12
- Re: [Qemu-devel] [PATCH v10 0/4] Add subcommand compare for qemu-img,
Miroslav Rezanina <=