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

(0.40) Add checkpoint delay when clinit is occuring #17652

Merged
merged 3 commits into from Jun 23, 2023

Conversation

tajila
Copy link
Contributor

@tajila tajila commented Jun 23, 2023

No description provided.

The is PR addresses cases where a class is being initialized on a non-checkpoint thread while a checkpoint occurs. This scenario may lead to cases where a deadlock occurs when when attempting to initialize a class during a pre/post checkpoint hook.

This PR address the issue by waiting for threads currently initializing a class to complete before taking a checkpoint up to 1ms. After the timeout, an exception will be thrown (unless -XX:-J9VM_CRIU_IS_THROW_ON_DELAYED_CHECKPOINT_ENABLED is specified)

Also, this PR adds more detailed messages to checkpoint dir exceptions.

Fixes: eclipse-openj9#15806

Signed-off-by: Tobi Ajila <atobia@ca.ibm.com>
Related: eclipse-openj9#15806

Signed-off-by: Tobi Ajila <atobia@ca.ibm.com>
Signed-off-by: Tobi Ajila <atobia@ca.ibm.com>
@tajila tajila requested a review from babsingh June 23, 2023 18:16
@babsingh babsingh merged commit c50c466 into eclipse-openj9:v0.40.0-release Jun 23, 2023
2 checks passed
@babsingh
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants