RFR: 8330849: Add test to verify memory usage with recursive locking [v4]

Leonid Mesnik lmesnik at openjdk.org
Tue Apr 23 19:03:31 UTC 2024


On Tue, 23 Apr 2024 16:19:52 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

>> Before recursive support has been added to lightweight locking, using recursive locking patterns would inflate lightweight locks to full monitors. In some scenarios (when churning lots of recursively locked objects), this could lead to excessive native memory usage. I'd like to add a test that verifies that this does not happen.
>> 
>> I verified that the new test fails reliably with an earlier commit that did not have LW recursive locking, yet, and passes with latest LW recursive locking.
>
> Roman Kennke has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Let the monitor object escape to prevent it from getting optimized away

Thanks for fixing. Looks good now.

Marked as reviewed by lmesnik (Reviewer).

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

Marked as reviewed by lmesnik (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18899#pullrequestreview-2018084432
PR Review: https://git.openjdk.org/jdk/pull/18899#pullrequestreview-2018084950


More information about the hotspot-runtime-dev mailing list