[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values |
Date: |
Mon, 10 Nov 2014 13:59:19 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 11/10/2014 06:45 AM, Max Reitz wrote:
> Refcounts may have a width of up to 64 bit, so qemu should use the same
s/bit/bits/
> width to represent refcount values internally.
>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> block/qcow2-cluster.c | 9 ++++++---
> block/qcow2-refcount.c | 37 ++++++++++++++++++++-----------------
> block/qcow2.h | 7 ++++---
> 3 files changed, 30 insertions(+), 23 deletions(-)
>
> diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c
> index df0b2c9..ab43902 100644
> --- a/block/qcow2-cluster.c
> +++ b/block/qcow2-cluster.c
> @@ -1640,7 +1640,7 @@ static int expand_zero_clusters_in_l1(BlockDriverState
> *bs, uint64_t *l1_table,
> for (i = 0; i < l1_size; i++) {
> uint64_t l2_offset = l1_table[i] & L1E_OFFSET_MASK;
> bool l2_dirty = false;
> - int l2_refcount;
> + int64_t l2_refcount;
You may want to mention in the commit message that you choose a signed
type to allow negative for errors, and therefore we really allow only up
to 63 useful bits. Or even mention that this is okay because no one
can feasibly generate an image with more than 2^63 refs to the same
cluster (there isn't that much storage or time to do such a task in our
lifetime...)
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 00/21] qcow2: Support refcount orders != 4, Max Reitz, 2014/11/10
- [Qemu-devel] [PATCH 01/21] qcow2: Add two new fields to BDRVQcowState, Max Reitz, 2014/11/10
- [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values, Max Reitz, 2014/11/10
- Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values,
Eric Blake <=
- Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values, Max Reitz, 2014/11/11
- Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values, Kevin Wolf, 2014/11/11
- Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values, Max Reitz, 2014/11/11
- Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values, Max Reitz, 2014/11/11
- Re: [Qemu-devel] [PATCH 03/21] qcow2: Use 64 bits for refcount values, Kevin Wolf, 2014/11/11
[Qemu-devel] [PATCH 02/21] qcow2: Add refcount_width to format-specific info, Max Reitz, 2014/11/10
[Qemu-devel] [PATCH 04/21] qcow2: Respect error in qcow2_alloc_bytes(), Max Reitz, 2014/11/10