8222371: Use UL for TraceNMethodInstalls code

Reviewed-by: dholmes
This commit is contained in:
Vladimir Kozlov 2019-06-06 16:06:10 -07:00
parent 4bd89c7401
commit bb4d2bc846
4 changed files with 21 additions and 22 deletions

View file

@ -44,6 +44,7 @@
#include "gc/shared/collectedHeap.inline.hpp" #include "gc/shared/collectedHeap.inline.hpp"
#include "interpreter/linkResolver.hpp" #include "interpreter/linkResolver.hpp"
#include "jfr/jfrEvents.hpp" #include "jfr/jfrEvents.hpp"
#include "logging/log.hpp"
#include "memory/allocation.inline.hpp" #include "memory/allocation.inline.hpp"
#include "memory/oopFactory.hpp" #include "memory/oopFactory.hpp"
#include "memory/resourceArea.hpp" #include "memory/resourceArea.hpp"
@ -1076,25 +1077,23 @@ void ciEnv::register_method(ciMethod* target,
old->make_not_used(); old->make_not_used();
} }
} }
if (TraceNMethodInstalls) {
LogTarget(Info, nmethod, install) lt;
if (lt.is_enabled()) {
ResourceMark rm; ResourceMark rm;
char *method_name = method->name_and_sig_as_C_string(); char *method_name = method->name_and_sig_as_C_string();
ttyLocker ttyl; lt.print("Installing method (%d) %s ",
tty->print_cr("Installing method (%d) %s ", task()->comp_level(), method_name);
task()->comp_level(),
method_name);
} }
// Allow the code to be executed // Allow the code to be executed
method->set_code(method, nm); method->set_code(method, nm);
} else { } else {
if (TraceNMethodInstalls) { LogTarget(Info, nmethod, install) lt;
if (lt.is_enabled()) {
ResourceMark rm; ResourceMark rm;
char *method_name = method->name_and_sig_as_C_string(); char *method_name = method->name_and_sig_as_C_string();
ttyLocker ttyl; lt.print("Installing osr method (%d) %s @ %d",
tty->print_cr("Installing osr method (%d) %s @ %d", task()->comp_level(), method_name, entry_bci);
task()->comp_level(),
method_name,
entry_bci);
} }
method->method_holder()->add_osr_nmethod(nm); method->method_holder()->add_osr_nmethod(nm);
} }

View file

@ -1543,25 +1543,23 @@ JVMCI::CodeInstallResult JVMCIRuntime::register_method(JVMCIEnv* JVMCIENV,
old->make_not_entrant(); old->make_not_entrant();
} }
} }
if (TraceNMethodInstalls) {
LogTarget(Info, nmethod, install) lt;
if (lt.is_enabled()) {
ResourceMark rm; ResourceMark rm;
char *method_name = method->name_and_sig_as_C_string(); char *method_name = method->name_and_sig_as_C_string();
ttyLocker ttyl; lt.print("Installing method (%d) %s [entry point: %p]",
tty->print_cr("Installing method (%d) %s [entry point: %p]", comp_level, method_name, nm->entry_point());
comp_level,
method_name, nm->entry_point());
} }
// Allow the code to be executed // Allow the code to be executed
method->set_code(method, nm); method->set_code(method, nm);
} else { } else {
if (TraceNMethodInstalls ) { LogTarget(Info, nmethod, install) lt;
if (lt.is_enabled()) {
ResourceMark rm; ResourceMark rm;
char *method_name = method->name_and_sig_as_C_string(); char *method_name = method->name_and_sig_as_C_string();
ttyLocker ttyl; lt.print("Installing osr method (%d) %s @ %d",
tty->print_cr("Installing osr method (%d) %s @ %d", comp_level, method_name, entry_bci);
comp_level,
method_name,
entry_bci);
} }
InstanceKlass::cast(method->method_holder())->add_osr_nmethod(nm); InstanceKlass::cast(method->method_holder())->add_osr_nmethod(nm);
} }

View file

@ -83,6 +83,7 @@
LOG_TAG(iklass) \ LOG_TAG(iklass) \
LOG_TAG(init) \ LOG_TAG(init) \
LOG_TAG(inlining) \ LOG_TAG(inlining) \
LOG_TAG(install) \
LOG_TAG(interpreter) \ LOG_TAG(interpreter) \
LOG_TAG(itables) \ LOG_TAG(itables) \
LOG_TAG(jfr) \ LOG_TAG(jfr) \

View file

@ -604,6 +604,7 @@ static AliasedLoggingFlag const aliased_logging_flags[] = {
{ "TraceSafepointCleanupTime", LogLevel::Info, true, LOG_TAGS(safepoint, cleanup) }, { "TraceSafepointCleanupTime", LogLevel::Info, true, LOG_TAGS(safepoint, cleanup) },
{ "TraceJVMTIObjectTagging", LogLevel::Debug, true, LOG_TAGS(jvmti, objecttagging) }, { "TraceJVMTIObjectTagging", LogLevel::Debug, true, LOG_TAGS(jvmti, objecttagging) },
{ "TraceRedefineClasses", LogLevel::Info, false, LOG_TAGS(redefine, class) }, { "TraceRedefineClasses", LogLevel::Info, false, LOG_TAGS(redefine, class) },
{ "TraceNMethodInstalls", LogLevel::Info, true, LOG_TAGS(nmethod, install) },
{ NULL, LogLevel::Off, false, LOG_TAGS(_NO_TAG) } { NULL, LogLevel::Off, false, LOG_TAGS(_NO_TAG) }
}; };