[foreign-memaccess+abi] RFR: 8264386: LibraryLookup should be more friendly with implicit unloading [v3]

[Jorn Vernee]

On Mon, 29 Mar 2021 20:51:59 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Quoting from the PR:
>> 
>> Dereferencing a segment from a LibraryLookup is possible, in the following form:
>> 
>> LibraryLookup libLookup = ...
>> MemorySegment segment = libLookup.lookup("foo").asSegmentRestricted(....)
>> 
>> The above code has a problem: there is no link between the produced segment and the original library lookup - meaning that, in principle, it's possible for the library to be unloaded when dereference occurs.
>> 
>> After having discussed many alternatives with Jorn, the best way to address this conundrum would be to add an overload of `LibraryLookup::lookup` which takes a lyout and returns a segment.
>> 
>> As part of this change I have also removed the LibraryLookup.Symbol API which seems unnecessary - now you can lookup an address or a segment. Of course, looking up a segment is creating a restricted segment (so it should be treated as unsafe).
>> 
>> I've added a test to make sure that the new API works as expected.
>
> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Address review comments

Marked as reviewed by jvernee (Committer).

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

PR: https://git.openjdk.java.net/panama-foreign/pull/480