Re: [PATCH 0/3] MR617: Draft: server: Always prefer synchronous I/O in nonblocking mode. (#53486)

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about server/sock.c:
> * asyncs will not consume all available data; if there's no data
>           * available, the current request won't be immediately satiable.
>           */
> -        struct pollfd pollfd;
> -        pollfd.fd = get_unix_fd( sock->fd );
> -        pollfd.events = req->oob ? POLLPRI : POLLIN;
> -        pollfd.revents = 0;
> -        if (poll(&pollfd, 1, 0) >= 0 && pollfd.revents)
> +        if (sock->nonblocking ||

Should this actually be `(sock->nonblocking && !req->force_async)`?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/617#note_6041