From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60950) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dv1dv-0005hP-7m for qemu-devel@nongnu.org; Thu, 21 Sep 2017 09:35:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dv1du-0005HK-B8 for qemu-devel@nongnu.org; Thu, 21 Sep 2017 09:35:51 -0400 Date: Thu, 21 Sep 2017 21:35:35 +0800 From: Fam Zheng Message-ID: <20170921133535.GI13703@lemon.lan> References: <20170921131707.17364-1-el13635@mail.ntua.gr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170921131707.17364-1-el13635@mail.ntua.gr> Subject: Re: [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: Manos Pitsidianakis Cc: qemu-devel , Kevin Wolf , Stefan Hajnoczi , qemu-block , Max Reitz On Thu, 09/21 16:17, Manos Pitsidianakis wrote: > 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. Is this a bug fix? I.e. do we need to Cc qemu-stable@? Fam