Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sel4test-sim can fail on GH runners #112

Closed
lsf37 opened this issue Jul 9, 2021 · 3 comments
Closed

sel4test-sim can fail on GH runners #112

lsf37 opened this issue Jul 9, 2021 · 3 comments
Labels
sel4test actions for sel4test and related repos

Comments

@lsf37
Copy link
Member

lsf37 commented Jul 9, 2021

On seL4/seL4#449 which does not change the binary (checked that by comparing said binaries), we saw the PC99_debug_clang_64 simulation test failing. It succeeded after re-running.

The specific test task that failed was Running test SCHED0021 (Test for pre-emption during running of many threads with equal prio), which depends on timer input.

It looks like that can be too unreliable on GitHub.

Just putting this information here in case we're seeing this more often. If we do we might either have to remove this test from simulation, or tweak parameters to make occurrence rare enough that it is not a nuisance.

@axel-h
Copy link
Member

axel-h commented Jul 9, 2021

I can confirm that we are also see SCHED0021 fail on various QEMU platform in our CI system. So it is not a Github issue.

@axel-h
Copy link
Member

axel-h commented Jul 9, 2021

Seems SCHED0021 is als suffering from a race condition. The threads are release and a yield() is called. It might happen that the round robin scheduler preempts the main thread after it released the other threads (by clearing the global variable), but before is can call yield(). In this case all threads can run 2 time slices and not just one.

@lsf37 lsf37 added the sel4test actions for sel4test and related repos label Jul 16, 2021
@lsf37
Copy link
Member Author

lsf37 commented Sep 7, 2021

This is now tracked in seL4/sel4test#42

@lsf37 lsf37 closed this as completed Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sel4test actions for sel4test and related repos
Projects
None yet
Development

No branches or pull requests

2 participants