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

CRIU Support Documentation Updates for 0.40 #1092

Closed
Tracked by #16853
dsouzai opened this issue Apr 28, 2023 · 9 comments · Fixed by #1127
Closed
Tracked by #16853

CRIU Support Documentation Updates for 0.40 #1092

dsouzai opened this issue Apr 28, 2023 · 9 comments · Fixed by #1127
Assignees

Comments

@dsouzai
Copy link
Contributor

dsouzai commented Apr 28, 2023

Overview:

Release target:
Eclipse OpenJ9 0.40.0*

Applies to the following JDK versions:
JDKs that implement OpenJ9 CRIU Support

Applies to the following platforms:
Linux only on x96, P, Z

@dsouzai
Copy link
Contributor Author

dsouzai commented Apr 28, 2023

I'm opening this issue mainly as a placeholder to catch all changes in the behaviour of the OpenJ9 CRIU Support feature post 0.38.

@Sreekala-Gopakumar
Copy link
Contributor

@dsouzai - Any further updates in this issue? Thanks!

@dsouzai
Copy link
Contributor Author

dsouzai commented Jun 13, 2023

No other updates, this is the only change from the JIT side for 0.40. @JasonFengJ9 may have other updates though for the VM.

@JasonFengJ9
Copy link
Member

There are a couple of VM updates in progress, one is CRIU throws JVMCRIUException in single threaded mode if parks no timeout, the other one is to adjust VirtualThread.DEFAULT_SCHEDULER.PARALLELISM for carrier thread scheduling according to Runtime.availableProcessors().
Will provide details after the PR approval/merge.

@Sreekala-Gopakumar
Copy link
Contributor

There are a couple of VM updates in progress, one is CRIU throws JVMCRIUException in single threaded mode if parks no timeout, the other one is to adjust VirtualThread.DEFAULT_SCHEDULER.PARALLELISM for carrier thread scheduling according to Runtime.availableProcessors().
Will provide details after the PR approval/merge.

Is this change approved and merged as yet (I can see that this PR is not closed)?

@JasonFengJ9
Copy link
Member

Is this change approved and merged as yet (I can see that this PR is not closed)?

Not yet, also opened

@Sreekala-Gopakumar
Copy link
Contributor

Sreekala-Gopakumar commented Jun 20, 2023

Is this change approved and merged as yet (I can see that this PR is not closed)?

Not yet, also opened

ok. I will take this up after it is merged (as per the Contribution guidelines).

Any update on this?

@Sreekala-Gopakumar
Copy link
Contributor

@JasonFengJ9 - The following issues seem more like API doc changes. So, I can skip these since I am documenting user docs?

eclipse-openj9/openj9#17618 and eclipse-openj9/openj9#17573

@JasonFengJ9
Copy link
Member

So, I can skip these since I am documenting user docs?

Right

Sreekala-Gopakumar added a commit to Sreekala-Gopakumar/openj9-docs that referenced this issue Jul 11, 2023
eclipse-openj9#1092 (comment)

Updated the topic with the information about when the VM client connects to a JITServer in the context of CRIU

Closes eclipse-openj9#1092
Signed-off-by: Sreekala Gopakumar <[email protected]>
Sreekala-Gopakumar added a commit to Sreekala-Gopakumar/openj9-docs that referenced this issue Jul 13, 2023
eclipse-openj9#1092 (comment)

Updated the topic with the information about when the VM client connects to a JITServer in the context of CRIU

Closes eclipse-openj9#1092
Signed-off-by: Sreekala Gopakumar <[email protected]>
Sreekala-Gopakumar added a commit to Sreekala-Gopakumar/openj9-docs that referenced this issue Jul 13, 2023
eclipse-openj9#1092 (comment)

Updated the topic with the information about when the VM client connects to a JITServer in the context of CRIU

Closes eclipse-openj9#1092
Signed-off-by: Sreekala Gopakumar <[email protected]>
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 a pull request may close this issue.

3 participants