[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v4 13/15] block: Stop passing flags to bdrv_reop
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v4 13/15] block: Stop passing flags to bdrv_reopen_queue_child() |
Date: |
Sun, 11 Nov 2018 21:47:06 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.0 |
On 07.11.18 13:59, Alberto Garcia wrote:
> Now that all callers are passing the new options using the QDict we no
> longer need the 'flags' parameter.
>
> This patch makes the following changes:
>
> 1) The update_options_from_flags() call is no longer necessary
> so it can be removed.
>
> 2) The update_flags_from_options() call is now used in all cases,
> and is moved down a few lines so it happens after the options
> QDict contains the final set of values.
>
> 3) The flags parameter is removed. Now the flags are initialized
> using the current value (for the top-level node) or the parent
> flags (after inherit_options()). In both cases the initial
> values are updated to reflect the new options in the QDict. This
> happens in bdrv_reopen_queue_child() (as explained above) and in
> bdrv_reopen_prepare().
>
> Signed-off-by: Alberto Garcia <address@hidden>
> ---
> block.c | 48 +++++++++++++++++++-----------------------------
> 1 file changed, 19 insertions(+), 29 deletions(-)
Reviewed-by: Max Reitz <address@hidden>
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-block] [PATCH v4 11/15] block: Clean up reopen_backing_file() in block/replication.c, (continued)
- [Qemu-block] [PATCH v4 04/15] block: Use bdrv_reopen_set_read_only() in bdrv_commit(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 09/15] block: Drop bdrv_reopen(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 06/15] block: Use bdrv_reopen_set_read_only() in qmp_change_backing_file(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 15/15] block: Assert that flags are up-to-date in bdrv_reopen_prepare(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 02/15] block: Use bdrv_reopen_set_read_only() in bdrv_backing_update_filename(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 05/15] block: Use bdrv_reopen_set_read_only() in stream_start/complete(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 12/15] block: Remove flags parameter from bdrv_reopen_queue(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 13/15] block: Stop passing flags to bdrv_reopen_queue_child(), Alberto Garcia, 2018/11/07
- Re: [Qemu-block] [PATCH v4 13/15] block: Stop passing flags to bdrv_reopen_queue_child(),
Max Reitz <=
- [Qemu-block] [PATCH v4 03/15] block: Use bdrv_reopen_set_read_only() in commit_start/complete(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 07/15] block: Use bdrv_reopen_set_read_only() in external_snapshot_commit(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 01/15] block: Add bdrv_reopen_set_read_only(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 10/15] qemu-io: Put flag changes in the options QDict in reopen_f(), Alberto Garcia, 2018/11/07
- [Qemu-block] [PATCH v4 14/15] block: Remove assertions from update_flags_from_options(), Alberto Garcia, 2018/11/07