Re: [Qemu-devel] [RFC V3 8/9] quorum: Add quorum_aio_readv.

[Stefan Hajnoczi]

On Tue, Aug 14, 2012 at 04:14:10PM +0200, Benoît Canet wrote:
> Signed-off-by: Benoit Canet <address@hidden>
> ---
>  block/quorum.c |   35 ++++++++++++++++++++++++++++++++++-
>  1 file changed, 34 insertions(+), 1 deletion(-)
> 
> diff --git a/block/quorum.c b/block/quorum.c
> index 86962b4..8b449fb 100644
> --- a/block/quorum.c
> +++ b/block/quorum.c
> @@ -190,10 +190,16 @@ static void quorum_aio_bh(void *opaque)
>  {
>      QuorumAIOCB *acb = opaque;
>      BDRVQuorumState *s = acb->bqs;
> -    int ret;
> +    int i, ret;
> 
>      ret = s->n <= acb->success_count ? 0 : -EIO;
> 
> +    for (i = 0; i < s->m; i++) {
> +        qemu_vfree(acb->aios[i].buf);
> +        acb->aios[i].buf = NULL;
> +        acb->aios[i].ret = 0;
> +    }
> +
>      qemu_bh_delete(acb->bh);
>      acb->common.cb(acb->common.opaque, ret);
>      if (acb->finished) {
> @@ -258,6 +264,32 @@ static void quorum_aio_cb(void *opaque, int ret)
>      qemu_bh_schedule(acb->bh);
>  }
> 
> +static BlockDriverAIOCB *quorum_aio_readv(BlockDriverState *bs,
> +                                         int64_t sector_num,
> +                                         QEMUIOVector *qiov,
> +                                         int nb_sectors,
> +                                         BlockDriverCompletionFunc *cb,
> +                                         void *opaque)
> +{
> +    BDRVQuorumState *s = bs->opaque;
> +    QuorumAIOCB *acb = quorum_aio_get(s, bs, qiov, sector_num,
> +                                      nb_sectors, cb, opaque);
> +    int i;
> +
> +    for (i = 0; i < s->m; i++) {
> +        acb->aios[i].buf = qemu_blockalign(bs->file, qiov->size);
> +        qemu_iovec_init(&acb->qiovs[i], qiov->niov);
> +        qemu_iovec_clone(&acb->qiovs[i], qiov, acb->aios[i].buf);
> +    }

Need to call qemu_iovec_destroy() to free &acb->qiovs[i] iovecs.

Stefan