RFR(s): 8029162: G1: Shared SATB queue never enabled
mikael.gerdin at oracle.com
Fri Nov 29 07:59:48 UTC 2013
On Thursday 28 November 2013 17.49.53 Per Liden wrote:
> Summary: When activating/deactivating the G1 pre-barrier all SATB queues
> are looped through and have their active state set accordingly. There is
> one queue per Java thread and a shared queue for all non-Java threads.
> The loop done to set the active state of all queues forgets to set the
> active state on the shared queue. I suspect this was missed because the
> logic in SATBMarkQueueSet::set_active_all_threads() is mixed with quite
> a bit of verification code. This patch also moves this verification code
> out into a separate function.
> Testing done: jprt, kitchensink (10 hours), gcbasher (10 hours)
The change looks good to me.
Since this change enables the bug in JDK-8029255 I assume that these changes
must be applied together?
More information about the hotspot-gc-dev