RFR: 8271163: G1 uses wrong degree of MT processing since JDK-8270169
Thomas Schatzl
tschatzl at openjdk.java.net
Fri Jul 23 12:05:27 UTC 2021
On Fri, 23 Jul 2021 10:33:18 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
> Hi all,
>
> JDK-8270169 changed the amount of resources supplied by the CM reference processor, assuming that the CM ref processor is only ever used with the marking threads.
> This is unfortunately not true, the CM reference processor is also used by the Remark pause for discovery using the parallel gc worker threads.
>
> This causes crashes because of the code assuming single threaded operation while it is not.
>
> The solution proposed here is to just revert JDK-8270169 which introduced that change. Applies cleanly.
>
> I also added a comment in the reference processor constructor to why we need to consider both thread counts.
>
> Thanks,
> Thomas
Since this is basically a clean backout of JDK-8270169, I'll skip the 24h rule requirement.
Thanks @kimbarrett @albertnetymk for your reviews.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4888
More information about the hotspot-gc-dev
mailing list