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

8245026: PsAdaptiveSizePolicy::_old_gen_policy_is_ready is unused

Browse Source 
Reviewed-by: sjohanss, pliden
This commit is contained in:
Joakim Nordström 2020-11-26 07:31:09 +00:00 committed by Stefan Johansson
parent b1d1499356
commit a8e3eabb6d
2 changed files with 0 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/hotspot/share/gc/parallel/psAdaptiveSizePolicy.cpp
View File

@ -60,7 +60,6 @@ PSAdaptiveSizePolicy::PSAdaptiveSizePolicy(size_t init_eden_size,
_live_at_last_full_gc(init_promo_size), _live_at_last_full_gc(init_promo_size),
_change_old_gen_for_min_pauses(0), _change_old_gen_for_min_pauses(0),
_change_young_gen_for_maj_pauses(0), _change_young_gen_for_maj_pauses(0),
_old_gen_policy_is_ready(false),
_young_gen_size_increment_supplement(YoungGenerationSizeSupplement), _young_gen_size_increment_supplement(YoungGenerationSizeSupplement),
_old_gen_size_increment_supplement(TenuredGenerationSizeSupplement) _old_gen_size_increment_supplement(TenuredGenerationSizeSupplement)
{ {
@ -164,12 +163,6 @@ void PSAdaptiveSizePolicy::major_collection_end(size_t amount_live,
// Update the amount live at the end of a full GC // Update the amount live at the end of a full GC
_live_at_last_full_gc = amount_live; _live_at_last_full_gc = amount_live;
// The policy does not have enough data until at least some major collections
// have been done.
if (_avg_major_pause->count() >= AdaptiveSizePolicyReadyThreshold) {
_old_gen_policy_is_ready = true;
}
// Interval times use this timer to measure the interval that // Interval times use this timer to measure the interval that
// the mutator runs. Reset after the GC pause has been measured. // the mutator runs. Reset after the GC pause has been measured.
_major_timer.reset(); _major_timer.reset();

6
src/hotspot/share/gc/parallel/psAdaptiveSizePolicy.hpp
View File

@ -106,10 +106,6 @@ class PSAdaptiveSizePolicy : public AdaptiveSizePolicy {
// increase/decrease the young generation for major pause time // increase/decrease the young generation for major pause time
int _change_young_gen_for_maj_pauses; int _change_young_gen_for_maj_pauses;
// Flag indicating that the adaptive policy is ready to use
bool _old_gen_policy_is_ready;
// To facilitate faster growth at start up, supplement the normal // To facilitate faster growth at start up, supplement the normal
// growth percentage for the young gen eden and the // growth percentage for the young gen eden and the
// old gen space for promotion with these value which decay // old gen space for promotion with these value which decay
@ -310,8 +306,6 @@ class PSAdaptiveSizePolicy : public AdaptiveSizePolicy {
} }
float major_collection_slope() { return _major_collection_estimator->slope();} float major_collection_slope() { return _major_collection_estimator->slope();}
bool old_gen_policy_is_ready() { return _old_gen_policy_is_ready; }
// Given the amount of live data in the heap, should we // Given the amount of live data in the heap, should we
// perform a Full GC? // perform a Full GC?
bool should_full_GC(size_t live_in_old_gen); bool should_full_GC(size_t live_in_old_gen);