[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_p
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH RFC 0/3] iothread: release iothread around aio_poll |
Date: |
Wed, 22 Apr 2015 11:26:43 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Apr 21, 2015 at 05:54:59PM +0200, Paolo Bonzini wrote:
>
>
> On 21/04/2015 17:40, Stefan Hajnoczi wrote:
> >> >
> >> > Stefan, can you put this on track for 2.4 or do you need a repost?
> > This series causes qemu-iotests -qcow2 091 to fail:
> >
> > 9f83aea22314d928bb272153ff37d2d7f5adbf06 is the first bad commit
> > commit 9f83aea22314d928bb272153ff37d2d7f5adbf06
> > Author: Paolo Bonzini <address@hidden>
> > Date: Fri Feb 20 17:26:50 2015 +0100
> >
> > aio-posix: move pollfds to thread-local storage
>
> Oops... what I intended is this:
>
> diff --git a/aio-posix.c b/aio-posix.c
> index 4a30b77..e411591 100644
> --- a/aio-posix.c
> +++ b/aio-posix.c
> @@ -254,8 +254,7 @@ bool aio_poll(AioContext *ctx, bool blocking)
> aio_set_dispatching(ctx, !blocking);
>
> ctx->walking_handlers++;
> -
> - npfd = 0;
> + assert(npfd == 0);
>
> /* fill pollfds */
> QLIST_FOREACH(node, &ctx->aio_handlers, node) {
> @@ -276,6 +275,7 @@ bool aio_poll(AioContext *ctx, bool blocking)
> }
> }
>
> + npfd = 0;
> ctx->walking_handlers--;
>
> /* Run dispatch even if there were no readable fds to run timers */
>
> but the above is totally untested, so feel free to just remove
> the assertion or also to drop the series.
Thanks, I squashed this change in and now qemu-iotests passes.
Stefan
pgpu9PEnb9nDd.pgp
Description: PGP signature