RFR: 8351996: Behavioral updates for ClassValue::remove [v2]
Viktor Klang
vklang at openjdk.org
Mon Apr 28 16:06:47 UTC 2025
On Mon, 28 Apr 2025 04:29:40 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Hello Chen, not a review of the code, but the tier1 failures in the GitHub actions jobs look related:
>>
>>
>> java.lang.StackOverflowError: Recursive initialization of class value
>> at java.base/java.lang.ClassValue$Entry.registerExtraThread(ClassValue.java:321)
>> at java.base/java.lang.ClassValue$ClassValueMap.startEntry(ClassValue.java:481)
>> at java.base/java.lang.ClassValue.getFromHashMap(ClassValue.java:196)
>> at java.base/java.lang.ClassValue.getFromBackup(ClassValue.java:183)
>> at java.base/java.lang.ClassValue.get(ClassValue.java:119)
>
> @jaikiran Since you have looked at this patch and used CountDownLatch frequently before, I wonder if you would like to review the use of concurrency utilities in this patch.
@liach I'll have a look as well.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24043#issuecomment-2835753236
More information about the core-libs-dev
mailing list