stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8241007: Shenandoah: remove ShenandoahCriticalControlThreadPriority support

Browse Source 
Reviewed-by: adityam, shade
This commit is contained in:
Nikola Grcevski 2020-08-10 08:36:56 +02:00 committed by Aleksey Shipilev
parent 970e251a54
commit 032a4d6b9f
3 changed files with 1 additions and 45 deletions
src/hotspot/share/gc/shenandoah
shenandoahControlThread.cppshenandoah_globals.hpp
test/hotspot/jtreg/gc/shenandoah/options
TestCriticalControlThreadPriority.java

2
src/hotspot/share/gc/shenandoah/shenandoahControlThread.cpp

@ -50,7 +50,7 @@ ShenandoahControlThread::ShenandoahControlThread() :
_allocs_seen(0) {
reset_gc_id();
create_and_start(ShenandoahCriticalControlThreadPriority ? CriticalPriority : NearMaxPriority);
create_and_start();
_periodic_task.enroll();
_periodic_satb_flush_task.enroll();
if (ShenandoahPacing) {

3
src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp

@ -191,9 +191,6 @@
"adjustment. Lower values make adjustments faster, at the " \
"expense of higher perf overhead. Time is in milliseconds.") \
\
experimental(bool, ShenandoahCriticalControlThreadPriority, false, \
"Run control thread runs at critical scheduling priority.") \
\
diagnostic(bool, ShenandoahVerify, false, \
"Enable internal verification. This would catch many GC bugs, " \
"but it would also stall the collector during the verification, " \

41
test/hotspot/jtreg/gc/shenandoah/options/TestCriticalControlThreadPriority.java

@ -1,41 +0,0 @@
/*
* Copyright (c) 2019, Red Hat, Inc. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*
*/
/*
* @test TestCriticalControlThreadPriority
* @summary Check that ShenandoahCriticalControlThreadPriority works
* @bug 8217343
* @requires vm.gc.Shenandoah
*
* @run main/othervm -XX:+UnlockExperimentalVMOptions -XX:+UseShenandoahGC -XX:-ShenandoahCriticalControlThreadPriority -Xmx1g TestCriticalControlThreadPriority
* @run main/othervm -XX:+UnlockExperimentalVMOptions -XX:+UseShenandoahGC -XX:+ShenandoahCriticalControlThreadPriority -Xmx1g TestCriticalControlThreadPriority
*/
public class TestCriticalControlThreadPriority {
public static void main(String[] args) throws Exception {
// checking the initialization before entering main()
}
}