RFR: 8295319: pending_cards_at_gc_start doesn't include cards in thread buffers [v2]

[Ivan Walulya]

On Tue, 1 Nov 2022 08:52:12 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Please review this change to G1 to include the per-thread buffers in the
>> number of pending cards at the start of a young GC.
>> 
>> DCQS::concatenate_logs has been renamed to concatenate_logs_and_stats, and now
>> also merges the per-thread refinement stats during the thread walk to flush
>> buffers.  That replaces the separate thread walk to merge and record these
>> stats earlier in the GC.  The merged stats and related info don't seem to be
>> needed until after the buffer flushing.
>> 
>> Also, when abandoning dirty card buffers and stats because of a full GC, fixed
>> to also abandon an buffers in the paused buffers lists.
>> 
>> Testing:
>> mach5 tier1-3
>> performance testing found no significant changes, as expected.
>
> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
> 
>   move per-thread dirty card log/stat concatenation earlier

Lgtm!

-------------

Marked as reviewed by iwalulya (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10914