From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36771) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZqbDT-0006jR-KV for qemu-devel@nongnu.org; Mon, 26 Oct 2015 02:25:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZqbDS-0004x9-R8 for qemu-devel@nongnu.org; Mon, 26 Oct 2015 02:25:11 -0400 From: Fam Zheng Date: Mon, 26 Oct 2015 14:24:44 +0800 Message-Id: <1445840693-3177-1-git-send-email-famz@redhat.com> Subject: [Qemu-devel] [PATCH 0/9] block: Fixes for bdrv_drain List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , Stefan Hajnoczi , qemu-block@nongnu.org, Peter Lieven , Ronnie Sahlberg , Paolo Bonzini Previously bdrv_drain and bdrv_drain_all don't handle ioctl, flush and discard requests (which are fundamentally the same as read and write requests that change disk state). Forgetting such requests leaves us in risk of violating the invariant that bdrv_drain() callers rely on - all asynchronous requests must have completed after bdrv_drain returns. Enrich the tracked request types, and add tracked_request_begin/end pairs to all three code paths. As a prerequisite, ioctl code is moved into coroutine too. The last two patches take care of QED's "need check" timer, so that after bdrv_drain returns, the driver is in a consistent state. Fam Fam Zheng (9): block: Add more types for tracked request block: Track flush requests block: Track discard requests iscsi: Emulate commands in iscsi_aio_ioctl as iscsi_ioctl block: Add ioctl parameter fields to BlockRequest block: Emulate bdrv_ioctl with bdrv_aio_ioctl and track both block: Drop BlockDriver.bdrv_ioctl block: Introduce BlockDriver.bdrv_drain callback qed: Implement .bdrv_drain block/io.c | 144 +++++++++++++++++++++++++++++++++++++++------- block/iscsi.c | 72 ++++++++++++----------- block/qed.c | 13 +++++ block/raw-posix.c | 9 --- block/raw_bsd.c | 6 -- include/block/block.h | 16 ++++-- include/block/block_int.h | 17 +++++- 7 files changed, 200 insertions(+), 77 deletions(-) -- 2.4.3