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)`?