RFR: 8342775: [Graal] java/util/concurrent/locks/Lock/OOMEInAQS.java fails OOME thrown from the UncaughtExceptionHandler [v2]

Viktor Klang vklang at openjdk.org
Tue Jan 21 10:13:51 UTC 2025


On Tue, 21 Jan 2025 03:05:06 GMT, Tom Rodriguez <never at openjdk.org> wrote:

>> Tom Rodriguez has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>> 
>>  - adjust OOMEInStampedLock.java too
>>  - Merge branch 'master' into tkr-oomeinaqs-ea
>>  - 8342775: [Graal] java/util/concurrent/locks/Lock/OOMEInAQS.java fails OOME thrown from the UncaughtExceptionHandler
>
> @AlanBateman Could you take a look at this one?

@tkrodriguez I'm a bit hesitant about the proposed change—it sounds to me like it relies too much on implementation details. If EA is decides to treat NTHREADS as effectively final (at some point) then this test will start failing again under Xcomp. What are the alternatives here?

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

PR Comment: https://git.openjdk.org/jdk/pull/21745#issuecomment-2604253649


More information about the core-libs-dev mailing list