Summary: Repaired or selectively disabled offending formats; future-proofed with additional checking
Reviewed-by: kvn, jrose, stefank

Summary: Eliminating duplicated code by introducing print_on(outputStream* st) methods in NamedThread
Reviewed-by: twisti, coleenp, dholmes

Reviewed-by: dholmes, kvn, stefank, twisti

Reviewed-by: jmasa, tschatzl, ehelin
Contributed-by: tony.printezis@oracle.com, bengt.rutisson@oracle.com

Summary: A few locations in the code use sizeof(this) which returns the size of the pointer instead of sizeof(classname) which returns the size of the sum of its members. This change fixes these errors and adds a few tests.
Reviewed-by: mgerdin, brutisso

Summary: temporarily disable the shutdown of the concurrent GC threads introduced in JDK-8037112
Reviewed-by: brutisso, tschatzl, jmasa

Reviewed-by: tschatzl, jmasa

Summary: Card re-dirtying code for verification and actual redirtying uses two different, almost completely identical card closures. Also the verification code still assumes a perm gen.
Reviewed-by: brutisso, jmasa

Summary: Parallelize card redirtying to decrease the time it takes.
Reviewed-by: brutisso

Summary: G1 does not retire allocation buffers after reference processing work when -XX:+ParallelRefProcEnabled is enabled. This causes wrong calculation of PLAB sizes, as the amount of space wasted is not updated correctly.
Reviewed-by: brutisso

Summary: The mentioned method is never used and out of date. So it is removed.
Reviewed-by: mgerdin, brutisso

Summary: After changes to the PerRegionTable where all these PRTs are linked together in an additional field, simplify iterating over all PRTs by using these links instead of walki
Reviewed-by: mgerdin, jwilhelm, brutisso

Summary: Instead of using a manually managed array for the in_cset_fast_test array, use a G1BiasedArray instance.
Reviewed-by: brutisso, mgerdin

8038930: G1CodeRootSet::test fails with assert(_num_chunks_handed_out == 0) failed: No elements must have been handed out yet
Summary: The test incorrectly assumed that it had been started with no other previous compilation activity. Fix this by allowing multiple code root free chunk lists, and use one separate from the global one to perform the test.
Reviewed-by: brutisso

Reviewed-by: jwilhelm, sjohanss, jmasa

Reviewed-by: brutisso, tschatzl, mgerdin

Reviewed-by: brutisso, mgerdin

Reviewed-by: jmasa, sjohanss
Contributed-by: stefan.karlsson@oracle.com, mikael.vidstedt@oracle.com

Reviewed-by: brutisso, tschatzl

Summary: Replaced the verification call to a conditional one that only runs verification in debug builds.
Reviewed-by: brutisso, tschatzl

Reviewed-by: tschatzl, jwilhelm

Reviewed-by: dholmes, fparain
Contributed-by: david.simms@oracle.com

Reviewed-by: jmasa, coleenp

Summary: Change 8035330: Remove G1ParScanPartialArrayClosure and G1ParScanHeapEvacClosure broke the debug build on AIX. The method do_oop_partial_array() is added in a header, but requires the inline function par_write_ref() through several inlined calls. In some cpp files, like arguments.cpp, par_write_ref() is not defined as the corresponding inline header and is not included. The AIX debug VM does not start because of the missing symbol. This change solves this by cleaning up include dependencies.
Reviewed-by: tschatzl, stefank

Reviewed-by: sjohanss, jmasa

Summary: Implementation of JEP 192, http://openjdk.java.net/jeps/192
Reviewed-by: brutisso, tschatzl, coleenp

Summary: Allocate temporary BitMaps in the VMThread's resource area
Reviewed-by: stefank, sjohanss

Summary: The free list is being maintained in a sorted fashion and old and humongous regions are allocated from the bottom of the heap while young regions are allocated at the top.
Reviewed-by: tschatzl, mgerdin
Contributed-by: jesper.wilhelmsson@oracle.com, staffan.friberg@oracle.com

Reviewed-by: brutisso, tschatzl

Reviewed-by: brutisso, tschatzl

Reviewed-by: tschatzl, pliden

Summary: Pad and cache-align BiasedMappedArray instances by default to avoid performance variability problems due to false sharing, as instances of this data structures are typically used for performance sensitive code.
Reviewed-by: brutisso, stefank

Summary: Mentioned closures are actually wrapped methods. This adds confusion to readers, and in this case also increases code size as G1ParScanHeapEvacClosure is part of the oop_oop_iterate() methods. Move them into G1ParScanThreadState as methods.
Reviewed-by: stefank

Summary: Refactor the From Card Cache into a separate class.
Reviewed-by: jmasa

Summary: Detailed breakdown of time spent in the evacuation failure handling phases to make the "Other" time roughly correspond to the sum of its parts.
Reviewed-by: jwilhelm, jmasa

Summary: Show the time taken by card redirtying during GC in a new "Redirty Cards" line.
Reviewed-by: jwilhelm, brutisso

Summary: Improve fast card cache iteration and avoid taking locks when freeing the collection set.
Reviewed-by: brutisso

Summary: The from card cache is a very frequently accessed data structure. It is essentially a 2d array of per-region values, one row of values for every GC thread. Pad and align the data structure to avoid false sharing.
Reviewed-by: stefank

Summary: Change the code cache remembered sets data structure from a GrowableArray to a chunked list of nmethods. This makes the data structure more amenable to parallelization, and decreases freeing time.
Reviewed-by: mgerdin, brutisso

Reviewed-by: tschatzl, pliden