From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57561) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1duKnF-0006lz-Uq for qemu-devel@nongnu.org; Tue, 19 Sep 2017 11:50:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1duKnF-00053n-3w for qemu-devel@nongnu.org; Tue, 19 Sep 2017 11:50:38 -0400 From: Stefan Hajnoczi Date: Tue, 19 Sep 2017 16:50:25 +0100 Message-Id: <20170919155025.4098-1-stefanha@redhat.com> Subject: [Qemu-devel] [PATCH] throttle-groups: update tg->any_timer_armed[] on detach List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Max Reitz , Kevin Wolf , qemu-block@nongnu.org, Manos Pitsidianakis , Alberto Garcia , Stefan Hajnoczi Clear tg->any_timer_armed[] when throttling timers are destroy during AioContext attach/detach. Failure to do so causes throttling to hang because we believe the timer is already scheduled! The following was broken at least since QEMU 2.10.0 with -drive iops=100: $ dd if=/dev/zero of=/dev/vdb oflag=direct count=1000 (qemu) stop (qemu) cont ...I/O is stuck... Reported-by: Yongxue Hong Signed-off-by: Stefan Hajnoczi --- block/throttle-groups.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/block/throttle-groups.c b/block/throttle-groups.c index 6ba992c8d7..2bfd03faa0 100644 --- a/block/throttle-groups.c +++ b/block/throttle-groups.c @@ -592,7 +592,20 @@ void throttle_group_attach_aio_context(ThrottleGroupMember *tgm, void throttle_group_detach_aio_context(ThrottleGroupMember *tgm) { ThrottleTimers *tt = &tgm->throttle_timers; + ThrottleGroup *tg = container_of(tgm->throttle_state, ThrottleGroup, ts); + throttle_timers_detach_aio_context(tt); + + /* Forget about these timers, they have been destroyed */ + qemu_mutex_lock(&tg->lock); + if (tg->tokens[0] == tgm) { + tg->any_timer_armed[0] = false; + } + if (tg->tokens[1] == tgm) { + tg->any_timer_armed[1] = false; + } + qemu_mutex_unlock(&tg->lock); + tgm->aio_context = NULL; } -- 2.13.5