[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 4/8] linux-aio: Implement .cancel_async
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v3 4/8] linux-aio: Implement .cancel_async |
Date: |
Tue, 2 Sep 2014 15:55:00 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Aug 27, 2014 at 10:49:12AM +0800, Fam Zheng wrote:
> @@ -110,6 +109,22 @@ static void qemu_laio_completion_cb(EventNotifier *e)
> }
> }
>
> +static void laio_cancel_async(BlockDriverAIOCB *blockacb)
> +{
> + struct qemu_laiocb *laiocb = (struct qemu_laiocb *)blockacb;
> + struct io_event event;
> + int ret;
> +
> + ret = io_cancel(laiocb->ctx->ctx, &laiocb->iocb, &event);
> + laiocb->ret = -ECANCELED;
> + if (!ret) {
> + /* iocb is not cancelled, cb will be called by the event loop later
> */
> + return;
> + }
This comment doesn't make sense, io_cancel() returns 0 on success so it
has been cancelled. We need to call the completion callback!
Stefan
pgp5ZfZoPoDrU.pgp
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 4/8] linux-aio: Implement .cancel_async,
Stefan Hajnoczi <=