RFR: JDK-8241007 Shenandoah: remove ShenandoahCriticalControlThreadPriority support

[Nikola Grcevski]

Thanks so much Aleksey! I'll make sure future reviews go to hotspot-gc-dev too.

-----Original Message-----
From: Aleksey Shipilev <shade at redhat.com> 
Sent: August 10, 2020 2:18 AM
To: Nikola Grcevski <Nikola.Grcevski at microsoft.com>; shenandoah-dev at openjdk.java.net
Cc: hotspot-gc-dev at openjdk.java.net
Subject: Re: RFR: JDK-8241007 Shenandoah: remove ShenandoahCriticalControlThreadPriority support

On 8/7/20 5:07 PM, Nikola Grcevski wrote:
> 
> Please review this small change that removes the 
> ShenandoahCriticalControlThreadPriority option and its usage:
> 
> Webrev: 
> http://cr.openjdk.java.net/~burban/nikola/JDK-8241007/webrev0.0/
> JBS: https://bugs.openjdk.java.net/browse/JDK-8241007

Process wrinkle: the changes to the upstream code need to be reviewed at hotspot-gc-dev@, possibly cc'ing shenandoah-dev at . CC'ed myself.

> I've tested build on both slowdebug and release and I ran few GC heavy workloads with it.
> 
> If the change looks good, I'll need a sponsor please.

The change looks good to me, I'll sponsor.

--
Thanks,
-Aleksey