From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40270) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WGxXI-0007CQ-KS for qemu-devel@nongnu.org; Fri, 21 Feb 2014 16:21:41 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WGxXA-0000gQ-3m for qemu-devel@nongnu.org; Fri, 21 Feb 2014 16:21:32 -0500 Received: from lnantes-156-75-100-125.w80-12.abo.wanadoo.fr ([80.12.84.125]:58647 helo=paradis.irqsave.net) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WGxX9-0000ft-TK for qemu-devel@nongnu.org; Fri, 21 Feb 2014 16:21:24 -0500 From: =?UTF-8?q?Beno=C3=AEt=20Canet?= Date: Fri, 21 Feb 2014 22:21:14 +0100 Message-Id: <1393017681-12794-6-git-send-email-benoit.canet@irqsave.net> In-Reply-To: <1393017681-12794-1-git-send-email-benoit.canet@irqsave.net> References: <1393017681-12794-1-git-send-email-benoit.canet@irqsave.net> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH V19 05/12] quorum: Add quorum_aio_readv. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, =?UTF-8?q?Beno=C3=AEt=20Canet?= , stefanha@redhat.com, mreitz@redhat.com From: Beno=C3=AEt Canet Add code to do num_children reads in parallel and cleanup the structures afterwards. Signed-off-by: Benoit Canet Reviewed-by: Max Reitz --- block/quorum.c | 39 ++++++++++++++++++++++++++++++++++++++- 1 file changed, 38 insertions(+), 1 deletion(-) diff --git a/block/quorum.c b/block/quorum.c index 99e0e03..fd19662 100644 --- a/block/quorum.c +++ b/block/quorum.c @@ -91,10 +91,18 @@ static AIOCBInfo quorum_aiocb_info =3D { =20 static void quorum_aio_finalize(QuorumAIOCB *acb) { - int ret =3D 0; + BDRVQuorumState *s =3D acb->common.bs->opaque; + int i, ret =3D 0; =20 acb->common.cb(acb->common.opaque, ret); =20 + if (acb->is_read) { + for (i =3D 0; i < s->num_children; i++) { + qemu_vfree(acb->qcrs[i].buf); + qemu_iovec_destroy(&acb->qcrs[i].qiov); + } + } + g_free(acb->qcrs); qemu_aio_release(acb); } @@ -149,6 +157,34 @@ static void quorum_aio_cb(void *opaque, int ret) quorum_aio_finalize(acb); } =20 +static BlockDriverAIOCB *quorum_aio_readv(BlockDriverState *bs, + int64_t sector_num, + QEMUIOVector *qiov, + int nb_sectors, + BlockDriverCompletionFunc *cb, + void *opaque) +{ + BDRVQuorumState *s =3D bs->opaque; + QuorumAIOCB *acb =3D quorum_aio_get(s, bs, qiov, sector_num, + nb_sectors, cb, opaque); + int i; + + acb->is_read =3D true; + + for (i =3D 0; i < s->num_children; i++) { + acb->qcrs[i].buf =3D qemu_blockalign(s->bs[i], qiov->size); + qemu_iovec_init(&acb->qcrs[i].qiov, qiov->niov); + qemu_iovec_clone(&acb->qcrs[i].qiov, qiov, acb->qcrs[i].buf); + } + + for (i =3D 0; i < s->num_children; i++) { + bdrv_aio_readv(s->bs[i], sector_num, qiov, nb_sectors, + quorum_aio_cb, &acb->qcrs[i]); + } + + return &acb->common; +} + static BlockDriverAIOCB *quorum_aio_writev(BlockDriverState *bs, int64_t sector_num, QEMUIOVector *qiov, @@ -176,6 +212,7 @@ static BlockDriver bdrv_quorum =3D { =20 .instance_size =3D sizeof(BDRVQuorumState), =20 + .bdrv_aio_readv =3D quorum_aio_readv, .bdrv_aio_writev =3D quorum_aio_writev, }; =20 --=20 1.8.3.2