[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/5] qcow2: add fragmentation and compression
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/5] qcow2: add fragmentation and compression info support |
Date: |
Tue, 12 Feb 2013 14:38:50 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Feb 07, 2013 at 05:15:00PM +0100, Stefan Hajnoczi wrote:
> Applies on kevin/block-next (2700d627e4d73bb2aacb832a93ffac8fec78b1fd).
>
> This series adds fragmentation info support to qcow2 and then adds compression
> info. Previously only QED supported fragmentation info. Note that
> fragmentation info only covers the current image, internal snapshots are not
> included.
>
> The new BlockFragInfo.compressed field contains the number of compressed
> clusters. qemu-img check now reports the percentage of compressed clusters
> out
> of all allocated clusters. This can be used to decide when to recompress an
> image or to double-check that an image is indeed compressed.
>
> qemu-iotests was to handle the new qemu-img check output.
>
> I have tested qcow2 images both compressed and uncompressed.
>
> v3:
> * Add space before '=' in qemu-img check output [kwolf]
> * Count compressed clusters as fragmented, explained in comment [kwolf]
>
> v2:
> * Fix indentation on modified block/qcow2-refcount.c lines [eblake]
> * Change qemu-img check output from just "compressed" to "compressed
> clusters" [eblake]
> * Improve grep and sed usage in common.rc [eblake]
> * Introduce sector_offset local variable for & ~511 [eblake]
> * Improve nb_csectors rounding down explanation in comment [eblake]
>
> Stefan Hajnoczi (5):
> qcow2: introduce check_refcounts_l1/l2() flags
> qcow2: record fragmentation statistics during check
> qemu-img: fix missing space in qemu-img check output
> qemu-img: add compressed clusters to BlockFragInfo
> qcow2: support compressed clusters in BlockFragInfo
>
> block/qcow2-refcount.c | 42 ++++++++++++++++++++++++++++++++++++------
> include/block/block.h | 1 +
> qapi-schema.json | 6 +++++-
> qemu-img.c | 12 ++++++++----
> tests/qemu-iotests/044.out | 1 +
> tests/qemu-iotests/common.rc | 2 +-
> 6 files changed, 52 insertions(+), 12 deletions(-)
>
> --
> 1.8.1
>
Applied to my block-next tree:
https://github.com/stefanha/qemu/commits/block-next
Stefan
- [Qemu-devel] [PATCH v3 0/5] qcow2: add fragmentation and compression info support, Stefan Hajnoczi, 2013/02/07
- [Qemu-devel] [PATCH v3 5/5] qcow2: support compressed clusters in BlockFragInfo, Stefan Hajnoczi, 2013/02/07
- [Qemu-devel] [PATCH v3 3/5] qemu-img: fix missing space in qemu-img check output, Stefan Hajnoczi, 2013/02/07
- [Qemu-devel] [PATCH v3 1/5] qcow2: introduce check_refcounts_l1/l2() flags, Stefan Hajnoczi, 2013/02/07
- [Qemu-devel] [PATCH v3 4/5] qemu-img: add compressed clusters to BlockFragInfo, Stefan Hajnoczi, 2013/02/07
- [Qemu-devel] [PATCH v3 2/5] qcow2: record fragmentation statistics during check, Stefan Hajnoczi, 2013/02/07
- Re: [Qemu-devel] [PATCH v3 0/5] qcow2: add fragmentation and compression info support, Eric Blake, 2013/02/07
- Re: [Qemu-devel] [PATCH v3 0/5] qcow2: add fragmentation and compression info support, Kevin Wolf, 2013/02/12
- Re: [Qemu-devel] [PATCH v3 0/5] qcow2: add fragmentation and compression info support,
Stefan Hajnoczi <=