RFR: JDK-8241007 Shenandoah: remove ShenandoahCriticalControlThreadPriority support
Aditya Mandaleeka
adityam at microsoft.com
Sat Aug 8 16:11:04 UTC 2020
Hey Nikola,
The code change looks good to me. There's also a test you'll need to
remove: gc/shenandoah/options/TestCriticalControlThreadPriority.java.
Thanks,
Aditya
-----Original Message-----
From: shenandoah-dev <shenandoah-dev-retn at openjdk.java.net> On Behalf Of Nikola Grcevski
Sent: Friday, August 7, 2020 8:08 AM
To: shenandoah-dev at openjdk.java.net
Subject: RFR: JDK-8241007 Shenandoah: remove ShenandoahCriticalControlThreadPriority support
Hello Shenandoah-dev,
Please review this small change that removes the ShenandoahCriticalControlThreadPriority option and its usage:
Webrev: https://nam06.safelinks.protection.outlook.com/?url=http:%2F%2Fcr.openjdk.java.net%2F~burban%2Fnikola%2FJDK-8241007%2Fwebrev0.0%2F&data=02%7C01%7Cadityam%40microsoft.com%7C2aa77141d5ab426ee02708d83ae3b6d2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637324096987175965&sdata=BNftRxcC%2F5GyaBwImBEsBxIiZLyUDmiPuOH%2FtvZ8zHA%3D&reserved=0
JBS: https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugs.openjdk.java.net%2Fbrowse%2FJDK-8241007&data=02%7C01%7Cadityam%40microsoft.com%7C2aa77141d5ab426ee02708d83ae3b6d2%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637324096987175965&sdata=Dpdqe3sd0HinoCcZ9ooABJ%2BrpkoV3u8hp0Qc6mIU6rc%3D&reserved=0
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.
Thank you!
Nikola
More information about the shenandoah-dev
mailing list