Re: [PATCH v1] perf tests sched: Avoid error in cleanup on loaded machines

[Arnaldo Carvalho de Melo <acme@kernel.org>]

On Thu, Jan 22, 2026 at 09:53:37AM -0800, Ian Rogers wrote:
> The stop_noploops function will kill the noploop processes that are
> running for 10 seconds. On a loaded machine they may have already
> terminated meaning the kill will return an error of no such
> process. This doesn't matter and so ignore the error to avoid the test
> terminating in the cleanup.
> 
> Fixes: 0e22c5ca44e6 ("perf test: Add sched latency and script shell tests")



Thanks, applied to perf-tools-next,

- Arnaldo

> Signed-off-by: Ian Rogers <irogers@google.com>
> ---
>  tools/perf/tests/shell/sched.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/perf/tests/shell/sched.sh b/tools/perf/tests/shell/sched.sh
> index b9b81eaf856e..b9637069adb1 100755
> --- a/tools/perf/tests/shell/sched.sh
> +++ b/tools/perf/tests/shell/sched.sh
> @@ -53,7 +53,7 @@ start_noploops() {
>  }
>  
>  cleanup_noploops() {
> -  kill "$PID1" "$PID2"
> +  kill "$PID1" "$PID2" || true
>  }
>  
>  test_sched_record() {
> -- 
> 2.52.0.457.g6b5491de43-goog