8213992: Rename and make DieOnSafepointTimeout the diagnostic option

Reviewed-by: dholmes, dcubed, coleenp, kvn
This commit is contained in:
Aleksey Shipilev 2018-11-20 22:59:27 +01:00
parent 521da29e44
commit a264dac26c
2 changed files with 4 additions and 4 deletions

View File

@ -498,8 +498,8 @@ define_pd_global(uint64_t,MaxRAM, 1ULL*G);
"Time out and warn or fail after SafepointTimeoutDelay " \
"milliseconds if failed to reach safepoint") \
\
develop(bool, DieOnSafepointTimeout, false, \
"Die upon failure to reach safepoint (see SafepointTimeout)") \
diagnostic(bool, AbortVMOnSafepointTimeout, false, \
"Abort upon failure to reach safepoint (see SafepointTimeout)") \
\
/* 50 retries * (5 * current_retry_count) millis = ~6.375 seconds */ \
/* typically, at most a few retries are needed */ \

View File

@ -978,9 +978,9 @@ void SafepointSynchronize::print_safepoint_timeout(SafepointTimeoutReason reason
}
}
// To debug the long safepoint, specify both DieOnSafepointTimeout &
// To debug the long safepoint, specify both AbortVMOnSafepointTimeout &
// ShowMessageBoxOnError.
if (DieOnSafepointTimeout) {
if (AbortVMOnSafepointTimeout) {
fatal("Safepoint sync time longer than " INTX_FORMAT "ms detected when executing %s.",
SafepointTimeoutDelay, VMThread::vm_safepoint_description());
}