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

Parse softmx on CRIU restore side #18242

Merged
merged 1 commit into from Oct 17, 2023

Conversation

kangyining
Copy link
Contributor

Give the user functionality to specify softmx inside an option file for CRIU restore.

Note here we separate gcParseReconfigurableArguments() into gcParseSovereignArguments() and gcParseReconfigurableCommandLine() to distinguish SovereignArguments from general command line options.

Signed-off-by: Frank Kang frank.kang@ibm.com

@amicic
Copy link
Contributor

amicic commented Oct 10, 2023

we will probably need to clone some of the logic in original independentMemoryParameterVerification for restore path too
look at opt_XsoftmxSet to get a clue what kind of verification we do

@kangyining
Copy link
Contributor Author

independentMemoryParameterVerification

I think here our concern would be still if we need a complete memoryParameterTable as we discussed before

@amicic amicic closed this Oct 11, 2023
@amicic amicic reopened this Oct 11, 2023
@kangyining
Copy link
Contributor Author

we will probably need to clone some of the logic in original independentMemoryParameterVerification for restore path too look at opt_XsoftmxSet to get a clue what kind of verification we do

Should be resolved by the newest commit.

@kangyining kangyining self-assigned this Oct 11, 2023
@kangyining kangyining added comp:gc criu Used to track CRIU snapshot related work labels Oct 11, 2023
@amicic
Copy link
Contributor

amicic commented Oct 13, 2023

please, squash

Give the user functionality to specify softmx inside an option
file for CRIU restore.

Note here we separate gcParseReconfigurableArguments() into
gcParseSovereignArguments() and gcParseReconfigurableCommandLine() to
distinguish SovereignArguments from general command line options.

Signed-off-by: Frank Kang frank.kang@ibm.com
@amicic
Copy link
Contributor

amicic commented Oct 16, 2023

jenkins test sanity win, aix jdk17

@amicic
Copy link
Contributor

amicic commented Oct 16, 2023

jenkins test sanity win,aix jdk17

@amicic
Copy link
Contributor

amicic commented Oct 16, 2023

jenkins test sanity xlinuxcriu jdk17

@amicic
Copy link
Contributor

amicic commented Oct 17, 2023

seems like special xlinuxcriu build does not exist anymore

@amicic amicic merged commit ebf9ebe into eclipse-openj9:master Oct 17, 2023
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:gc criu Used to track CRIU snapshot related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants