[jdk8u-dev] RFR: 8352942: jdk/jfr/startupargs/TestMemoryOptions.java fails with 32-bit build

[Paul Hohensee]

On Wed, 26 Mar 2025 08:53:12 GMT, KIRIYAMA Takuya <duke at openjdk.org> wrote:

> Hi All,
> I would like to add this bug fix for the bug in jdk/jfr/startupargs/TestMemoryOptions.java. This test contains 24 test cases and fails the "ThreadBufferSizeExceedMemorySize" case.
> The cause of this bug is the memory allocation issue, which occurs only on 32-bit Server VM, not on Client VM or 64-bit JDK. The failure happens because Server VM's default heap size reduces available memory space, causing JFR to fail memory allocation.
> To resolve this issue, -Xmx256M is explicitly set, matching the Client VM default heap size, ensuring sufficient memory space remains available for JFR.
> I believe that this test verifies that the combination of memory options for JFR is valid or invalid and that the MaxHeapSize setting does not affect the verification.
> Change has been verified locally, test-fix only, no risk.
> 
> Would someone please review this fix?

Please check later JDKs (11, 17, 21, 24, tip). If it happens in all of them, please fix it in tip and then backport to earlier releases. You can use the existing JBS issue, just change version info.

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

PR Review: https://git.openjdk.org/jdk8u-dev/pull/641#pullrequestreview-2717694242