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

[Goetz Lindenmaier]

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.

This pull request has now been integrated.

Changeset: 158f91a8
Author:    Goetz Lindenmaier <goetz at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/158f91a888020d1ea67682a07f6138737ea0cc67
Stats:     28 lines in 4 files changed: 20 ins; 6 del; 2 mod

8272651: G1 heap region info print order changed by JDK-8269914

Reviewed-by: rrich
Backport-of: f11e099a149adfecc474ba37276ec8672067d090

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

PR: https://git.openjdk.org/jdk17u-dev/pull/2261