Re: [PATCH v2 1/7] vhost-user: Drop misleading EAGAIN checks in slave_read()

2021-03-15 Thread Stefan Hajnoczi
On Fri, Mar 12, 2021 at 10:22:06AM +0100, Greg Kurz wrote: > slave_read() checks EAGAIN when reading or writing to the socket > fails. This gives the impression that the slave channel is in > non-blocking mode, which is certainly not the case with the current > code base. And the rest of the code i

[PATCH v2 1/7] vhost-user: Drop misleading EAGAIN checks in slave_read()

2021-03-12 Thread Greg Kurz
slave_read() checks EAGAIN when reading or writing to the socket fails. This gives the impression that the slave channel is in non-blocking mode, which is certainly not the case with the current code base. And the rest of the code isn't actually ready to cope with non-blocking I/O. Just drop the c