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

[Jaikiran Pai]

On Fri, 31 Jan 2025 22:18:32 GMT, Tom Rodriguez <never at openjdk.org> wrote:

>> Deoptimization with escape analysis can fail when trying to rematerialize objects as described in JDK-8227309.  In this test this can happen in Xcomp mode in the framework of the test resulting in a test failure.  Making the number of threads non-final avoids scalar replacement and thus the OOM during deopt.
>
> 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 five additional commits since the last revision:
> 
>  - Disable test for Graal with Xcomp
>  - Merge branch 'master' into tkr-oomeinaqs-ea
>  - 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

Hello Tom, the change to  the `OOMEInAQS.java` test to skip it when running the combination of Graal and `-Xcomp` looks fine to me. I see that the `OOMEInStampedLock.java` has been updated too to do the same. The original bug report doesn't mention this test. Is the change to `OOMEInStampedLock.java` done as precaution against the same issue?

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

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