RFR: 8271163: G1 uses wrong degree of MT processing since JDK-8270169
Kim Barrett
kbarrett 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
As this is a simple backout (plus commentary), I suggest not waiting the usual 24 hours before pushing, to reduce testing noise.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4888
More information about the hotspot-gc-dev
mailing list