RFR: 8330576: ZYoungCompactionLimit should have range check
Erik Österlund
eosterlund at openjdk.org
Thu Apr 18 12:46:02 UTC 2024
On Thu, 18 Apr 2024 12:18:11 GMT, Jonas Norlinder <duke at openjdk.org> wrote:
> Currently any double is allowed for `ZYoungCompactionLimit`. This is wrong for two reasons, it should express the percentage of fragmentation on a page, so only [0,100] would be valid numbers. Moreover, if a user inputs a negative or larger than 100 number the method `ZHeuristics::significant_young_overhead` would return improper values leading to broken heuristics.
>
> This patch adds a range check for `ZYoungCompactionLimit`.
Looks good.
-------------
Marked as reviewed by eosterlund (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/18838#pullrequestreview-2008782471
More information about the hotspot-gc-dev
mailing list