Re: [PATCH] selftests: kselftest: Use strerror() on nolibc

From: Shuah Khan
Date: Wed Sep 11 2024 - 13:41:56 EST


On 9/11/24 09:44, Thomas Weißschuh wrote:
Hi Shuah,

On 2024-09-11 09:36:50+0000, Shuah Khan wrote:
On 9/10/24 22:42, zhangjiao2 wrote:
From: zhang jiao <zhangjiao2@xxxxxxxxxxxxxxxxxxxx>

Nolibc gained an implementation of strerror() recently.
Use it and drop the ifndef.

Signed-off-by: zhang jiao <zhangjiao2@xxxxxxxxxxxxxxxxxxxx>
---
tools/testing/selftests/kselftest.h | 8 --------
1 file changed, 8 deletions(-)

diff --git a/tools/testing/selftests/kselftest.h b/tools/testing/selftests/kselftest.h
index e195ec156859..29fedf609611 100644
--- a/tools/testing/selftests/kselftest.h
+++ b/tools/testing/selftests/kselftest.h
@@ -373,15 +373,7 @@ static inline __noreturn __printf(1, 2) void ksft_exit_fail_msg(const char *msg,
static inline __noreturn void ksft_exit_fail_perror(const char *msg)
{
-#ifndef NOLIBC
ksft_exit_fail_msg("%s: %s (%d)\n", msg, strerror(errno), errno);
-#else
- /*
- * nolibc doesn't provide strerror() and it seems
- * inappropriate to add one, just print the errno.
- */
- ksft_exit_fail_msg("%s: %d)\n", msg, errno);
-#endif
}
static inline __noreturn void ksft_exit_xfail(void)

Adding nolibc maintainers for review.

Willy and Thomas, please review.

Acked-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx>

I did the same for another kselftests function when introducing
strerror(). This one was apparently missed or didn't exist yet.




Thank you. Applied to linux-kselftest next for Linux 6.12-rc1.

thanks,
-- Shuah