RFR: 8338482: com/sun/jdi/ThreadMemoryLeakTest.java requires that compressed oops are enabled
Kevin Walls
kevinw at openjdk.org
Tue Aug 20 09:16:56 UTC 2024
On Fri, 16 Aug 2024 05:41:01 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
> com/sun/jdi/ThreadMemoryLeakTest.java purposely runs with a small heap so it is easier to detect a memory leak with an OOME. Running without compressed oops uses more memory leading to an OOME even though there is no leak. We should require that this test be run only with compressed oops.
>
> Tested by running just this test without compress oops and verifying that the test is not run, and also running with compressed oops and verifying that it is run. I also did an `-Xcomp` run to verify that it still is not run with `-Xcomp`.
Looks good.
-------------
Marked as reviewed by kevinw (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/20606#pullrequestreview-2247436712
More information about the serviceability-dev
mailing list