Re: [PATCH net-next 2/2] vsock/test: check also expected errno on sigpipe test

[Stefano Garzarella <sgarzare@xxxxxxxxxx>]

On Tue, May 13, 2025 at 12:41:17PM +0200, Paolo Abeni wrote:
> 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?

Ooops, copy & paste where we waited successful send().

> Same thing below.

I'll fix both!

Thanks,
Stefano