Mike Kaplinskiy <mike.kaplins...@gmail.com> writes:

> Hope this makes the rationale a bit clearer. Although admittedly the
> patch does allow for some busy waiting since
> if (mask & FD_READ  || async_waiting( sock->read_q )) ev |= POLLIN | POLLPRI;
> should be more like
> if ( (mask & FD_READ && !async_busy( sock->read_q ))  ||
> async_waiting( sock->read_q )) ev |= POLLIN | POLLPRI;
> which probably should've went into #3.
>
> Should I merge 2&3 and resend?

I think there's more to it than merging them. If you have to ignore
events you are doing something wrong, because you shouldn't get the
events in the first place.

-- 
Alexandre Julliard
julli...@winehq.org


Reply via email to