Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files
8245026: PsAdaptiveSizePolicy::_old_gen_policy_is_ready is unused
Reviewed-by: sjohanss, pliden
  • Loading branch information
jaokim authored and kstefanj committed Nov 26, 2020
1 parent b1d1499 commit a8e3eabb6df7d8304083bfdaa4f8f0b33f768dab
Showing with 0 additions and 13 deletions.
  1. +0 −7 src/hotspot/share/gc/parallel/psAdaptiveSizePolicy.cpp
  2. +0 −6 src/hotspot/share/gc/parallel/psAdaptiveSizePolicy.hpp
@@ -60,7 +60,6 @@ PSAdaptiveSizePolicy::PSAdaptiveSizePolicy(size_t init_eden_size,
_live_at_last_full_gc(init_promo_size),
_change_old_gen_for_min_pauses(0),
_change_young_gen_for_maj_pauses(0),
_old_gen_policy_is_ready(false),
_young_gen_size_increment_supplement(YoungGenerationSizeSupplement),
_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
_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
// the mutator runs. Reset after the GC pause has been measured.
_major_timer.reset();
@@ -106,10 +106,6 @@ class PSAdaptiveSizePolicy : public AdaptiveSizePolicy {
// increase/decrease the young generation for major pause time
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
// growth percentage for the young gen eden and the
// 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();}

bool old_gen_policy_is_ready() { return _old_gen_policy_is_ready; }

// Given the amount of live data in the heap, should we
// perform a Full GC?
bool should_full_GC(size_t live_in_old_gen);

0 comments on commit a8e3eab

Please sign in to comment.