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

Goetz Lindenmaier goetz at openjdk.org
Fri Mar 1 12:07:58 UTC 2024


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.

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

Commit messages:
 - Backport f11e099a149adfecc474ba37276ec8672067d090

Changes: https://git.openjdk.org/jdk17u-dev/pull/2261/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=2261&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8272651
  Stats: 28 lines in 4 files changed: 20 ins; 6 del; 2 mod
  Patch: https://git.openjdk.org/jdk17u-dev/pull/2261.diff
  Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/2261/head:pull/2261

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


More information about the jdk-updates-dev mailing list