stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8295269: G1: Improve slow startup due to predictor initialization

Browse Source 
Reviewed-by: iwalulya, sjohanss
This commit is contained in:
Thomas Schatzl 2024-10-25 15:24:55 +00:00
parent 5cbd578fbe
commit c202a2f7b2
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/hotspot/share/gc/g1/g1AnalyticsSequences.hpp
View File

@ -35,6 +35,7 @@ class G1Predictions;
// Container for TruncatedSeqs that need separate predictors by GC phase.
class G1PhaseDependentSeq {
TruncatedSeq _young_only_seq;
double _initial_value;
TruncatedSeq _mixed_seq;
NONCOPYABLE(G1PhaseDependentSeq);

7
src/hotspot/share/gc/g1/g1AnalyticsSequences.inline.hpp
View File

@ -34,6 +34,7 @@ bool G1PhaseDependentSeq::enough_samples_to_use_mixed_seq() const {
G1PhaseDependentSeq::G1PhaseDependentSeq(int length) :
_young_only_seq(length),
_initial_value(0.0),
_mixed_seq(length)
{ }
@ -42,7 +43,7 @@ TruncatedSeq* G1PhaseDependentSeq::seq_raw(bool use_young_only_phase_seq) {
}
void G1PhaseDependentSeq::set_initial(double value) {
_young_only_seq.add(value);
_initial_value = value;
}
void G1PhaseDependentSeq::add(double value, bool for_young_only_phase) {
@ -51,8 +52,12 @@ void G1PhaseDependentSeq::add(double value, bool for_young_only_phase) {
double G1PhaseDependentSeq::predict(const G1Predictions* predictor, bool use_young_only_phase_seq) const {
if (use_young_only_phase_seq || !enough_samples_to_use_mixed_seq()) {
if (_young_only_seq.num() == 0) {
return _initial_value;
}
return predictor->predict(&_young_only_seq);
} else {
assert(_mixed_seq.num() > 0, "must not ask this with no samples");
return predictor->predict(&_mixed_seq);
}
}