Re: [PATCH 1/1] selftests/sched_ext: Fix unused-result warning for read()

[Andrea Righi <arighi@nvidia.com>]

On Sat, Feb 21, 2026 at 08:56:14PM +0800, Cheng-Yang Chou wrote:
> The read() call in run_test() triggers a warn_unused_result compiler
> warning, which breaks the build under -Werror.
> 
> Check the return value of read() and exit the child process upon
> failure to satisfy the compiler and prevent unexpected pipe errors.

To be really picky I'd rephrase this part as:

Check the return value of read() and exit the child process on failure to
satisfy the compiler and handle pipe read errors.

"Prevent" suggests avoiding the error, which isn't what the patch does.
But it also looks good as it is.

Reviewed-by: Andrea Righi <arighi@nvidia.com>

Thanks,
-Andrea

> 
> Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
> ---
>  tools/testing/selftests/sched_ext/init_enable_count.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/testing/selftests/sched_ext/init_enable_count.c b/tools/testing/selftests/sched_ext/init_enable_count.c
> index 82c71653977b..44577e30e764 100644
> --- a/tools/testing/selftests/sched_ext/init_enable_count.c
> +++ b/tools/testing/selftests/sched_ext/init_enable_count.c
> @@ -57,7 +57,8 @@ static enum scx_test_status run_test(bool global)
>  			char buf;
>  
>  			close(pipe_fds[1]);
> -			read(pipe_fds[0], &buf, 1);
> +			if (read(pipe_fds[0], &buf, 1) < 0)
> +				exit(1);
>  			close(pipe_fds[0]);
>  			exit(0);
>  		}
> -- 
> 2.48.1
>