Commit 598b670a authored by Thomas Weißschuh's avatar Thomas Weißschuh
Browse files

selftests/nolibc: don't skip tests for unimplemented syscalls anymore 



The automatic skipping of tests on ENOSYS returns was introduced in
commit 349afc8a ("selftests/nolibc: skip tests for unimplemented
syscalls"). It handled the fact that nolibc would return ENOSYS for many
syscall wrappers on riscv32.

Nowadays nolibc handles all these correctly, so this logic is not used
anymore. To make missing nolibc functionality more obvious fail the
tests again if something is not implemented.

Revert the mentioned commit again.

Signed-off-by: default avatarThomas Weißschuh <linux@weissschuh.net>
Acked-by: default avatarWilly Tarreau <w@1wt.eu>
Link: https://patch.msgid.link/20260406-nolibc-no-skip-enosys-v1-2-c046b1ac7d73@weissschuh.net/
parent 9a5206f2

tools/testing/selftests/nolibc/nolibc-test.c

+3 −11
Original line number Diff line number Diff line
@@ -320,10 +320,7 @@ int expect_syszr(int expr, int llen) 
{

	int ret = 0;



	if (errno == ENOSYS) {

		llen += printf(" = ENOSYS");

		result(llen, SKIPPED);

	} else if (expr) {

	if (expr) {

		ret = 1;

		llen += printf(" = %d %s ", expr, errorname(errno));

		result(llen, FAIL);
@@ -363,10 +360,7 @@ int expect_sysne(int expr, int llen, int val) 
{

	int ret = 0;



	if (errno == ENOSYS) {

		llen += printf(" = ENOSYS");

		result(llen, SKIPPED);

	} else if (expr == val) {

	if (expr == val) {

		ret = 1;

		llen += printf(" = %d %s ", expr, errorname(errno));

		result(llen, FAIL);
@@ -391,9 +385,7 @@ int expect_syserr2(int expr, int expret, int experr1, int experr2, int llen) 
	int _errno = errno;



	llen += printf(" = %d %s ", expr, errorname(_errno));

	if (errno == ENOSYS) {

		result(llen, SKIPPED);

	} else if (expr != expret || (_errno != experr1 && _errno != experr2)) {

	if (expr != expret || (_errno != experr1 && _errno != experr2)) {

		ret = 1;

		if (experr2 == 0)

			llen += printf(" != (%d %s) ", expret, errorname(experr1));