Summary: Clean up usage of idx_t and uintptr_t when using it in conjunction with BitMap::set_map(), casting to the appropriate type. Fixes compilation on S390.
Reviewed-by: tschatzl
Contributed-by: NDan Horak <dhorak@redhat.com>

Reviewed-by: brutisso, sjohanss

Reviewed-by: brutisso, ehelin

Summary: Removed all write barriers during reference processing and added explicit write barriers when iterating through the discovered list.
Reviewed-by: pliden, jmasa, tschatzl

Reviewed-by: brutisso, jmasa, tschatzl

Reviewed-by: tschatzl, mgerdin

Summary: Only prune metaspace virtual spaces at safepoint so walking them is safe outside a safepoint.
Reviewed-by: mgerdin, mgronlun, hseigel, stefank

Reviewed-by: brutisso, tschatzl

Reviewed-by: brutisso, ehelin

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

Reviewed-by: twisti, stefank

Summary: Removed duplicate code in RSHashTable to fetch SparsePRTEntries
Reviewed-by: tschatzl, brutisso
Contributed-by: NAndreas Sjoberg <andreas.sjoberg@oracle.com>

Summary: Refactor stringTable class out of symbolTable, making sure all includes are minimal set and are sorted.
Reviewed-by: coleenp, stefank

Summary: Renamed gen0 and gen1 to young and old throughout the GC code.
Reviewed-by: sjohanss, jcoomes

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

Reviewed-by: mgerdin, brutisso
Contributed-by: michal@frajt.eu

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

Reviewed-by: jmasa, stefank