From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56295) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dv1Mf-00050P-On for qemu-devel@nongnu.org; Thu, 21 Sep 2017 09:18:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dv1Me-00044Z-Rs for qemu-devel@nongnu.org; Thu, 21 Sep 2017 09:18:01 -0400 From: Manos Pitsidianakis Date: Thu, 21 Sep 2017 16:17:04 +0300 Message-Id: <20170921131707.17364-1-el13635@mail.ntua.gr> Subject: [Qemu-devel] [PATCH v2 0/3] add bdrv_co_drain_begin/end BlockDriver callbacks List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel Cc: qemu-block , Kevin Wolf , Fam Zheng , Stefan Hajnoczi , Max Reitz This patch series renames bdrv_co_drain to bdrv_co_drain_begin and adds a new bdrv_co_drain_end callback to match bdrv_drained_begin/end and drained_begin/end of BdrvChild. This is needed because the throttle driver (block/throttle.c) needs a way to mark the end of the drain in order to toggle io_limits_disabled correctly. Based-on: <20170918202529.28379-1-el13635@mail.ntua.gr> "block/throttle-groups.c: allocate RestartData on the heap" Which fixes a coroutine crash in block/throttle-groups.c v2: add doc for callbacks and change order of request polling for completion [Stefan] Manos Pitsidianakis (3): block: add bdrv_co_drain_end callback block: rename bdrv_co_drain to bdrv_co_drain_begin block/throttle.c: add bdrv_co_drain_begin/end callbacks include/block/block_int.h | 8 +++++++- block/io.c | 48 +++++++++++++++++++++++++++++++++-------------- block/qed.c | 6 +++--- block/throttle.c | 18 ++++++++++++++++++ 4 files changed, 62 insertions(+), 18 deletions(-) -- 2.11.0