On Tue, 21 Jan 2025 03:05:06 GMT, Tom Rodriguez <ne...@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