Re: [Qemu-devel] [QEMU-devel][PATCH v2] aio-posix: Fix concurrent aio_poll/set_fd_handler.

[Remy NOEL]

On 12/10/18 8:05 PM, Stefan Hajnoczi wrote:

> On Thu, Dec 06, 2018 at 11:14:23AM +0100, address@hidden wrote:
> > +        if (is_new) {
> > +            new_node->pfd.fd = fd;
> > +        } else {
> > +            deleted = aio_remove_fd_handler(ctx, node);
> > +            new_node->pfd = node->pfd;
> Does this drop revents?  Imagine revents has bits set, then
> aio_remove_fd_handler() will clear them and new_node->pfd = node->pfd is
> too late to preserve them.

Indeed, i will invert those.

Thanks !