qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 2/2] aio: Introduce aio-epoll.c


From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH v2 2/2] aio: Introduce aio-epoll.c
Date: Fri, 16 Oct 2015 11:32:31 +0200
User-agent: Mutt/1.5.24 (2015-08-30)

On Tue, Oct 13, 2015 at 07:10:55PM +0800, Fam Zheng wrote:
> +static bool aio_epoll_try_enable(AioContext *ctx)
> +{
> +    AioHandler *node;
> +    struct epoll_event event;
> +    if (!ctx->epoll_available) {
> +        return false;
> +    }

Why check this here since aio_epoll_check_poll() already checks it?

> +static int aio_epoll(AioContext *ctx, GPollFD *pfds,
> +                     unsigned npfd, int64_t timeout)
> +{
> +    AioHandler *node;
> +    int i, ret = 0;
> +    struct epoll_event events[128];

The strategy is to support up to 128 events per epoll_wait(2) call and
then wait for the next event loop iteration to harvest any remaining
events?

I just want to make sure I understand how this constant affects epoll
behavior.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]