[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v5 01/42] block: Mark commit and mirror as filte
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [Qemu-block] [PATCH v5 01/42] block: Mark commit and mirror as filter drivers |
Date: |
Thu, 13 Jun 2019 10:47:30 +0000 |
13.06.2019 1:09, Max Reitz wrote:
> The commit and mirror block nodes are filters, so they should be marked
> as such. (Strictly speaking, BDS.is_filter's documentation states that
> a filter's child must be bs->file. The following patch will relax this
> restriction, however.)
>
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: Alberto Garcia <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> ---
> block/commit.c | 2 ++
> block/mirror.c | 2 ++
> 2 files changed, 4 insertions(+)
>
> diff --git a/block/commit.c b/block/commit.c
> index c815def89a..f20a26fecd 100644
> --- a/block/commit.c
> +++ b/block/commit.c
> @@ -256,6 +256,8 @@ static BlockDriver bdrv_commit_top = {
> .bdrv_co_block_status = bdrv_co_block_status_from_backing,
> .bdrv_refresh_filename = bdrv_commit_top_refresh_filename,
> .bdrv_child_perm = bdrv_commit_top_child_perm,
> +
> + .is_filter = true,
> };
>
> void commit_start(const char *job_id, BlockDriverState *bs,
> diff --git a/block/mirror.c b/block/mirror.c
> index f8bdb5b21b..4fa8f57c80 100644
> --- a/block/mirror.c
> +++ b/block/mirror.c
> @@ -1480,6 +1480,8 @@ static BlockDriver bdrv_mirror_top = {
> .bdrv_co_block_status = bdrv_co_block_status_from_backing,
> .bdrv_refresh_filename = bdrv_mirror_top_refresh_filename,
> .bdrv_child_perm = bdrv_mirror_top_child_perm,
> +
> + .is_filter = true,
> };
>
> static void mirror_start_job(const char *job_id, BlockDriverState *bs,
>
--
Best regards,
Vladimir
- [Qemu-block] [PATCH v5 00/42] block: Deal with filters, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 01/42] block: Mark commit and mirror as filter drivers, Max Reitz, 2019/06/12
- Re: [Qemu-block] [PATCH v5 01/42] block: Mark commit and mirror as filter drivers,
Vladimir Sementsov-Ogievskiy <=
- [Qemu-block] [PATCH v5 02/42] copy-on-read: Support compressed writes, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 03/42] throttle: Support compressed writes, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 04/42] block: Add child access functions, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 05/42] block: Add chain helper functions, Max Reitz, 2019/06/12