diff options
| author | David Carlier <devnexen@gmail.com> | 2026-03-13 08:17:55 +0300 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2026-03-13 20:00:45 +0300 |
| commit | 1d02346fec8d13b05e54296ddc6ae29b7e1067df (patch) | |
| tree | 490bde04a634484015914a96ad811712f8f14e5f /tools | |
| parent | bd377af0970164a4d12479bf36049619201be2f0 (diff) | |
| download | linux-1d02346fec8d13b05e54296ddc6ae29b7e1067df.tar.xz | |
selftests/sched_ext: Add missing error check for exit__load()
exit__load(skel) was called without checking its return value.
Every other test in the suite wraps the load call with
SCX_FAIL_IF(). Add the missing check to be consistent with the
rest of the test suite.
Fixes: a5db7817af78 ("sched_ext: Add selftests")
Signed-off-by: David Carlier <devnexen@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/testing/selftests/sched_ext/exit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/sched_ext/exit.c b/tools/testing/selftests/sched_ext/exit.c index ee25824b1cbe..b987611789d1 100644 --- a/tools/testing/selftests/sched_ext/exit.c +++ b/tools/testing/selftests/sched_ext/exit.c @@ -33,7 +33,7 @@ static enum scx_test_status run(void *ctx) skel = exit__open(); SCX_ENUM_INIT(skel); skel->rodata->exit_point = tc; - exit__load(skel); + SCX_FAIL_IF(exit__load(skel), "Failed to load skel"); link = bpf_map__attach_struct_ops(skel->maps.exit_ops); if (!link) { SCX_ERR("Failed to attach scheduler"); |