Code Review for open

Prepared by:dcubed on Fri Mar 22 16:30:46 EDT 2019
Workspace:/work/shared/bug_hunt/8183909/8153224_exp_for_jdk13/open
Compare against: /work/shared/bug_hunt/8183909/8153224_base_for_jdk13/open
Compare against version:dcubed.more_monitor_logging
Summary of changes: 1169 lines changed: 977 ins; 27 del; 165 mod; 23570 unchg
Patch of changes: open.patch
Bug id: JDK-8153224 : Monitor deflation prolong safepoints
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/prims/jvm.cpp

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
6 lines changed: 6 ins; 0 del; 0 mod; 3656 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
5 lines changed: 4 ins; 0 del; 1 mod; 77 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
3 lines changed: 3 ins; 0 del; 0 mod; 2691 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
185 lines changed: 171 ins; 0 del; 14 mod; 1910 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
65 lines changed: 62 ins; 0 del; 3 mod; 315 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
79 lines changed: 76 ins; 1 del; 2 mod; 107 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
5 lines changed: 4 ins; 0 del; 1 mod; 1274 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
11 lines changed: 8 ins; 0 del; 3 mod; 205 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
8 lines changed: 6 ins; 0 del; 2 mod; 3246 unchg

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

rev 54109 : Add logging to ObjectSynchronizer::omFlush(), add global count logging at Info level to ObjectSynchronizer::finish_deflate_idle_monitors(); for monitorinflation logging, switch from cumulative "deflating per-thread idle monitors" mesgs to per-thread "deflating per-thread idle monitors" mesgs; fix timer bug in deflate_thread_local_monitors() where time to acquire gListLock wasn't counted; fix misc typos.
rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
736 lines changed: 577 ins; 26 del; 133 mod; 2035 unchg

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

rev 54109 : Add logging to ObjectSynchronizer::omFlush(), add global count logging at Info level to ObjectSynchronizer::finish_deflate_idle_monitors(); for monitorinflation logging, switch from cumulative "deflating per-thread idle monitors" mesgs to per-thread "deflating per-thread idle monitors" mesgs; fix timer bug in deflate_thread_local_monitors() where time to acquire gListLock wasn't counted; fix misc typos.
rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
23 lines changed: 20 ins; 0 del; 3 mod; 224 unchg

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

rev 54109 : Add logging to ObjectSynchronizer::omFlush(), add global count logging at Info level to ObjectSynchronizer::finish_deflate_idle_monitors(); for monitorinflation logging, switch from cumulative "deflating per-thread idle monitors" mesgs to per-thread "deflating per-thread idle monitors" mesgs; fix timer bug in deflate_thread_local_monitors() where time to acquire gListLock wasn't counted; fix misc typos.
rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
1 line changed: 1 ins; 0 del; 0 mod; 5074 unchg

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

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
1 line changed: 1 ins; 0 del; 0 mod; 2342 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/hotspot/jtreg/runtime/logging/SafepointCleanupTest.java

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
27 lines changed: 26 ins; 0 del; 1 mod; 78 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/rmi/server/UnicastRemoteObject/unexportObject/UnexportLeak.java

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
5 lines changed: 3 ins; 0 del; 2 mod; 88 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/tools/jlink/multireleasejar/JLinkMultiReleaseJarTest.java

rev 54110 : Checkpoint latest preliminary review patches for full OpenJDK review.
9 lines changed: 9 ins; 0 del; 0 mod; 248 unchg

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