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