The following changes since commit 508ba0f7e2092d3ca56e3f75e894d52d8b94818e:

Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20171109' into staging (2017-11-13 11:41:47 +0000)

are available in the git repository at:

git://github.com/stefanha/qemu.git tags/block-pull-request

for you to fetch changes up to 0761562687e0d8135310a94b1d3e08376387c027:

qemu-iotests: Test I/O limits with removable media (2017-11-13 15:46:26 +0000)

Alberto Garcia (3):

block: Check for inserted BlockDriverState in blk_io_limits_disable()

block: Leave valid throttle timers when removing a BDS from a backend

qemu-iotests: Test I/O limits with removable media

Stefan Hajnoczi (1):

throttle-groups: drain before detaching ThrottleState

Zhengui (1):

block: all I/O should be completed before removing throttle timers.

block/block-backend.c | 36 ++++++++++++++++++---------

block/throttle-groups.c | 6 +++++

tests/qemu-iotests/093 | 62 ++++++++++++++++++++++++++++++++++++++++++++++

tests/qemu-iotests/093.out | 4 +--

4 files changed, 94 insertions(+), 14 deletions(-)

2.13.6

From: Alberto Garcia <berto@igalia.com>

If a BlockBackend has I/O limits set then its ThrottleGroupMember

structure uses the AioContext from its attached BlockDriverState.

Those two contexts must be kept in sync manually. This is not

ideal and will be fixed in the future by removing the throttling

configuration from the BlockBackend and storing it in an implicit

filter node instead, but for now we have to live with this.

When you remove the BlockDriverState from the backend then the

throttle timers are destroyed. If a new BlockDriverState is later

inserted then they are created again using the new AioContext.

There are a couple of problems with this:

a) The code manipulates the timers directly, leaving the

ThrottleGroupMember.aio_context field in an inconsisent state.

b) If you remove the I/O limits (e.g by destroying the backend)

when the timers are gone then throttle_group_unregister_tgm()

will attempt to destroy them again, crashing QEMU.

While b) could be fixed easily by allowing the timers to be freed

twice, this would result in a situation in which we can no longer

guarantee that a valid ThrottleState has a valid AioContext and

timers.

This patch ensures that the timers and AioContext are always valid

when I/O limits are set, regardless of whether the BlockBackend has a

BlockDriverState inserted or not.

[Fixed "There'a" typo as suggested by Max Reitz <mreitz@redhat.com>

--Stefan]

Reported-by: sochin jiang <sochin.jiang@huawei.com>

Signed-off-by: Alberto Garcia <berto@igalia.com>

Reviewed-by: Max Reitz <mreitz@redhat.com>

Message-id: e089c66e7c20289b046d782cea4373b765c5bc1d.1510339534.git.berto@igalia.com

block/block-backend.c | 16 ++++++++--------

1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/block/block-backend.c b/block/block-backend.c

--- a/block/block-backend.c

+++ b/block/block-backend.c

@@ -XXX,XX +XXX,XX @@ BlockBackend *blk_by_public(BlockBackendPublic *public)

*/

void blk_remove_bs(BlockBackend *blk)

{

+ ThrottleGroupMember *tgm = &blk->public.throttle_group_member;

BlockDriverState *bs;

- ThrottleTimers *tt;

notifier_list_notify(&blk->remove_bs_notifiers, blk);

- if (blk->public.throttle_group_member.throttle_state) {

- tt = &blk->public.throttle_group_member.throttle_timers;

+ if (tgm->throttle_state) {

bs = blk_bs(blk);

bdrv_drained_begin(bs);

- throttle_timers_detach_aio_context(tt);

+ throttle_group_detach_aio_context(tgm);

+ throttle_group_attach_aio_context(tgm, qemu_get_aio_context());

bdrv_drained_end(bs);

@@ -XXX,XX +XXX,XX @@ void blk_remove_bs(BlockBackend *blk)

*/

int blk_insert_bs(BlockBackend *blk, BlockDriverState *bs, Error **errp)

{

+ ThrottleGroupMember *tgm = &blk->public.throttle_group_member;

blk->root = bdrv_root_attach_child(bs, "root", &child_root,

blk->perm, blk->shared_perm, blk, errp);

if (blk->root == NULL) {

@@ -XXX,XX +XXX,XX @@ int blk_insert_bs(BlockBackend *blk, BlockDriverState *bs, Error **errp)

bdrv_ref(bs);

notifier_list_notify(&blk->insert_bs_notifiers, blk);

- if (blk->public.throttle_group_member.throttle_state) {

- throttle_timers_attach_aio_context(

- &blk->public.throttle_group_member.throttle_timers,

- bdrv_get_aio_context(bs));

+ if (tgm->throttle_state) {

+ throttle_group_detach_aio_context(tgm);

+ throttle_group_attach_aio_context(tgm, bdrv_get_aio_context(bs));

return 0;

2.13.6