On 5/8/25 4:20 PM, Stefano Garzarella wrote: > diff --git a/tools/testing/vsock/vsock_test.c > b/tools/testing/vsock/vsock_test.c > index 7de870dee1cf..533d9463a297 100644 > --- a/tools/testing/vsock/vsock_test.c > +++ b/tools/testing/vsock/vsock_test.c > @@ -1074,9 +1074,13 @@ static void test_stream_check_sigpipe(int fd) > do { > res = send(fd, "A", 1, 0); > timeout_check("send"); > - } while (res != -1); > + } while (res != -1 && errno == EINTR);
I'm low on coffee, but should the above condition be: res != -1 || errno == EINTR instead? Same thing below. /P