[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