@@ -11,51 +11,24 @@
#include "enq_select_cpu_fails.bpf.skel.h"
#include "scx_test.h"
-static enum scx_test_status setup(void **ctx)
-{
- struct enq_select_cpu_fails *skel;
-
- skel = enq_select_cpu_fails__open();
- SCX_FAIL_IF(!skel, "Failed to open");
- SCX_ENUM_INIT(skel);
- SCX_FAIL_IF(enq_select_cpu_fails__load(skel), "Failed to load skel");
-
- *ctx = skel;
-
- return SCX_TEST_PASS;
-}
-
static enum scx_test_status run(void *ctx)
{
- struct enq_select_cpu_fails *skel = ctx;
- struct bpf_link *link;
+ struct enq_select_cpu_fails *skel;
- link = bpf_map__attach_struct_ops(skel->maps.enq_select_cpu_fails_ops);
- if (!link) {
- SCX_ERR("Failed to attach scheduler");
+ skel = enq_select_cpu_fails__open_and_load();
+ if (skel) {
+ enq_select_cpu_fails__destroy(skel);
+ SCX_ERR("This program should fail to load");
return SCX_TEST_FAIL;
}
- sleep(1);
-
- bpf_link__destroy(link);
-
return SCX_TEST_PASS;
}
-static void cleanup(void *ctx)
-{
- struct enq_select_cpu_fails *skel = ctx;
-
- enq_select_cpu_fails__destroy(skel);
-}
-
struct scx_test enq_select_cpu_fails = {
.name = "enq_select_cpu_fails",
.description = "Verify we fail to call scx_bpf_select_cpu_dfl() "
"from ops.enqueue()",
- .setup = setup,
.run = run,
- .cleanup = cleanup,
};
REGISTER_SCX_TEST(&enq_select_cpu_fails)
After kfunc filters support struct_ops context information, SCX programs that call incorrect context-sensitive kfuncs will fail to load. This patch updates the enq_select_cpu_fails test case to adapt to the failed load situation. Signed-off-by: Juntong Deng <juntong.deng@outlook.com> --- .../sched_ext/enq_select_cpu_fails.c | 37 +++---------------- 1 file changed, 5 insertions(+), 32 deletions(-)