-
Notifications
You must be signed in to change notification settings - Fork 722
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
Support the new VirtualThread states #18421
Conversation
Related: eclipse-openj9#18183 New states added: - TIMED_PARKING = 7 - TIMED_PARKED = 8 - TIMED_PINNED = 9 Old states removed: - RUNNABLE_SUSPENDED = (RUNNABLE | SUSPENDED) - PARKED_SUSPENDED = (PARKED | SUSPENDED) Old states modified: - YIELDING = 7 -> 10 Signed-off-by: Babneet Singh <[email protected]>
This PR depends on ibmruntimes/openj9-openjdk-jdk21#59. This PR and ibmruntimes/openj9-openjdk-jdk21#59 will need to be merged together. fyi @tajila |
@hangshao0 Requesting your review. |
Are these state changes safe for jdk21? ie. are they JDK22+ |
Yes, it fixes a bug, which also exists in JDK21. See https://bugs.openjdk.org/browse/JDK-8312498 for more details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Jenkins test sanity.functional,sanity.openjdk aix jdk21 depends ibmruntimes/openj9-openjdk-jdk21#59 |
All PR builds have passed; no test failures are seen. There is an infra issue where the scripts cannot complete because |
Related: eclipse-openj9#18421 (comment) Signed-off-by: Babneet Singh <[email protected]>
Related: #18183
New states added:
Old states removed:
Old states modified: