RFR: 8367131: Test com/sun/jdi/ThreadMemoryLeakTest.java fails on 32 bits

Leonid Mesnik lmesnik at openjdk.org
Tue Sep 9 23:57:19 UTC 2025


On Tue, 9 Sep 2025 22:44:40 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> test/jdk/com/sun/jdi/ThreadMemoryLeakTest.java line 31:
>> 
>>> 29:  * @comment Don't allow -Xcomp or -Xint as they impact memory useage and number of iterations.
>>> 30:  *          Require compressed oops because not doing so increases memory usage.
>>> 31:  * @requires (vm.compMode == "Xmixed") & ((vm.bits == 64 & vm.opt.final.UseCompressedOops) | vm.bits == 32)
>> 
>> would
>> `@requires (vm.compMode == "Xmixed") & (vm.bits == 32 | vm.opt.final.UseCompressedOops)`
>> works? (not sure if jtreg is going to evaluate coops mode lazy in this case)
>
> "bits == 64 & vm.opt.final.UseCompressedOops" is a common pattern used in testbase, so I just followed it.

this common pattern is good to run with specific coops and on 64-bit only, however in your case expression could be simplified.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27177#discussion_r2335121999


More information about the serviceability-dev mailing list