diff mbox series

[-next] selftests/pidfd_test: Remove the erroneous ','

Message ID 20221101035602.40623-1-zhaogongyi@huawei.com (mailing list archive)
State Accepted
Headers show
Series [-next] selftests/pidfd_test: Remove the erroneous ',' | expand

Commit Message

Zhao Gongyi Nov. 1, 2022, 3:56 a.m. UTC 
Remove the erroneous ',', otherwise it might result in wrong output
and report:
 ...
 Bail out! (errno %d)
  test: Unexpected epoll_wait result (c=4208480, events=2)
 ...

Fixes: 740378dc7834 ("pidfd: add polling selftests")
Signed-off-by: Zhao Gongyi <zhaogongyi@huawei.com>
---
 tools/testing/selftests/pidfd/pidfd_test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.17.1

Comments

Shuah Khan Nov. 2, 2022, 9:11 a.m. UTC | #1 
On 10/31/22 21:56, Zhao Gongyi wrote:
> Remove the erroneous ',', otherwise it might result in wrong output
> and report:
>   ...
>   Bail out! (errno %d)
>    test: Unexpected epoll_wait result (c=4208480, events=2)
>   ...
> 
> Fixes: 740378dc7834 ("pidfd: add polling selftests")
> Signed-off-by: Zhao Gongyi <zhaogongyi@huawei.com>
> ---

Applied to linux-kselftest next for the next rc.

thanks,
-- Shuah
diff mbox series

Patch

diff --git a/tools/testing/selftests/pidfd/pidfd_test.c b/tools/testing/selftests/pidfd/pidfd_test.c
index d36654265b7a..e2dd4ed84984 100644
--- a/tools/testing/selftests/pidfd/pidfd_test.c
+++ b/tools/testing/selftests/pidfd/pidfd_test.c
@@ -413,7 +413,7 @@  static void poll_pidfd(const char *test_name, int pidfd)

 	c = epoll_wait(epoll_fd, events, MAX_EVENTS, 5000);
 	if (c != 1 || !(events[0].events & EPOLLIN))
-		ksft_exit_fail_msg("%s test: Unexpected epoll_wait result (c=%d, events=%x) ",
+		ksft_exit_fail_msg("%s test: Unexpected epoll_wait result (c=%d, events=%x) "
 				   "(errno %d)\n",
 				   test_name, c, events[0].events, errno);