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