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