[jdk17u-dev] RFR: 8272651: G1 heap region info print order changed by JDK-8269914

[Richard Reingruber]

On Fri, 1 Mar 2024 12:03:58 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.
> 
> src/hotspot/share/gc/g1/g1CollectedHeap.cpp
> 
> Some lines removed are not in 17: 
> -  set_young_collection_default_active_worker_threads();
> some later:
> -    G1YoungGCNotifyPauseMark npm;
> 
> These are added again a bit further down.
> 
> As adding them would modify the behaviour, I only added
> them uncommented.
> 
> test/hotspot/jtreg/gc/g1/TestEagerReclaimHumongousRegionsLog.java
> The file already has the correct coding. Obviously these lines
> were changed after 17 and later reverted again.

Marked as reviewed by rrich (Reviewer).

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

PR Review: https://git.openjdk.org/jdk17u-dev/pull/2261#pullrequestreview-1915397895