Skip to content
This repository has been archived by the owner on Jun 18, 2024. It is now read-only.

More selftests #130

Merged
merged 3 commits into from
Jan 29, 2024
Merged

More selftests #130

merged 3 commits into from
Jan 29, 2024

Conversation

Byte-Lab
Copy link
Collaborator

Adds a couple of new selftest testcases:

  • Loading a "maximal" scheduler that has every callback defined
  • Loading schedulers repeatedly, concurrently in tight loops

@Byte-Lab Byte-Lab requested a review from htejun January 28, 2024 21:21
@Byte-Lab
Copy link
Collaborator Author

The test failure here is scx_qmap hitting the watchdog timeout. Given that these schedulers are running on shared github hosts in VMs, perhaps we should bump the timeout?

We recently added a "minimal" scheduler testcase that loaded the bare
minimum of fields to exercise the default codepaths for ext.c. Let's add
a "maximal" testcase which defines every single defined sched_ext_ops
callback and verifies that it's successfully able to be loaded.

Signed-off-by: David Vernet <void@manifault.com>
In #129, Andrea fixed a bug
where we trip over a NULL pointer deref by trying to load multiple
schedulers at a time. Let's add a stress test that tries to load
multiple schedulers in a tight loop at the same time.

Additionally, do a bit of cleanup in the build system to have testcases
take all BPF progs as dependencies. We don't really gain anything by
artificially coupling the name of testcases to the BPF progs they use.

Signed-off-by: David Vernet <void@manifault.com>
This testcase was mixing tabs and spaces, let's make it use tabs.

Signed-off-by: David Vernet <void@manifault.com>
@Byte-Lab Byte-Lab merged commit d0e0b1a into sched_ext Jan 29, 2024
2 checks passed
@Byte-Lab Byte-Lab deleted the more_selftests branch January 29, 2024 21:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants