RFR: 8347408: Create an internal method handle adapter for system calls with errno [v4]

Shaojin Wen swen at openjdk.org
Wed May 7 03:28:14 UTC 2025


On Tue, 6 May 2025 12:32:38 GMT, Per Minborg <pminborg at openjdk.org> wrote:

>> As we advance, converting older JDK code to use the relatively new FFM API requires system calls that can provide `errno` and the likes to explicitly allocate a MemorySegment to capture potential error states. This can lead to negative performance implications if not designed carefully and also introduces unnecessary code complexity.
>> 
>> Hence, this PR proposes adding a JDK internal method handle adapter that can be used to handle system calls with `errno`, `GetLastError`, and `WSAGetLastError`.
>> 
>> It relies on an efficient carrier-thread-local cache of memory regions to allide allocations.
>
> Per Minborg has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Address comments

src/java.base/share/classes/jdk/internal/foreign/CaptureStateUtil.java line 215:

> 213:         }
> 214:         // Last takes first
> 215:         perm[target.type().parameterCount()] = 0;

Suggestion:

        for (int i = 0; i < perm.length - 1; i++) {
            perm[i] = i;
        }
        // Last takes first
        perm[perm.length - 1] = 0;

Here you can use perm.length - 1

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25043#discussion_r2076701381


More information about the core-libs-dev mailing list