From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37564) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b5Mft-0005Yv-Ra for qemu-devel@nongnu.org; Tue, 24 May 2016 20:27:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b5Mfr-0004PZ-Q1 for qemu-devel@nongnu.org; Tue, 24 May 2016 20:27:48 -0400 References: <1464097654-12977-1-git-send-email-kwolf@redhat.com> <1464097654-12977-7-git-send-email-kwolf@redhat.com> From: Eric Blake Message-ID: <5744F17C.8050902@redhat.com> Date: Tue, 24 May 2016 18:27:40 -0600 MIME-Version: 1.0 In-Reply-To: <1464097654-12977-7-git-send-email-kwolf@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="02UQm69UrJf38LSOGMif0uQnuXnk7uCTm" Subject: Re: [Qemu-devel] [PATCH v2 06/14] stream: Use BlockBackend for I/O List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf , qemu-block@nongnu.org Cc: berto@igalia.com, jcody@redhat.com, qemu-devel@nongnu.org, mreitz@redhat.com, jsnow@redhat.com This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --02UQm69UrJf38LSOGMif0uQnuXnk7uCTm Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 05/24/2016 07:47 AM, Kevin Wolf wrote: > This changes the streaming block job to use the job's BlockBackend for > performing the COR reads. job->bs isn't used by the streaming code any > more afterwards. >=20 > Signed-off-by: Kevin Wolf > --- > block/io.c | 9 --------- > block/stream.c | 15 +++++++++------ > include/block/block.h | 2 -- > trace-events | 1 - > 4 files changed, 9 insertions(+), 18 deletions(-) >=20 > +++ b/trace-events > @@ -72,7 +72,6 @@ bdrv_aio_readv(void *bs, int64_t sector_num, int nb_s= ectors, void *opaque) "bs % > bdrv_aio_writev(void *bs, int64_t sector_num, int nb_sectors, void *op= aque) "bs %p sector_num %"PRId64" nb_sectors %d opaque %p" > bdrv_aio_write_zeroes(void *bs, int64_t sector_num, int nb_sectors, in= t flags, void *opaque) "bs %p sector_num %"PRId64" nb_sectors %d flags %#= x opaque %p" > bdrv_co_readv(void *bs, int64_t sector_num, int nb_sector) "bs %p sect= or_num %"PRId64" nb_sectors %d" > -bdrv_co_copy_on_readv(void *bs, int64_t sector_num, int nb_sector) "bs= %p sector_num %"PRId64" nb_sectors %d" Yay - one less sector-based trace point - we're slowly getting to an all-bytes destination. Reviewed-by: Eric Blake --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org --02UQm69UrJf38LSOGMif0uQnuXnk7uCTm Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBCAAGBQJXRPF8AAoJEKeha0olJ0Nq8X8H/2vRogS+FffNNe/VRaxcRWZe qSe5YhrLf2nohf7JAkMXY5VcPu3XYGLEjfK4Z+U8v93+0xrEbtow8hR+ciXxtMLJ MIOZ0Vg6OfwQl0bz1lrQ2hWOrXrxtxvDcBUeg3rmK4qZ+dc+boo5YtvDRt/3l6sV 6f3Cj1IBw+tggD5wYJ4iHVvTb6xsW2B1iEWQTwZ54uqUXP8jt7UXytB0+uAhHl7i nVmPJ5NYjgRi5lAa2OX7MT6YN5T/07n3Cj5fKIJHlncKzYNwidS+lfe3WmDsGxBE f/mO0r35O2avlZXLW10KLm3sctSS1EWQHfy7ML+qcfp/Cq8AkyRkACakVsoJEWM= =aRbn -----END PGP SIGNATURE----- --02UQm69UrJf38LSOGMif0uQnuXnk7uCTm--