Skip to content

[FFI/Test] Fix the wrong invocation on the native function in downcall #20080

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

Conversation

ChengJin01
Copy link

The PR resolves the test issue with the wrong invocation in downcall
to ensure the test code in java calls the correct native function in
native.

Fixes: #20048

Signed-off-by: ChengJin01 [email protected]

The PR resolves the test issue with the wrong invocation in downcall
to ensure the test code in java calls the correct native function in
native.

Fixes: eclipse-openj9#20048

Signed-off-by: ChengJin01 <[email protected]>
@ChengJin01 ChengJin01 added comp:test project:panama Used to track Project Panama related work labels Aug 28, 2024
@ChengJin01
Copy link
Author

ChengJin01 commented Aug 28, 2024

The test fix was verified on all supported platforms at #20048 (comment), which are simply changes from returnStruct1 to returnStruct2 as called by returnStruct2 specific test suites in java.

@ChengJin01
Copy link
Author

Reviewer: @pshipton
FYI: @tajila

@ChengJin01 ChengJin01 requested a review from pshipton August 28, 2024 23:44
@pshipton pshipton merged commit aed7b7d into eclipse-openj9:master Aug 29, 2024
2 checks passed
@pshipton
Copy link
Member

I assume we need a PR created for 0.48 as well?

ChengJin01 pushed a commit to ChengJin01/openj9 that referenced this pull request Aug 29, 2024
The PR resolves the test issue with the wrong invocation in downcall
to ensure the test code in java calls the correct native function in
native.

Cherry pick of eclipse-openj9#20080 for 0.48

Signed-off-by: ChengJin01 <[email protected]>
@ChengJin01
Copy link
Author

I assume we need a PR created for 0.48 as well?

PR is created at #20082

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:test project:panama Used to track Project Panama related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Jep454Tests_testLinkerFfi_DownCall_0_FAILED Segmentation error vmState=0x00000000
2 participants