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

Ignore -Xgc:enableArrayletDoubleMapping silently #18109

Merged
merged 1 commit into from Sep 11, 2023

Conversation

dmitripivkine
Copy link
Contributor

Arraylet Double Mapping feature is not supported for all platforms. Current behaviour is return "unsupported option error". This prevents to use the same Java command line across platforms.
New behaviour is to ignore -Xgc:enableArrayletDoubleMapping and -Xgc:disbleArrayletDoubleMapping silently on platforms where this feature is not supported.

Arraylet Double Mapping feature is not supported for all platforms.
Current behaviour is return "unsupported option error". This prevents to
use the same Java command line across platforms.
New behaviour is to ignore -Xgc:enableArrayletDoubleMapping and
-Xgc:disbleArrayletDoubleMapping silently on platforms where this
feature is not supported.

Signed-off-by: Dmitri Pivkine <Dmitri_Pivkine@ca.ibm.com>
@amicic
Copy link
Contributor

amicic commented Sep 11, 2023

Ideally we should let isArrayletDoubleMapRequested be changed by cmdline option but leaving isArrayletDoubleMapAvailable false for unsupported platfroms, since from VGC it would be clear if it was supported or not (we report both of these flags there).

But since it would be a larger scope of changes, and this feature is anyway to be deprecated eventually, I'm ok with this quick fix.

@amicic
Copy link
Contributor

amicic commented Sep 11, 2023

jenkins compile win jdk8

@amicic
Copy link
Contributor

amicic commented Sep 11, 2023

jenkins compile aix jdk17

@amicic amicic merged commit 5c84d87 into eclipse-openj9:master Sep 11, 2023
5 checks passed
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.

None yet

2 participants