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

Skip halt check after explict GC, with external exclusive VM access #15271

Merged

Conversation

amicic
Copy link
Contributor

@amicic amicic commented Jun 8, 2022

If caller of explicit GC (for example RAS) already obtained exclusive VM
access, checking if the thread needs to halt after completing the GC
could lead to a deadlock. For example exclusive VM access could prevent
an inspector thread to proceed, while also this thread just blocked
expecting to be inspected.

The fix is to make an exception and skip the halt check if we know that
we hold exclusive VM access.

Signed-off-by: Aleksandar Micic amicic@ca.ibm.com

If caller of explicit GC (for example RAS) already obtained exclusive VM
access, checking if we the thread needs to halt after completing the GC
could lead to a deadlock. For example exclusive VM access could prevent
an inspector thread to proceed, while also this thread just blocked
expecting to be inspected.

The fix is to make an excpetion and skip the halt check if we know that
we hold exclusive VM access.

Signed-off-by: Aleksandar Micic <amicic@ca.ibm.com>
@amicic amicic requested a review from gacholio June 8, 2022 19:13
@gacholio
Copy link
Contributor

gacholio commented Jun 8, 2022

jenkins test sanity zlinux jdk11

@amicic
Copy link
Contributor Author

amicic commented Jun 8, 2022

Fixes both: #15243 and #15258

@gacholio gacholio merged commit 26653e2 into eclipse-openj9:master Jun 8, 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.

3 participants