On 23/06/20 22:44, Christian Brauner wrote:
>> ret = sys_pidfd_send_signal(pidfd, 0, NULL, 0);
>> if (ret < 0) {
>> - if (errno == ENOSYS)
>> - ksft_exit_skip(
>> + if (errno == ENOSYS) {
>> + ksft_test_result_skip(
>> "%s test: pidfd_send_signal() syscall not supported\n",
>> test_name);
> If pidfd_send_signal() is not supported, you're falling through and then
> you're reporting:
>
> ok 5 # SKIP pidfd_send_signal check for support test: pidfd_send_signal() syscall not supported
> ok 6 pidfd_send_signal check for support test: pidfd_send_signal() syscall is supported. Tests can be executed
You're right, this needs a "return".
Paolo
