RFR: 8325259: Serial: Inline OldGenScanClosure during Young GC [v3]
Thomas Schatzl
tschatzl at openjdk.org
Thu Feb 8 09:35:54 UTC 2024
On Wed, 7 Feb 2024 14:14:24 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:
>> Simple inlining oop closure during card-scanning of young-gc.
>>
>> Test: using the attached card-scanning-stressing benchmark, ~5x improvement (in terms of young-gc pause time) is observed on my box.
>
> Albert Mingkun Yang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>
> - review
> - reviw
> - reviw
> - Merge branch 'master' into s1-inline
> - review
> - s1-inline
Lgtm. Maybe fix one preexisting typo.
src/hotspot/share/gc/serial/defNewGeneration.cpp line 707:
> 705: MarkingCodeBlobClosure code_cl(&root_cl,
> 706: CodeBlobToOopClosure::FixRelocations,
> 707: false /* keepalive nmethods */);
Suggestion:
false /* keepalive_nmethods */);
-------------
Marked as reviewed by tschatzl (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/17711#pullrequestreview-1869656398
PR Review Comment: https://git.openjdk.org/jdk/pull/17711#discussion_r1482666451
More information about the hotspot-gc-dev
mailing list