RFR: 8333825: GenShen: Revert/Remove ShenandoahMaxEvacLABRatio
Y. Srinivas Ramakrishna
ysr at openjdk.org
Fri Jun 7 18:29:51 UTC 2024
ShenandoahMaxEvacLABRatio was intended as an experimental/research option and guardrail to manually bound the ceiling of PLABs used in Shenandoah and GenShen at the time that GenShen's LAB auto-sizing and allocation heuristics were somewhat nascent and immature. There is however a natural upper-bound of the heap region size for these local allocation buffers which prevent runaway growth, and the manual throttle has never had to be used, and the option thus appears somewhat vestigial. Removing it also reduces differences with upstream Shenandoah which should simplify GenShen's upstreaming reviews.
- Removed the option and its uses in Shen and GenShen code
- Adjusted logging to be consistent, and adjusted some comments in GenShen
- Tested with codepipeline, GC tests, and exercised the new log lines
-------------
Commit messages:
- Cosmetic change to debug level log.
- Try and minimize changes wrt jdk tip.
- Remove vestigial option ShenandoahMaxEvacLABRatio; move and adjust debug
Changes: https://git.openjdk.org/shenandoah/pull/446/files
Webrev: https://webrevs.openjdk.org/?repo=shenandoah&pr=446&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8333825
Stats: 53 lines in 3 files changed: 7 ins; 36 del; 10 mod
Patch: https://git.openjdk.org/shenandoah/pull/446.diff
Fetch: git fetch https://git.openjdk.org/shenandoah.git pull/446/head:pull/446
PR: https://git.openjdk.org/shenandoah/pull/446
More information about the shenandoah-dev
mailing list