Thread (7 messages) read the whole thread 7 messages, 2 authors, 2025-05-13

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

From: Paolo Abeni <pabeni@redhat.com>
Date: 2025-05-13 10:41:22
Also in: lkml, virtualization

On 5/8/25 4:20 PM, Stefano Garzarella wrote:
quoted hunk ↗ jump to hunk
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help