Skip to content
This repository has been archived by the owner on Dec 13, 2024. It is now read-only.

Set interrupt field before native call #46

Merged
merged 1 commit into from
Oct 29, 2022

Conversation

tajila
Copy link
Member

@tajila tajila commented Oct 28, 2022

Set interrupt field before native call

There is a potential race if the interrupt field is set after the interrupt call where another thread may query interrupt status before the field is set.

Fixes: eclipse-openj9/openj9#16174

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

There is a potential race if the interrupt field is set after the
interrupt call where another thread may query interrupt status before
the field is set.

Fixes: eclipse-openj9/openj9#16174

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

tajila commented Oct 28, 2022

@JasonFengJ9 please review these changes

@tajila
Copy link
Member Author

tajila commented Oct 28, 2022

jenkins test sanity zlinux jdk19

Copy link
Member

@JasonFengJ9 JasonFengJ9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This restores the order of invoking interrupt0() after setting the field interrupted before #41

LGTM

@JasonFengJ9 JasonFengJ9 merged commit 9d79dbc into ibmruntimes:openj9 Oct 29, 2022
@JasonFengJ9
Copy link
Member

@tajila please cherry-pick for JDK next.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants