[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 4/9] qcow2: Support BDRV_REQ_ZERO_WRITE for truncate
|
From: |
Alberto Garcia |
|
Subject: |
Re: [PATCH v4 4/9] qcow2: Support BDRV_REQ_ZERO_WRITE for truncate |
|
Date: |
Tue, 21 Apr 2020 12:50:16 +0200 |
|
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Tue 21 Apr 2020 10:47:17 AM CEST, Vladimir Sementsov-Ogievskiy
<address@hidden> wrote:
>> + if ((flags & BDRV_REQ_ZERO_WRITE) && offset > old_length) {
>> + ret = qcow2_cluster_zeroize(bs, old_length, offset - old_length, 0);
>
> Hmm. As I understand, qcow2_cluster_zeroize is unprepared to
> cluster-unaligned offset/size. I think we should handle it somehow.
You're right, it actually hits an assertion :-/
I suppose you can simply round the size up to the next cluster.
Berto
- Re: [PATCH v4 5/9] raw-format: Support BDRV_REQ_ZERO_WRITE for truncate, (continued)
[PATCH v4 6/9] file-posix: Support BDRV_REQ_ZERO_WRITE for truncate, Kevin Wolf, 2020/04/20
[PATCH v4 4/9] qcow2: Support BDRV_REQ_ZERO_WRITE for truncate, Kevin Wolf, 2020/04/20
[PATCH v4 8/9] iotests: Filter testfiles out in img_info_log(), Kevin Wolf, 2020/04/20
[PATCH v4 7/9] block: truncate: Don't make backing file data visible, Kevin Wolf, 2020/04/20
[PATCH v4 2/9] block: Add flags to bdrv(_co)_truncate(), Kevin Wolf, 2020/04/20
[PATCH v4 9/9] iotests: Test committing to short backing file, Kevin Wolf, 2020/04/20