jdk/jdk: f3fb856749cf (original) (raw)
OpenJDK / jdk / jdk
changeset 15806:f3fb856749cf
8008546: Wrong G1ConfidencePercent results in GUARANTEE(VARIANCE() > -1.0) FAILED Reviewed-by: brutisso, johnc Contributed-by: vladimir.kempik@oracle.com
| poonam | |
|---|---|
| date | Thu, 21 Feb 2013 23:58:05 -0800 |
| parents | 6d3dcd34b5b9 |
| children | 7c4b3c798a0c |
| files | hotspot/src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp hotspot/src/share/vm/gc_implementation/g1/g1_globals.hpp |
| diffstat | 2 files changed, 10 insertions(+), 2 deletions(-)[+] [-] hotspot/src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp 10 hotspot/src/share/vm/gc_implementation/g1/g1_globals.hpp 2 |
line wrap: on
line diff
--- a/hotspot/src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp Wed Jul 05 18:41:33 2017 +0200 +++ b/hotspot/src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp Thu Feb 21 23:58:05 2013 -0800 @@ -267,7 +267,15 @@ double max_gc_time = (double) MaxGCPauseMillis / 1000.0; double time_slice = (double) GCPauseIntervalMillis / 1000.0; _mmu_tracker = new G1MMUTrackerQueue(time_slice, max_gc_time);
- uintx confidence_perc = G1ConfidencePercent;
- // Put an artificial ceiling on this so that it's not set to a silly value.
- if (confidence_perc > 100) {
- confidence_perc = 100;
- warning("G1ConfidencePercent is set to a value that is too large, "
"it's been updated to %u", confidence_perc);[](#l1.14)- }
- _sigma = (double) confidence_perc / 100.0; // start conservatively (around 50ms is about right) _concurrent_mark_remark_times_ms->add(0.05);
--- a/hotspot/src/share/vm/gc_implementation/g1/g1_globals.hpp Wed Jul 05 18:41:33 2017 +0200 +++ b/hotspot/src/share/vm/gc_implementation/g1/g1_globals.hpp Thu Feb 21 23:58:05 2013 -0800 @@ -32,7 +32,7 @@ #define G1_FLAGS(develop, develop_pd, product, product_pd, diagnostic, experimental, notproduct, manageable, product_rw) [](#l2.5) [](#l2.6)
- product(intx, G1ConfidencePercent, 50, [](#l2.7)
- product(uintx, G1ConfidencePercent, 50, [](#l2.8) "Confidence level for MMU/pause predictions") [](#l2.9) [](#l2.10) develop(intx, G1MarkingOverheadPercent, 0, [](#l2.11)