Integrated: 8330576: ZYoungCompactionLimit should have range check

Jonas Norlinder duke at openjdk.org
Mon Apr 22 09:03:30 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`.

This pull request has now been integrated.

Changeset: c4f5c514
Author:    Jonas Norlinder <jonas at norlinder.nu>
Committer: Erik Österlund <eosterlund at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c4f5c5140a85818152c6e17414a36deabe491f7b
Stats:     1 line in 1 file changed: 1 ins; 0 del; 0 mod

8330576: ZYoungCompactionLimit should have range check

Reviewed-by: eosterlund, stefank

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

PR: https://git.openjdk.org/jdk/pull/18838


More information about the hotspot-gc-dev mailing list