git: c9c2452a2535 - main - unix tests: Skip random_eor_and_waitall unconditionally
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 25 Sep 2024 07:13:47 UTC
The branch main has been updated by markj:
URL: https://cgit.FreeBSD.org/src/commit/?id=c9c2452a25355db7d89bfc93fd9d50f46690949c
commit c9c2452a25355db7d89bfc93fd9d50f46690949c
Author: Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2024-09-21 12:41:06 +0000
Commit: Mark Johnston <markj@FreeBSD.org>
CommitDate: 2024-09-25 11:44:14 +0000
unix tests: Skip random_eor_and_waitall unconditionally
This test always fails, I don't see any reason to make it conditional on
the "CI" test parameter.
There is at least one test bug here, we're using the wrong sysctl to
obtain the receive buffer size, but fixing that is not sufficient.
PR: 279354
Reviewed by: glebius
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D46726
---
tests/sys/kern/unix_seqpacket_test.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tests/sys/kern/unix_seqpacket_test.c b/tests/sys/kern/unix_seqpacket_test.c
index ba3de0adb1e9..d142e228b036 100644
--- a/tests/sys/kern/unix_seqpacket_test.c
+++ b/tests/sys/kern/unix_seqpacket_test.c
@@ -1197,8 +1197,7 @@ ATF_TC_BODY(random_eor_and_waitall, tc)
size_t off;
int fd[2], eor;
- if (atf_tc_get_config_var_as_bool_wd(tc, "ci", false))
- atf_tc_skip("https://bugs.freebsd.org/279354");
+ atf_tc_skip("https://bugs.freebsd.org/279354");
arc4random_buf(params.seed, sizeof(params.seed));
printf("Using seed:");
@@ -1210,7 +1209,7 @@ ATF_TC_BODY(random_eor_and_waitall, tc)
for (u_int i = 0; i < RANDOM_TESTSIZE / (u_int )sizeof(long); i++)
((long *)params.sendbuf)[i] = nrand48(¶ms.seed[0]);
- ATF_REQUIRE(sysctlbyname("net.local.stream.recvspace",
+ ATF_REQUIRE(sysctlbyname("net.local.seqpacket.recvspace",
¶ms.recvspace, &(size_t){sizeof(u_long)}, NULL, 0) != -1);
ATF_REQUIRE((recvbuf =
malloc(RANDOM_RECVSIZE * params.recvspace)) != NULL);