infernap12/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2026-01-28 12:09:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

Go even lower, don't check for safepoint when taking the locks

Browse Source
This commit is contained in:
William Kemper 2026-01-15 16:33:59 -08:00
parent 2f9fa4283c
commit 1d8e7a2a14
4 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/hotspot/share/gc/shenandoah/shenandoahControlThread.cpp
View File

@ -44,7 +44,7 @@ ShenandoahControlThread::ShenandoahControlThread() :
ShenandoahController(),
_requested_gc_cause(GCCause::_no_gc),
_degen_point(ShenandoahGC::_degenerated_outside_cycle),
_control_lock(LOCK_RANK, "ShenandoahGCRequest_lock", true) {
_control_lock(LOCK_RANK - 1, "ShenandoahControl_lock", true) {
set_name("Shenandoah Control Thread");
create_and_start();
}
@ -380,7 +380,7 @@ void ShenandoahControlThread::handle_requested_gc(GCCause::Cause cause) {
// opportunities for cleanup that were made available before the caller
// requested the GC.
MonitorLocker ml(&_gc_waiters_lock);
MonitorLocker ml(&_gc_waiters_lock, Mutex::_no_safepoint_check_flag);
size_t current_gc_id = get_gc_id();
size_t required_gc_id = current_gc_id + 1;
while (current_gc_id < required_gc_id && !should_terminate()) {
@ -392,6 +392,6 @@ void ShenandoahControlThread::handle_requested_gc(GCCause::Cause cause) {
void ShenandoahControlThread::notify_gc_waiters() {
_gc_requested.unset();
MonitorLocker ml(&_gc_waiters_lock);
MonitorLocker ml(&_gc_waiters_lock, Mutex::_no_safepoint_check_flag);
ml.notify_all();
}

4
src/hotspot/share/gc/shenandoah/shenandoahController.cpp
View File

@ -51,7 +51,7 @@ void ShenandoahController::handle_alloc_failure(const ShenandoahAllocRequest& re
}
if (block) {
MonitorLocker ml(&_alloc_failure_waiters_lock);
MonitorLocker ml(&_alloc_failure_waiters_lock, Mutex::_no_safepoint_check_flag);
while (!should_terminate() && ShenandoahCollectorPolicy::is_allocation_failure(heap->cancelled_cause())) {
ml.wait();
}
@ -70,6 +70,6 @@ void ShenandoahController::handle_alloc_failure_evac(size_t words) {
}
void ShenandoahController::notify_alloc_failure_waiters() {
MonitorLocker ml(&_alloc_failure_waiters_lock);
MonitorLocker ml(&_alloc_failure_waiters_lock, Mutex::_no_safepoint_check_flag);
ml.notify_all();
}

4
src/hotspot/share/gc/shenandoah/shenandoahController.hpp
View File

@ -56,8 +56,8 @@ protected:
public:
ShenandoahController():
_gc_id(0),
_alloc_failure_waiters_lock(LOCK_RANK, "ShenandoahAllocFailureGC_lock", true),
_gc_waiters_lock(LOCK_RANK, "ShenandoahRequestedGC_lock", true)
_alloc_failure_waiters_lock(LOCK_RANK, "ShenandoahAllocFailureWaiters_lock", true),
_gc_waiters_lock(LOCK_RANK, "ShenandoahGCWaiters_lock", true)
{ }
// Request a collection cycle. This handles "explicit" gc requests

6
src/hotspot/share/gc/shenandoah/shenandoahGenerationalControlThread.cpp
View File

@ -47,7 +47,7 @@
#include "utilities/events.hpp"
ShenandoahGenerationalControlThread::ShenandoahGenerationalControlThread() :
_control_lock(LOCK_RANK, "ShenandoahGCRequest_lock", true),
_control_lock(LOCK_RANK - 1, "ShenandoahControl_lock", true),
_requested_gc_cause(GCCause::_no_gc),
_requested_generation(nullptr),
_gc_mode(none),
@ -759,7 +759,7 @@ void ShenandoahGenerationalControlThread::handle_requested_gc(GCCause::Cause cau
// opportunities for cleanup that were made available before the caller
// requested the GC.
MonitorLocker ml(&_gc_waiters_lock);
MonitorLocker ml(&_gc_waiters_lock, Mutex::_no_safepoint_check_flag);
size_t current_gc_id = get_gc_id();
const size_t required_gc_id = current_gc_id + 1;
while (current_gc_id < required_gc_id && !should_terminate()) {
@ -771,7 +771,7 @@ void ShenandoahGenerationalControlThread::handle_requested_gc(GCCause::Cause cau
}
void ShenandoahGenerationalControlThread::notify_gc_waiters() {
MonitorLocker ml(&_gc_waiters_lock);
MonitorLocker ml(&_gc_waiters_lock, Mutex::_no_safepoint_check_flag);
ml.notify_all();
}