archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-26 22:34:27 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8237375: SimpleThresholdPolicy misses CounterDecay timestamp initialization

Browse source 
Reviewed-by: simonis, dholmes
This commit is contained in:
Martin Doerr 2020-01-23 13:55:10 +01:00
parent a0882bf4cc
commit da7cebabb2
1 changed files with 45 additions and 41 deletions
Download patch file Download diff file Expand all files Collapse all files

86
src/hotspot/share/compiler/compilationPolicy.cpp
View file

@ -190,6 +190,50 @@ CompileTask* CompilationPolicy::select_task_helper(CompileQueue* compile_queue)
return compile_queue->first(); return compile_queue->first();
} }
//
// CounterDecay for SimpleCompPolicy
//
// Iterates through invocation counters and decrements them. This
// is done at each safepoint.
//
class CounterDecay : public AllStatic {
static jlong _last_timestamp;
static void do_method(Method* m) {
MethodCounters* mcs = m->method_counters();
if (mcs != NULL) {
mcs->invocation_counter()->decay();
}
}
public:
static void decay();
static bool is_decay_needed() {
return nanos_to_millis(os::javaTimeNanos() - _last_timestamp) > CounterDecayMinIntervalLength;
}
static void update_last_timestamp() { _last_timestamp = os::javaTimeNanos(); }
};
jlong CounterDecay::_last_timestamp = 0;
void CounterDecay::decay() {
update_last_timestamp();
// This operation is going to be performed only at the end of a safepoint
// and hence GC's will not be going on, all Java mutators are suspended
// at this point and hence SystemDictionary_lock is also not needed.
assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint");
size_t nclasses = ClassLoaderDataGraph::num_instance_classes();
size_t classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 /
CounterHalfLifeTime);
for (size_t i = 0; i < classes_per_tick; i++) {
InstanceKlass* k = ClassLoaderDataGraph::try_get_next_class();
if (k != NULL) {
k->methods_do(do_method);
}
}
}
#ifndef PRODUCT #ifndef PRODUCT
void SimpleCompPolicy::trace_osr_completion(nmethod* osr_nm) { void SimpleCompPolicy::trace_osr_completion(nmethod* osr_nm) {
if (TraceOnStackReplacement) { if (TraceOnStackReplacement) {
@ -223,6 +267,7 @@ void SimpleCompPolicy::initialize() {
} else { } else {
_compiler_count = CICompilerCount; _compiler_count = CICompilerCount;
} }
CounterDecay::update_last_timestamp();
} }
// Note: this policy is used ONLY if TieredCompilation is off. // Note: this policy is used ONLY if TieredCompilation is off.
@ -272,47 +317,6 @@ void SimpleCompPolicy::reset_counter_for_back_branch_event(const methodHandle& m
b->set(b->state(), CompileThreshold / 2); b->set(b->state(), CompileThreshold / 2);
} }
//
// CounterDecay
//
// Iterates through invocation counters and decrements them. This
// is done at each safepoint.
//
class CounterDecay : public AllStatic {
static jlong _last_timestamp;
static void do_method(Method* m) {
MethodCounters* mcs = m->method_counters();
if (mcs != NULL) {
mcs->invocation_counter()->decay();
}
}
public:
static void decay();
static bool is_decay_needed() {
return nanos_to_millis(os::javaTimeNanos() - _last_timestamp) > CounterDecayMinIntervalLength;
}
};
jlong CounterDecay::_last_timestamp = 0;
void CounterDecay::decay() {
_last_timestamp = os::javaTimeNanos();
// This operation is going to be performed only at the end of a safepoint
// and hence GC's will not be going on, all Java mutators are suspended
// at this point and hence SystemDictionary_lock is also not needed.
assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint");
size_t nclasses = ClassLoaderDataGraph::num_instance_classes();
size_t classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 /
CounterHalfLifeTime);
for (size_t i = 0; i < classes_per_tick; i++) {
InstanceKlass* k = ClassLoaderDataGraph::try_get_next_class();
if (k != NULL) {
k->methods_do(do_method);
}
}
}
// Called at the end of the safepoint // Called at the end of the safepoint
void SimpleCompPolicy::do_safepoint_work() { void SimpleCompPolicy::do_safepoint_work() {
if(UseCounterDecay && CounterDecay::is_decay_needed()) { if(UseCounterDecay && CounterDecay::is_decay_needed()) {