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

Disable OMR_THR_YIELD_ALG on MacOS #3921

Merged
merged 1 commit into from
Jan 14, 2019

Conversation

babsingh
Copy link
Contributor

@babsingh babsingh commented Dec 4, 2018

OMR_THR_YIELD_ALG enables support for -Xthr:[cfsYield|noCfsYield].
[cfsYield|noCfsYield] should only be used with Completely Fair Scheduler
(CFS).

MacOS schedulers can be classified as multilevel feedback queue
schedulers. MacOS doesn't use CFS. Thus, disabling OMR_THR_YIELD_ALG on
MacOS.

Signed-off-by: Babneet Singh [email protected]

OMR_THR_YIELD_ALG enables support for -Xthr:[cfsYield|noCfsYield].
[cfsYield|noCfsYield] should only be used with Completely Fair Scheduler
(CFS).

MacOS schedulers can be classified as multilevel feedback queue
schedulers. MacOS doesn't use CFS. Thus, disabling OMR_THR_YIELD_ALG on
MacOS.

Signed-off-by: Babneet Singh <[email protected]>
@babsingh
Copy link
Contributor Author

babsingh commented Dec 4, 2018

fyi - @DanHeidinga
related to #36

@babsingh
Copy link
Contributor Author

@DanHeidinga @pshipton can this be merged?

@pshipton
Copy link
Member

jenkins test sanity osx jdk8,jdk11

@pshipton pshipton merged commit 4702476 into eclipse-openj9:master Jan 14, 2019
dnakamura added a commit to dnakamura/openj9 that referenced this pull request May 27, 2020
Since its only enabled on linux platforms, move it from common.cmake to
linux.cmake

See eclipse-openj9#3921

Signed-off-by: Devin Nakamura <[email protected]>
@babsingh babsingh deleted the disable_cfs_support_osx branch August 17, 2020 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants