Code Review for open

Prepared by:kbarrett on Sun Nov 4 19:00:16 EST 2018
Workspace:/home/kab/sandboxes/pq_alloc/jdk/open
Compare against version:qparent
Summary of changes: 338 lines changed: 215 ins; 95 del; 28 mod; 9783 unchg
Patch of changes: open.patch
Bug id: JDK-8213352 : Separate BufferNode allocation from PtrQueueSet
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/dirtyCardQueue.cpp

rev 52405 : [mq]: use_allocators
6 lines changed: 0 ins; 3 del; 3 mod; 343 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/dirtyCardQueue.hpp

rev 52405 : [mq]: use_allocators
2 lines changed: 0 ins; 1 del; 1 mod; 172 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/g1BarrierSet.cpp

rev 52404 : [mq]: make_allocators
10 lines changed: 10 ins; 0 del; 0 mod; 204 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/g1BarrierSet.hpp

rev 52404 : [mq]: make_allocators
5 lines changed: 5 ins; 0 del; 0 mod; 124 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/g1CollectedHeap.cpp

rev 52405 : [mq]: use_allocators
43 lines changed: 22 ins; 16 del; 5 mod; 4657 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/g1ConcurrentMark.cpp

rev 52405 : [mq]: use_allocators
3 lines changed: 0 ins; 3 del; 0 mod; 3058 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/g1SATBMarkQueueSet.cpp

rev 52405 : [mq]: use_allocators
4 lines changed: 2 ins; 0 del; 2 mod; 127 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/g1/g1SATBMarkQueueSet.hpp

rev 52405 : [mq]: use_allocators
2 lines changed: 1 ins; 0 del; 1 mod; 49 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/shared/ptrQueue.cpp

rev 52403 : [mq]: allocator
rev 52405 : [mq]: use_allocators
126 lines changed: 72 ins; 46 del; 8 mod; 260 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/shared/ptrQueue.hpp

rev 52403 : [mq]: allocator
rev 52405 : [mq]: use_allocators
57 lines changed: 26 ins; 26 del; 5 mod; 340 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/shared/satbMarkQueue.cpp

rev 52405 : [mq]: use_allocators
3 lines changed: 1 ins; 0 del; 2 mod; 261 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/gc/shared/satbMarkQueue.hpp

rev 52405 : [mq]: use_allocators
2 lines changed: 1 ins; 0 del; 1 mod; 188 unchg

------ ------ ------ ------ --- New Patch Raw test/hotspot/gtest/gc/shared/test_ptrQueueBufferAllocator.cpp

rev 52403 : [mq]: allocator
75 lines changed: 75 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /home/kab/devtools/jdk/code-tools/webrev/webrev.ksh (vers 25.17-hg+openjdk.java.net).