[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 20/24] blockdev: Keep track of monitor-owned
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH v7 20/24] blockdev: Keep track of monitor-owned BDS |
Date: |
Tue, 10 Nov 2015 02:25:45 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 09.11.2015 23:39, Max Reitz wrote:
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: Kevin Wolf <address@hidden>
> ---
> blockdev.c | 25 +++++++++++++++++++++++++
> include/block/block_int.h | 4 ++++
> stubs/Makefile.objs | 1 +
> stubs/blockdev-close-all-bdrv-states.c | 5 +++++
> 4 files changed, 35 insertions(+)
> create mode 100644 stubs/blockdev-close-all-bdrv-states.c
>
> diff --git a/blockdev.c b/blockdev.c
> index a70a684..8211eae 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -50,6 +50,9 @@
> #include "trace.h"
> #include "sysemu/arch_init.h"
>
> +static QTAILQ_HEAD(, BlockDriverState) monitor_bdrv_states =
> + QTAILQ_HEAD_INITIALIZER(monitor_bdrv_states);
> +
> static const char *const if_name[IF_COUNT] = {
> [IF_NONE] = "none",
> [IF_IDE] = "ide",
> @@ -662,6 +665,19 @@ fail:
> return NULL;
> }
>
> +void blockdev_close_all_bdrv_states(void)
> +{
> + BlockDriverState *bs, *next_bs;
> +
> + QTAILQ_FOREACH_SAFE(bs, &monitor_bdrv_states, monitor_list, next_bs) {
> + AioContext *ctx = bdrv_get_aio_context(bs);
> +
> + aio_context_acquire(ctx);
> + bdrv_unref(bs);
> + aio_context_release(ctx);
> + }
> +}
> +
> static void qemu_opt_rename(QemuOpts *opts, const char *from, const char *to,
> Error **errp)
> {
> @@ -3464,6 +3480,8 @@ void qmp_blockdev_add(BlockdevOptions *options, Error
> **errp)
> if (!bs) {
> goto fail;
> }
> +
> + QTAILQ_INSERT_TAIL(&monitor_bdrv_states, bs, monitor_list);
This is missing a QTAILQ_REMOVE(&monitor_bdrv_states, bs, monitor_list)
in the if (bs && bdrv_key_required(bs)) { if (!blk) { ... } } branch
below this place.
Max
> }
>
> if (bs && bdrv_key_required(bs)) {
> @@ -3534,11 +3552,18 @@ void qmp_x_blockdev_del(bool has_id, const char *id,
> bdrv_get_device_or_node_name(bs));
> goto out;
> }
> +
> + if (!blk && !bs->monitor_list.tqe_prev) {
> + error_setg(errp, "Node %s is not owned by the monitor",
> + bs->node_name);
> + goto out;
> + }
> }
>
> if (blk) {
> blk_unref(blk);
> } else {
> + QTAILQ_REMOVE(&monitor_bdrv_states, bs, monitor_list);
> bdrv_unref(bs);
> }
>
> diff --git a/include/block/block_int.h b/include/block/block_int.h
> index 8c82747..7408eef 100644
> --- a/include/block/block_int.h
> +++ b/include/block/block_int.h
> @@ -424,6 +424,8 @@ struct BlockDriverState {
> QTAILQ_ENTRY(BlockDriverState) device_list;
> /* element of the list of all BlockDriverStates (all_bdrv_states) */
> QTAILQ_ENTRY(BlockDriverState) bs_list;
> + /* element of the list of monitor-owned BDS */
> + QTAILQ_ENTRY(BlockDriverState) monitor_list;
> QLIST_HEAD(, BdrvDirtyBitmap) dirty_bitmaps;
> int refcnt;
>
> @@ -680,4 +682,6 @@ void blk_dev_resize_cb(BlockBackend *blk);
> void bdrv_set_dirty(BlockDriverState *bs, int64_t cur_sector, int
> nr_sectors);
> bool bdrv_requests_pending(BlockDriverState *bs);
>
> +void blockdev_close_all_bdrv_states(void);
> +
> #endif /* BLOCK_INT_H */
> diff --git a/stubs/Makefile.objs b/stubs/Makefile.objs
> index 330e1a4..c1f02be 100644
> --- a/stubs/Makefile.objs
> +++ b/stubs/Makefile.objs
> @@ -1,5 +1,6 @@
> stub-obj-y += arch-query-cpu-def.o
> stub-obj-y += bdrv-commit-all.o
> +stub-obj-y += blockdev-close-all-bdrv-states.o
> stub-obj-y += clock-warp.o
> stub-obj-y += cpu-get-clock.o
> stub-obj-y += cpu-get-icount.o
> diff --git a/stubs/blockdev-close-all-bdrv-states.c
> b/stubs/blockdev-close-all-bdrv-states.c
> new file mode 100644
> index 0000000..12d2442
> --- /dev/null
> +++ b/stubs/blockdev-close-all-bdrv-states.c
> @@ -0,0 +1,5 @@
> +#include "block/block_int.h"
> +
> +void blockdev_close_all_bdrv_states(void)
> +{
> +}
>
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v7 15/24] block: Remove BDS close notifier, (continued)
- [Qemu-devel] [PATCH v7 15/24] block: Remove BDS close notifier, Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 16/24] block: Use blk_remove_bs() in blk_delete(), Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 17/24] blockdev: Use blk_remove_bs() in do_drive_del(), Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 18/24] block: Make bdrv_close() static, Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 19/24] block: Add list of all BlockDriverStates, Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 20/24] blockdev: Keep track of monitor-owned BDS, Max Reitz, 2015/11/09
- Re: [Qemu-devel] [PATCH v7 20/24] blockdev: Keep track of monitor-owned BDS,
Max Reitz <=
- [Qemu-devel] [PATCH v7 21/24] block: Add blk_remove_all_bs(), Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 22/24] block: Rewrite bdrv_close_all(), Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 23/24] iotests: Add test for multiple BB on BDS tree, Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 24/24] iotests: Add test for block jobs and BDS ejection, Max Reitz, 2015/11/09
- Re: [Qemu-devel] [PATCH v7 for-2.6 00/24] block: Rework bdrv_close_all(), Kevin Wolf, 2015/11/25