From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42846) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bD895-0003aM-UI for qemu-devel@nongnu.org; Wed, 15 Jun 2016 06:34:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bD893-0004Qa-Rs for qemu-devel@nongnu.org; Wed, 15 Jun 2016 06:34:02 -0400 Date: Wed, 15 Jun 2016 12:33:51 +0200 From: Kevin Wolf Message-ID: <20160615103351.GG4566@noname.redhat.com> References: <1465911155-19002-1-git-send-email-kwolf@redhat.com> <20160615090251.GA25151@stefanha-x1.localdomain> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="pvezYHf7grwyp3Bc" Content-Disposition: inline In-Reply-To: <20160615090251.GA25151@stefanha-x1.localdomain> Subject: Re: [Qemu-devel] [PATCH v2 0/6] block: Enable byte granularity I/O List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: qemu-block@nongnu.org, famz@redhat.com, eblake@redhat.com, mreitz@redhat.com, qemu-devel@nongnu.org --pvezYHf7grwyp3Bc Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Am 15.06.2016 um 11:02 hat Stefan Hajnoczi geschrieben: > On Tue, Jun 14, 2016 at 03:32:29PM +0200, Kevin Wolf wrote: > > Previous series have already converted some block drivers to byte-based= rather > > than sector-based interfaces. However, the common I/O path as well as r= aw-posix > > still enforced a minimum alignment of 512 bytes because some sector-bas= ed logic > > was involved. > >=20 > > This patch series removes these limitations and a sub-sector request ac= tually > > ends up as a sub-sector syscall now. > >=20 > > v2: > > - Updated trace-events for bdrv_co_do_copy_on_readv() [Eric] > > - Added some assertions [Eric] > > - Renamed laio_submit_co() -> laio_co_submit() and added coroutine_fn > > to its prototype [Stefan] > > - linux-aio: Include block/block.h and use BDRV_SECTOR_SIZE instead > > of 512 [Eric] > >=20 > > Kevin Wolf (6): > > block: Byte-based bdrv_co_do_copy_on_readv() > > block: Prepare bdrv_aligned_preadv() for byte-aligned requests > > block: Prepare bdrv_aligned_pwritev() for byte-aligned requests > > raw-posix: Switch to bdrv_co_* interfaces > > raw-posix: Implement .bdrv_co_preadv/pwritev > > block: Don't enforce 512 byte minimum alignment > >=20 > > block.c | 2 +- > > block/io.c | 128 ++++++++++++++++++++++++++----------------= -------- > > block/linux-aio.c | 84 ++++++++++++++++++++++++--------- > > block/mirror.c | 10 ++-- > > block/raw-aio.h | 3 ++ > > block/raw-posix.c | 62 ++++++++++++------------ > > include/block/block.h | 10 ++-- > > trace-events | 2 +- > > 8 files changed, 176 insertions(+), 125 deletions(-) >=20 > Reviewed-by: Stefan Hajnoczi Thanks. Applied to my block branch. Kevin --pvezYHf7grwyp3Bc Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJXYS8PAAoJEH8JsnLIjy/WIzkQALBPOdHWX1FZWJnJWI7+myXR 08UpB5deG0jyAKAJhUX8JbgZUkQ/MXwa9M1BPDBWW9VElYy9rRTaFhdYukOgFFU5 UR6vi1D1WKFfduDcS30nfIxMIGJO4YsWkF+NZL1WbFjdSVKvy4oxaCASRSS7CkYN 4ToSPkHjnmuPcs/M17afDgWFaT4VMK6wAls/nVlqmWJ0G+RH9QAC1Pw2OV6NxRkv p1nicAeYtrtxtJvBQqgtxHQoPNMto9yUpeh2/cxX1JNLSp/w+HQPlrQYSp9bCjlQ adXFj8AQ35ozuW2k92QtaY/QdcU9Jq3y2OMpqUYQKdi2IE08jcC8zlTfgdcwfmih aPXEnUue9wBiOHw4bvQfFPHkgNzuBEW1rp0e2oNTv44XXh3qCo6S2Thto4aZQRV1 HrhwQapfIvw90O5dHO/B9G6d8ouQax4I4oxkAR+i4NydGXjPBuaiXGd5eqG1iBSZ E3L9Cx5cKioB3XFiwur0gTK/+np6UR//cRnlCKgeuZqXP1EzNpY4cM5XZdK0Ctad vK7mTYChAcdZZ/HpDTgHitcemlant/gTjTqLy4X+aZkz5bpyJ+/X0v3ei8xmDm6X z9vrY45jwdKtSXGf6hcmTI90EQhtnpIHTMEkKKpfvTxZYDRG1ArDvQOJr5HgVu4T Kck4/Hi/ZKF+wA2G9aMz =BaU4 -----END PGP SIGNATURE----- --pvezYHf7grwyp3Bc--