[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Integrated: 8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC
From:       Chris Plummer <cjplummer () openjdk ! java ! net>
Date:       2022-01-29 21:39:09
Message-ID: RvSj_hEJE0WC2FVbUpxdq4GjPZI69hI-4c7AgR2MWg4=.95e1111f-362e-4894-a441-e573ceeb4f7b () github ! com
[Download RAW message or body]

On Wed, 26 Jan 2022 20:03:21 GMT, Chris Plummer <cjplummer@openjdk.org> wrote:

> This test is failing in the loom repo when using -Xcomp. The reason is because loom \
> introduced doing a full GC in the codecache sweeper, which causes the large array \
> that the test allocates to be GC'd before the heap dump is done. The fix is to make \
> the large array static rather than a local variable. 
> I'm choosing to fix this in the jdk repo rather than the loom repo since it is a \
> latent bug that theoretically could occur even without the loom changes, and also \
> to help reduce the amount of changes to be reviewed when loom is integrated into \
> jdk.

This pull request has now been integrated.

Changeset: be9f984c
Author:    Chris Plummer <cjplummer@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/be9f984caec32c3fe1deef30efe40fa115409ca0
                
Stats:     4 lines in 1 file changed: 3 ins; 0 del; 1 mod

8280553: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java can fail if GC \
occurs

Reviewed-by: alanb, amenkov

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

PR: https://git.openjdk.java.net/jdk/pull/7236


[prev in list] [next in list] [prev in thread] [next in thread]