[jdk17u-dev] RFR: 8269914: Factor out heap printing for G1 young and full gc

Paul Hohensee phh at openjdk.org
Thu Feb 29 22:20:54 UTC 2024


On Wed, 28 Feb 2024 13:32:10 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

> I backport this as prerequisite for [8291753](https://bugs.openjdk.org/browse/JDK-8291753) as proposed before in https://github.com/openjdk/jdk17u-dev/pull/1900.
> The backport has been tested by [releaseing it in SapMachine in 17.0.10](https://github.com/SAP/SapMachine/commit/4b0c2c7a070c7ad13f0da24a9418aea2883473af). This patch here is identical to the one in SapMachine. It will go to 17.0.12 giving enough time to settle.
> 
> Also, this will bring G1 code of 17 closer to head probably simplifying further backports.
> 
> This change solves two JBS issue, so I need approval for two.
> 
> Some simple resolves needed:
> 
> src/hotspot/share/gc/g1/g1CollectedHeap.cpp
> Basically trivial resolve due to context. One of the calls removed
> in the original change is not in 17.
> The patch brings a call to _g1h->rem_set()->print_coarsen_stats()
> which is not available in 17. I uncommented this call to make
> it build.
> 
> src/hotspot/share/gc/g1/g1FullGCScope.cpp
> Trivial resolve.
> Removes a destructor that calls several routines. 
> The destructor in 17 has one addtitional call that remains after removing the same ones as in the
> original change. Thus I kept the destructor with this single call.
> To make it build, I need to also keep the declaration of the destructor in the .hpp file.

Marked as reviewed by phh (Reviewer).

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

PR Review: https://git.openjdk.org/jdk17u-dev/pull/2250#pullrequestreview-1909975892


More information about the jdk-updates-dev mailing list