Skip to content

Enable SMP for selected tests, add SMP option to nix (Depends on #2275) - #2276

Closed
alfreb wants to merge 18 commits into
includeos:v0.16.0-releasefrom
alfreb:multicore-tests
Closed

Enable SMP for selected tests, add SMP option to nix (Depends on #2275)#2276
alfreb wants to merge 18 commits into
includeos:v0.16.0-releasefrom
alfreb:multicore-tests

Conversation

@alfreb

@alfreb alfreb commented Sep 7, 2024

Copy link
Copy Markdown
Contributor
  • Add smp parameter to nix default, shell, and overlay. Enable with --arg smp true.
  • Added multiple cores to paging, UDP and exceptions tests, add them to an SMP test set.
  • Enable boot logger to tests make it easier to verify that multiple cores come up.
  • Disable SMP for chainloader. It would be nice to test 32 bit multicore, but not in there.

This work depends on #2275

fwsGonzo and others added 17 commits September 7, 2024 18:02
- Copy patches from github.com/IncludeOS/musl, commit 4f331b7.
- Copy src/musl/nanosleep.ccp with clock_nanosleep from commit 9ceff78 in this repo.

Original author of both patches is fwsGonzo.
adds getrandom + stubs for fchdir, getrusage, sched_setaffinity, sched_setscheduler, sigaltstack
- stub faccessat, faccessat2, fchmodat2, pwritev2, preadv2, statx
- include time64 versions of calls to time functions.

As it looks like IncludeOS is already using 64 bit time on 32 bit,
the *_time64 calls just redirect to the existing calls. On 64 bit systems
there should be no change, as the regular calls are 64 bit.
This makes crash loop detection easier
Missing RDSEED/RDRAND will only emit warning, not terminate
Enable SMP by default.
- Add smp parameter to nix default, shell, and overlay. Enable with --arg smp true.
- Added multiple cores to paging, UDP and exceptions tests, add them to an SMP test set.
- Enable boot logger to tests make it easier toverify that multiple cores come up.
- Disable SMP for chainloader. It would be nice to test 32 bit multicore, but not in there.
@alfreb alfreb closed this Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants