On Wed, 3 Dec 2025 14:59:10 GMT, Joel Sikström <[email protected]> wrote:

> Hello,
> 
> If the initial heap size is set too low in 
> serviceability/sa/ClhsdbScanOops.java, a GC migh run, which will interfere 
> with the test and might cause it to fail. 
> 
> The test is scanning the oops in a region of the heap, and after a GC that 
> region appears to be empty, so the output that the test expects is not 
> present. Running the test with a larger explicit InitialHeapSize gives enough 
> headroom to not run a GC.
> 
> Testing: 
> * serviceability/sa/ClhsdbScanOops.java originally failed when run with 
> `-XX:InitialRAMPercentage=0` (which is the new default). We now explicitly 
> set `-XX:InitialHeapSize=100M`. I've rerun the test 10 times with Serial and 
> Parallel for each test and they all pass.

This pull request has now been integrated.

Changeset: 786833cd
Author:    Joel Sikström <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/786833cd1bf8eda1cef25da392a055f4eb371abf
Stats:     7 lines in 2 files changed: 3 ins; 3 del; 1 mod

8373022: serviceability/sa/ClhsdbScanOops.java assumes no GC should occur

Reviewed-by: cjplummer, stefank, ayang, tschatzl

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

PR: https://git.openjdk.org/jdk/pull/28637

Reply via email to