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