RFR: 8313962: G1: Refactor G1ConcurrentMark::_num_concurrent_workers
Thomas Schatzl
tschatzl at openjdk.org
Wed Aug 9 09:36:01 UTC 2023
On Tue, 8 Aug 2023 17:30:52 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:
> Simple cleanup to make this field track the correct number of gc-workers during marking. Some redundant asserts are removed as well.
>
> Test: hotspot_gc
lgtm, although this changes mixes a bugfix (num_concurrent_workers value at startup) and an enhancement (set num workers during root region scan). So consider splitting these.
-------------
Marked as reviewed by tschatzl (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/15194#pullrequestreview-1569128580
More information about the hotspot-gc-dev
mailing list