8182720: In InstanceKlass.cpp, get init_lock from handle

Replace init_lock() calls with h_init_lock() calls

Reviewed-by: sspitsyn, coleenp
This commit is contained in:
Harold Seigel 2017-06-23 09:54:52 -04:00
parent 08361aedba
commit 5e4cca403b

View file

@ -436,7 +436,7 @@ void InstanceKlass::eager_initialize_impl() {
EXCEPTION_MARK; EXCEPTION_MARK;
HandleMark hm(THREAD); HandleMark hm(THREAD);
Handle h_init_lock(THREAD, init_lock()); Handle h_init_lock(THREAD, init_lock());
ObjectLocker ol(h_init_lock, THREAD, init_lock() != NULL); ObjectLocker ol(h_init_lock, THREAD, h_init_lock() != NULL);
// abort if someone beat us to the initialization // abort if someone beat us to the initialization
if (!is_not_initialized()) return; // note: not equivalent to is_initialized() if (!is_not_initialized()) return; // note: not equivalent to is_initialized()
@ -584,7 +584,7 @@ bool InstanceKlass::link_class_impl(bool throw_verifyerror, TRAPS) {
{ {
HandleMark hm(THREAD); HandleMark hm(THREAD);
Handle h_init_lock(THREAD, init_lock()); Handle h_init_lock(THREAD, init_lock());
ObjectLocker ol(h_init_lock, THREAD, init_lock() != NULL); ObjectLocker ol(h_init_lock, THREAD, h_init_lock() != NULL);
// rewritten will have been set if loader constraint error found // rewritten will have been set if loader constraint error found
// on an earlier link attempt // on an earlier link attempt
// don't verify or rewrite if already rewritten // don't verify or rewrite if already rewritten
@ -710,7 +710,7 @@ void InstanceKlass::initialize_impl(TRAPS) {
// Step 1 // Step 1
{ {
Handle h_init_lock(THREAD, init_lock()); Handle h_init_lock(THREAD, init_lock());
ObjectLocker ol(h_init_lock, THREAD, init_lock() != NULL); ObjectLocker ol(h_init_lock, THREAD, h_init_lock() != NULL);
Thread *self = THREAD; // it's passed the current thread Thread *self = THREAD; // it's passed the current thread
@ -846,13 +846,13 @@ void InstanceKlass::initialize_impl(TRAPS) {
void InstanceKlass::set_initialization_state_and_notify(ClassState state, TRAPS) { void InstanceKlass::set_initialization_state_and_notify(ClassState state, TRAPS) {
Handle h_init_lock(THREAD, init_lock()); Handle h_init_lock(THREAD, init_lock());
if (init_lock() != NULL) { if (h_init_lock() != NULL) {
ObjectLocker ol(h_init_lock, THREAD); ObjectLocker ol(h_init_lock, THREAD);
set_init_state(state); set_init_state(state);
fence_and_clear_init_lock(); fence_and_clear_init_lock();
ol.notify_all(CHECK); ol.notify_all(CHECK);
} else { } else {
assert(init_lock() != NULL, "The initialization state should never be set twice"); assert(h_init_lock() != NULL, "The initialization state should never be set twice");
set_init_state(state); set_init_state(state);
} }
} }