From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:32780) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asXI3-0008DL-Kj for qemu-devel@nongnu.org; Tue, 19 Apr 2016 11:10:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1asXHz-0008Um-Lp for qemu-devel@nongnu.org; Tue, 19 Apr 2016 11:10:11 -0400 Date: Tue, 19 Apr 2016 17:09:58 +0200 From: Kevin Wolf Message-ID: <20160419150958.GC4498@noname.str.redhat.com> References: <1460046816-102846-1-git-send-email-pbonzini@redhat.com> <20160419091027.GB16312@stefanha-x1.localdomain> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="RnlQjJ0d97Da+TV1" Content-Disposition: inline In-Reply-To: <20160419091027.GB16312@stefanha-x1.localdomain> Subject: Re: [Qemu-devel] [Qemu-block] [PATCH v4 0/8] bdrv_flush_io_queue removal, shared LinuxAioState List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Paolo Bonzini , qemu-devel@nongnu.org, famz@redhat.com, qemu-block@nongnu.org, stefanha@redhat.com --RnlQjJ0d97Da+TV1 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Am 19.04.2016 um 11:10 hat Stefan Hajnoczi geschrieben: > On Thu, Apr 07, 2016 at 06:33:28PM +0200, Paolo Bonzini wrote: > > Patch 1 comes from Kevin's series to do BlockBackend throttling. > >=20 > > Patches 2-6 are from my bdrv_drain patches. They apply on top of Fam's > > patch (which will be in 2.6) that introduces bdrv_co_drain. Patch 4 > > is new in this version, compared to v3. > >=20 > > Patches 7-8 are new but based on Ming Lei's old submission. > > I'm including them here because they apply on top of patches 2-6. > >=20 > > Kevin Wolf (1): > > block: Don't disable I/O throttling on sync requests > >=20 > > Paolo Bonzini (7): > > block: make bdrv_start_throttled_reqs return void > > block: move restarting of throttled reqs to block/throttle-groups.c > > block: extract bdrv_drain_poll/bdrv_co_yield_to_drain from > > bdrv_drain/bdrv_co_drain > > block: introduce bdrv_no_throttling_begin/end > > block: plug whole tree at once, introduce bdrv_io_unplugged_begin/end > > linux-aio: make it more type safe > > linux-aio: share one LinuxAioState within an AioContext > >=20 > > async.c | 23 +++++ > > block.c | 1 - > > block/block-backend.c | 6 +- > > block/io.c | 168 ++++++++++++++++++++++-------= -------- > > block/linux-aio.c | 60 ++++++------- > > block/raw-posix.c | 133 ++++------------------------- > > block/raw-win32.c | 2 +- > > block/throttle-groups.c | 18 ++++ > > include/block/aio.h | 13 +++ > > include/block/block.h | 3 +- > > include/block/block_int.h | 14 +++- > > {block =3D> include/block}/raw-aio.h | 15 ++-- > > include/block/throttle-groups.h | 1 + > > 13 files changed, 216 insertions(+), 241 deletions(-) > > rename {block =3D> include/block}/raw-aio.h (80%) >=20 > Looks good overall. I have reservations about the last patch, see my > reply there. >=20 > Acked-by: Stefan Hajnoczi Thanks, applied patches 1-7 to block-next. Kevin --RnlQjJ0d97Da+TV1 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJXFkpGAAoJEH8JsnLIjy/WBx8P/3G4ylAA0Z8YKrCb4MJMaPvJ ygfE0u19E9F4JX6CrZuRBVOXArSnqo8FBFwqkvZzzGpEoduVRH5bKtFmc7pzhxNN dGVewXQevo9O1VU6oSOBpGXgjwd6R4EtwnSmfTq2v7zadIrUd+ijR4l/x06JxNtJ aWiZavhAcf5N/VZwDor+DK6OBQ1mlST4jZiPpswMCYxENVGTKQX0UGZJMSMpqDYl 779SxjZr4bTinGg5n0BVPNB3lGOewE3S2oX/YbvGsBRpB4MO7x6amR05E9Pj2xgW wWeVp9LyeCxLPup38qUXItJhzuzcGMQRtzr1tgz3JvtoAdnoFIC4ei/e2iB77rbJ OXvA1yL2ZXoJmp7xGiUzNhqwgqOrHaNJMT5G11JpAg71kuNaKuEu4lSqkC1disec PcUcHibTwt0tukN2YEYENJTp5DcKtVtSXcrDCYMeMNRiAW49T9bZrOvquTVt91mP x6HFUJ67tGHb7B2HcR88ODORQ4DMjYRXWejZQu1eV39olb2p3cAhnBD2/Ol6Hu/e ha8W/LQCFzXpzPt/70CK8PM/ltDa+mFlghjHbfh6/+NZ4YZrEEh5cYcA3qJO9zst TShBj3eVr8HCtjOfODKVVjhfw0wws6xAturspdvIxNQDmFk0y1f1hkw+eCjDj0eA S/UwH6vARjIuiUosoGbX =j52a -----END PGP SIGNATURE----- --RnlQjJ0d97Da+TV1--