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

JDK next adds JVM_VirtualThreadDisableSuspend #18663

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

JasonFengJ9
Copy link
Member

JDK next adds JVM_VirtualThreadDisableSuspend

This fixes JDK next abuild failure https://openj9-jenkins.osuosl.org/job/Build_JDKnext_aarch64_linux_OpenJDK/446/consoleFull

23:44:10  /home/jenkins/workspace/Build_JDKnext_aarch64_linux_OpenJDK/build/linux-aarch64-server-release/support/native/java.base/libjava/VirtualThread.o:(.data.rel+0x88): undefined reference to `JVM_VirtualThreadDisableSuspend'
23:44:10  collect2: error: ld returned 1 exit status

Signed-off-by: Jason Feng [email protected]

@pshipton pshipton merged commit 005be12 into eclipse-openj9:master Dec 20, 2023
@pshipton
Copy link
Member

I merged it already, but shouldn't it be specific to 23 and not 22? jdk22 has branched already.

@JasonFengJ9
Copy link
Member Author

@pshipton could you please review #18667?

@JasonFengJ9 JasonFengJ9 deleted the vtdisablesuspend branch December 21, 2023 12:55
babsingh added a commit to babsingh/openj9 that referenced this pull request Dec 22, 2023
Enable JVM_VirtualThreadDisableSuspend in JDK22. A PR has been
opened to move it to JDK22 in the RI. Eventually, we will see this
method in JDK22.

The above method is invoked in the VirtualThread class. So, the
assertion in the stub will trigger and cause failures in the builds.

Leaving the stub empty should be fine. See eclipse-openj9#18671 for more details.

Related:
- eclipse-openj9#18671
- eclipse-openj9#18667
- eclipse-openj9#18663

Signed-off-by: Babneet Singh <[email protected]>
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.

2 participants