[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc |
Date: |
Fri, 22 Aug 2014 12:54:29 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 28.07.2014 um 10:48 hat Hu Tao geschrieben:
> ping...
>
> All the 6 patches have reviewed-by now.
Looks mostly good to me, I have only a few minor comments that wouldn't
block inclusion but could be addressed in follow-up patches.
However, you have a dependency on a patch series from Max (you use
minimal_blob_size()), which hasn't been reviewed and merged yet, so your
series is blocked on that.
If you want to get your series merged quicker, you could replace this
with a rough estimate that excludes the clusters used by refcount table
and blocks. If full preallocation isn't really full, but only
preallocation of 99.9%, that's probably good enough in practice.
Kevin
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc,
Kevin Wolf <=
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc, Richard W.M. Jones, 2014/08/22
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc, Daniel P. Berrange, 2014/08/22
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc, Kevin Wolf, 2014/08/22
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc, Richard W.M. Jones, 2014/08/22
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc, Daniel P. Berrange, 2014/08/22
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc, Kevin Wolf, 2014/08/22
- Re: [Qemu-devel] [PATCH v12 0/6] qcow2, raw: add preallocation=full and preallocation=falloc, Richard W.M. Jones, 2014/08/22