Problematic backport to 8u: "Ensure tasks use correct number of workers"

Zhengyu Gu zgu at redhat.com
Mon Oct 2 18:04:19 UTC 2017


Updated: 
http://cr.openjdk.java.net/~zgu/shenandoah/assert_workers_8u/webrev.01/

So, you are going to push this with others, right?

Thanks,

-Zhengyu


On 10/02/2017 01:56 PM, Aleksey Shipilev wrote:
> On 10/02/2017 07:50 PM, Zhengyu Gu wrote:
>> Simply introduce ShenandoahWorkGang to bypass assertion in FlexibleWorkGang (and avoid contaminate
>> upstream).
>>
>> Webrev: http://cr.openjdk.java.net/~zgu/shenandoah/assert_workers_8u/webrev.00/
> 
> Thank you, I shall fold it into outstanding jdk8 backports RFR.
> 
> For my own checks:
> 
> *) Webrev misses the test, do:
>   $ hg add test/gc/shenandoah/TestGCThreadGroups.java
> 
> *) Excess whitespace after "bool":
> 
>    58                    bool  are_ConcurrentGC_threads) :
> 
> Otherwise seems fine.
> 
> Thanks,
> -Aleksey
> 


More information about the shenandoah-dev mailing list