-
Notifications
You must be signed in to change notification settings - Fork 740
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 heap memory for ffi CaptureCallState #21197
Conversation
@theresa-m The changes LGTM. Could you please squash the commits? |
Done. |
jenkins test sanity.functional,sanity.openjdk amac jdk24 |
jenkins test sanity.functional,sanity.openjdk zlinux jdk21 |
JDK24 builds failed due to an unrelated issue, which has been resolved through #21224. Relaunching the JDK24 builds: jenkins test sanity.functional,sanity.openjdk amac jdk24 |
Note that Enable off-heap as default on Balanced GC is causing a number of failures in balanced modes. |
|
|
@theresa-m The failure in #21197 (comment) is related to this PR. The failures in #21197 (comment) are due to off-heap changes (a known issue: #21229). |
Please note off-heap is disabled by default back. It can be enabled for test purposes using |
Thanks. This was passing in my personal build, I'll see what I can reproduce. |
It seems that the test fails with |
I think the Jep454Tests_testLinkerFfi_DownCall_HeapArray is caused by the off-heap changes, I misunderstood the options when I tested this on Friday. In this grinder I retested including the test changes: theresa-m@357b1ab edit: I also rebuilt my changes with the latest master branch and the test passes: https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/48535/ |
Signed-off-by: Theresa Mammarella <[email protected]>
I rebased my branch with the changes that revert the off-heap by default changes. previously failing pr build links: |
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.
The previous builds and recent grinder runs confirm that these changes are ready for merging.
Previous PR builds:
@theresa-m Please backport these changes to the 0.51 release branch. |
Pr for 0.51: #21261 |
Starting in Java 24 the ffi CaptureCallState and Critical options can be used together. When heap memory use is allowed through the critical option the captured call state should be returned to the given heap memory segment.
Related: #21018
Related: #21019
OpenJDK foreign test suite (failure is related to #21018): https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/48403/