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

Add criu vmstates #15867

Merged
merged 1 commit into from
Sep 22, 2022
Merged

Add criu vmstates #15867

merged 1 commit into from
Sep 22, 2022

Conversation

tajila
Copy link
Contributor

@tajila tajila commented Sep 10, 2022

Add criu vmstates

Signed-off-by: Tobi Ajila atobia@ca.ibm.com

@tajila
Copy link
Contributor Author

tajila commented Sep 10, 2022

Jenkins test sanity xlinuxcriu jdk17

@tajila
Copy link
Contributor Author

tajila commented Sep 11, 2022

Jenkins test sanity xlinuxcriu jdk17

@tajila tajila marked this pull request as draft September 11, 2022 21:15
@tajila tajila marked this pull request as ready for review September 21, 2022 20:35
@tajila
Copy link
Contributor Author

tajila commented Sep 21, 2022

Jenkins test sanity xlinuxcriu jdk17

@tajila
Copy link
Contributor Author

tajila commented Sep 21, 2022

@gacholio please review

@tajila
Copy link
Contributor Author

tajila commented Sep 21, 2022

@gacholio
Copy link
Contributor

I think it would make sense to use the VMHelpers implementation in the interpreter.

@gacholio
Copy link
Contributor

I would also be tempted to change the helper name to setVMState since the APIs are not actually maintaining the stack.

@gacholio
Copy link
Contributor

jenkins compile win jdk8

@gacholio
Copy link
Contributor

jenkins test sanity xlinuxcriu jdk11

Signed-off-by: Tobi Ajila <atobia@ca.ibm.com>
@tajila
Copy link
Contributor Author

tajila commented Sep 22, 2022

jenkins compile win jdk8

@tajila
Copy link
Contributor Author

tajila commented Sep 22, 2022

jenkins test sanity xlinuxcriu jdk11

@gacholio gacholio merged commit a56e7b1 into eclipse-openj9:master Sep 22, 2022
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.

None yet

2 participants