[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v11 07/10] file-posix: support BDRV_REQ_ALLOCATE
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [Qemu-devel] [PATCH v11 07/10] file-posix: support BDRV_REQ_ALLOCATE |
Date: |
Fri, 21 Dec 2018 14:18:55 +0000 |
18.12.2018 10:57, Anton Nefedov wrote:
> Current write_zeroes implementation is good enough to satisfy this flag too
>
> Signed-off-by: Anton Nefedov <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> ---
> block/file-posix.c | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/block/file-posix.c b/block/file-posix.c
> index a65e464cbc..c3fbf53853 100644
> --- a/block/file-posix.c
> +++ b/block/file-posix.c
> @@ -607,6 +607,7 @@ static int raw_open_common(BlockDriverState *bs, QDict
> *options,
> } else {
> s->discard_zeroes = true;
> s->has_fallocate = true;
> + bs->supported_zero_flags = BDRV_REQ_ALLOCATE;
> }
> } else {
> if (!(S_ISCHR(st.st_mode) || S_ISBLK(st.st_mode))) {
> @@ -650,10 +651,11 @@ static int raw_open_common(BlockDriverState *bs, QDict
> *options,
> #ifdef CONFIG_XFS
> if (platform_test_xfs_fd(s->fd)) {
> s->is_xfs = true;
> + bs->supported_zero_flags = BDRV_REQ_ALLOCATE;
> }
> #endif
>
> - bs->supported_zero_flags = BDRV_REQ_MAY_UNMAP;
> + bs->supported_zero_flags |= BDRV_REQ_MAY_UNMAP;
> ret = 0;
> fail:
> if (filename && (bdrv_flags & BDRV_O_TEMPORARY)) {
> @@ -1552,6 +1554,10 @@ static int handle_aiocb_write_zeroes(void *opaque)
> s->has_fallocate = false;
> #endif
>
> + if (!s->has_fallocate) {
> + aiocb->bs->supported_zero_flags &= ~BDRV_REQ_ALLOCATE;
> + }
> +
> return -ENOTSUP;
> }
>
>
--
Best regards,
Vladimir
- [Qemu-devel] [PATCH v11 02/10] blkverify: set supported write/zero flags, (continued)
- [Qemu-devel] [PATCH v11 02/10] blkverify: set supported write/zero flags, Anton Nefedov, 2018/12/18
- [Qemu-devel] [PATCH v11 03/10] quorum: set supported write flags, Anton Nefedov, 2018/12/18
- [Qemu-devel] [PATCH v11 06/10] file-posix: reset fallocate-related flags without CONFIG_FALLOCATE*, Anton Nefedov, 2018/12/18
- [Qemu-devel] [PATCH v11 05/10] block: treat BDRV_REQ_ALLOCATE as serialising, Anton Nefedov, 2018/12/18
- [Qemu-devel] [PATCH v11 04/10] block: introduce BDRV_REQ_ALLOCATE flag, Anton Nefedov, 2018/12/18
- [Qemu-devel] [PATCH v11 08/10] block: support BDRV_REQ_ALLOCATE in passthrough drivers, Anton Nefedov, 2018/12/18
- [Qemu-devel] [PATCH v11 09/10] qcow2: skip writing zero buffers to empty COW areas, Anton Nefedov, 2018/12/18
- [Qemu-devel] [PATCH v11 07/10] file-posix: support BDRV_REQ_ALLOCATE, Anton Nefedov, 2018/12/18
- Re: [Qemu-devel] [PATCH v11 07/10] file-posix: support BDRV_REQ_ALLOCATE,
Vladimir Sementsov-Ogievskiy <=
- [Qemu-devel] [PATCH v11 10/10] iotest 134: test cluster-misaligned encrypted write, Anton Nefedov, 2018/12/18