Code Review for open

Prepared by:dcubed on Mon Jan 27 15:15:23 EST 2020
Workspace:/work/shared/bug_hunt/8183909/8235795_exp_for_jdk14/open
Compare against: /work/shared/bug_hunt/8183909/8153224_base_for_jdk14/open
Compare against version:qparent
Summary of changes: 827 lines changed: 522 ins; 117 del; 188 mod; 6729 unchg
Patch of changes: open.patch
Bug id: JDK-8235795 : replace monitor list mux{Acquire,Release}(&gListLock) with spin locks
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/objectMonitor.cpp

rev 57560 : imported patch 8235795.patch.cr0
1 line changed: 0 ins; 0 del; 1 mod; 2007 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/objectMonitor.hpp

rev 57560 : imported patch 8235795.patch.cr0
15 lines changed: 9 ins; 5 del; 1 mod; 310 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/synchronizer.cpp

rev 57560 : imported patch 8235795.patch.cr0
rev 57561 : dholmes CR - refactor common code, refactor atomic load of LVars.population in monitors_used_above_threshold, simplify list walking in ObjectSynchronizer::om_release() so we lock fewer ObjectMonitors, remove unnecessary locking from ObjectSynchronizer::deflate_monitor_list(), add NoSafepointVerifier helpers to main list management functions, remove unnecessary storestore(), remove unnecessary comments, clarify/fix comments.
800 lines changed: 512 ins; 107 del; 181 mod; 1902 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/synchronizer.hpp

rev 57560 : imported patch 8235795.patch.cr0
10 lines changed: 1 ins; 4 del; 5 mod; 208 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/thread.hpp

rev 57560 : imported patch 8235795.patch.cr0
1 line changed: 0 ins; 1 del; 0 mod; 2302 unchg

This code review page was prepared using /Users/dcubed/Bin/webrev (vers 25.18-hg+openjdk.java.net).