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

JDK21 adds JVM_VirtualThreadMount/JVM_VirtualThreadUnmount #16985

Merged
merged 1 commit into from
Mar 30, 2023

Conversation

JasonFengJ9
Copy link
Member

@JasonFengJ9 JasonFengJ9 commented Mar 21, 2023

JVM_VirtualThreadMount invokes JVM_VirtualThreadMountBegin if hide is true, otherwise JVM_VirtualThreadMountEnd;
Similarly for JVM_VirtualThreadUnmount;
Added stub methods UpcallLinker.makeFactory().

Related #16984
Required by ibmruntimes/openj9-openjdk-jdk#572

Signed-off-by: Jason Feng fengj@ca.ibm.com

runtime/j9vm/javanextvmi.cpp Outdated Show resolved Hide resolved
runtime/j9vm/j9vmnatives.xml Show resolved Hide resolved
runtime/j9vm/exports.cmake Show resolved Hide resolved
@JasonFengJ9 JasonFengJ9 force-pushed the jdk21jvmmethods branch 8 times, most recently from 969e266 to eb85c3c Compare March 27, 2023 13:25
JVM_VirtualThreadMount invokes JVM_VirtualThreadMountBegin if hide is
true, otherwise JVM_VirtualThreadMountEnd;
Similarly for JVM_VirtualThreadUnmount;
Added a stub method UpcallLinker.makeFactory().

Signed-off-by: Jason Feng <fengj@ca.ibm.com>
@keithc-ca
Copy link
Contributor

This looks good, pending changes in ibmruntimes/openj9-openjdk-jdk#572.

@keithc-ca
Copy link
Contributor

Now that ibmruntimes/openj9-openjdk-jdk#577 is merged, this should be safe to merge even without ibmruntimes/openj9-openjdk-jdk#572.

@keithc-ca
Copy link
Contributor

Jenkins compile alinux64 jdk20,jdknext

@keithc-ca keithc-ca merged commit db59af5 into eclipse-openj9:master Mar 30, 2023
@JasonFengJ9 JasonFengJ9 deleted the jdk21jvmmethods branch March 30, 2023 18:33
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