RFR: 8352565: Add native method implementation of Reference.get() [v4]

Kim Barrett kbarrett at openjdk.org
Fri Apr 11 08:17:47 UTC 2025


On Wed, 2 Apr 2025 18:38:17 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Kim Barrett has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - remove timeout by using waitForReferenceProcessing
>>  - make ill-timed gc in non-concurrent case less likely
>>  - fix test package use
>
> test/hotspot/jtreg/gc/TestNativeReferenceGet.java line 137:
> 
>> 135:             }
>> 136:             checkQueue();       // One last check after refproc complete.
>> 137:         } catch (InterruptedException e) {
> 
> Rather than using Reference.remove with a timeout, I've changed this to use waitForReferenceProcessing.
> That removes false passes (from reference processing being slow to deliver) and also removes the delay
> until timeout for the passing case.

https://github.com/openjdk/jdk/pull/24527 is proposing a different way to access
`waitForReferenceProcessing`.  I'll wait for that.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24315#discussion_r2039040491


More information about the core-libs-dev mailing list