RFR(s): 8029162: G1: Shared SATB queue never enabled

[Per Liden]

Thanks Bengt!

/Per

On 2014-01-08 14:40, Bengt Rutisson wrote:
>
> Looks good, Per.
>
> Thanks,
> Bengt
>
> On 2013-11-28 17:49, 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)
>>
>> http://cr.openjdk.java.net/~pliden/8029162/webrev.0/
>>
>> https://bugs.openjdk.java.net/browse/JDK-8029162
>>
>> cheers,
>> /Per
>