src/hotspot/share/runtime/synchronizer.cpp

rev 57560 : imported patch 8236035.patch.cr0
rev 57561 : dholmes CR - rename simply_set_owner_from() -> set_owner_from() and simply_set_owner_from_BasicLock() -> set_owner_from_BasicLock(); rename release_clear_owner_with_barrier() -> release_clear_owner() and refactor barrier code back into the call sites.

*** 1402,1412 ****
        // Optimization: if the mark.locker stack address is associated
        // with this thread we could simply set m->_owner = self.
        // Note that a thread can inflate an object
        // that it has stack-locked -- as might happen in wait() -- directly
        // with CAS.  That is, we can avoid the xchg-NULL .... ST idiom.
!       m->simply_set_owner_from(NULL, mark.locker());
        m->set_object(object);
        // TODO-FIXME: assert BasicLock->dhw != 0.
  
        // Must preserve store ordering. The monitor state must
        // be stable at the time of publishing the monitor address.
--- 1402,1412 ----
        // Optimization: if the mark.locker stack address is associated
        // with this thread we could simply set m->_owner = self.
        // Note that a thread can inflate an object
        // that it has stack-locked -- as might happen in wait() -- directly
        // with CAS.  That is, we can avoid the xchg-NULL .... ST idiom.
!       m->set_owner_from(NULL, mark.locker());
        m->set_object(object);
        // TODO-FIXME: assert BasicLock->dhw != 0.
  
        // Must preserve store ordering. The monitor state must
        // be stable at the time of publishing the monitor address.