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

Added getcpu, sched_yield, setaffinity syscalls to Vanadis. Added OpenMP examples. #2442

Open
wants to merge 3 commits into
base: devel
Choose a base branch
from

Conversation

PlatinumCD
Copy link

Pull Request Description

This PR adds several new features and updates to the Vanadis component in SST:

New System Calls

  • Added support for:
    • sched_yield
    • sched_setaffinity
    • getcpu

Code Updates

  • Updated headers to include new syscalls and fixed inconsistencies in naming.

Testing Additions

  • Introduced OpenMP test examples to demonstrate and validate new syscall functionality:
  • Added test scripts and binary files for easier validation.

No breaking changes are expected.

@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

1 similar comment
@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@gvoskuilen
Copy link
Contributor

Please rebase so that all of the commits are your new commits rather than old merges from devel. See https://github.com/sstsimulator/sst-core/blob/master/CONTRIBUTING.md#forked-development for one way to avoid the extra commits when branching.

@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@PlatinumCD
Copy link
Author

@gvoskuilen I believe I have rebased correctly. Let me know what you think. Thank you.

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