[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 0/4] block: Trivial fixes in offloading code
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH 0/4] block: Trivial fixes in offloading code |
Date: |
Mon, 2 Jul 2018 14:35:53 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 2018-07-02 04:58, Fam Zheng wrote:
> These are the low priority ones spotted by Kevin and Max last week.
>
> Fam Zheng (4):
> qcow2: Drop unused cluster_data
> file-posix: Fix fd_open check in raw_co_copy_range_to
> qcow2: Drop unreachable break
> raw: Drop superfluous semicolon
>
> block/file-posix.c | 2 +-
> block/qcow2.c | 3 ---
> block/raw-format.c | 2 +-
> 3 files changed, 2 insertions(+), 5 deletions(-)
Thanks, applied to my block branch:
https://git.xanclic.moe/XanClic/qemu/commits/branch/block
Do you want to make qcow2_co_copy_range_to() do something special on
BDRV_REQ_ZERO_WRITE? To me, it seems natural, but on the other hand
maybe it wouldn't bring anything. If the protocol layer supports copy
offloading, then it'll probably do that zero write efficiently anyway.
If it doesn't, qemu-img convert will just fall back to the usual
implementation which involves writing zeroes when zeroes are read, so...
What's your opinion?
Max
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH 0/4] block: Trivial fixes in offloading code, Fam Zheng, 2018/07/01
- [Qemu-block] [PATCH 1/4] qcow2: Drop unused cluster_data, Fam Zheng, 2018/07/01
- [Qemu-block] [PATCH 2/4] file-posix: Fix fd_open check in raw_co_copy_range_to, Fam Zheng, 2018/07/01
- [Qemu-block] [PATCH 3/4] qcow2: Drop unreachable break, Fam Zheng, 2018/07/01
- [Qemu-block] [PATCH 4/4] raw: Drop superfluous semicolon, Fam Zheng, 2018/07/01
- Re: [Qemu-block] [PATCH 0/4] block: Trivial fixes in offloading code,
Max Reitz <=
- Re: [Qemu-block] [PATCH 0/4] block: Trivial fixes in offloading code, Stefan Hajnoczi, 2018/07/02