8255595: delay_to_keep_mmu passes wrong arguments to Monitor wait

Kim Barrett · Kim Barrett · commit 379ba80eb799 · 2020-10-30T04:24:34.000Z
Remove improper wait argument.

Reviewed-by: sjohanss, tschatzl, ayang
diff --git a/src/hotspot/share/gc/g1/g1ConcurrentMarkThread.cpp b/src/hotspot/share/gc/g1/g1ConcurrentMarkThread.cpp
@@ -108,7 +108,7 @@ void G1ConcurrentMarkThread::delay_to_keep_mmu(bool remark) {
       jlong sleep_time_ms = ceil(sleep_time_sec * MILLIUNITS);
       if (sleep_time_ms <= 0) {
         break;                  // Passed end time.
-      } else if (ml.wait(sleep_time_ms, Monitor::_no_safepoint_check_flag)) {
+      } else if (ml.wait(sleep_time_ms)) {
         break;                  // Timeout => reached end time.
       }
       // Other (possibly spurious) wakeup.  Retry with updated sleep time.

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ void G1ConcurrentMarkThread::delay_to_keep_mmu(bool remark) {`
`108`	`108`	`jlong sleep_time_ms = ceil(sleep_time_sec * MILLIUNITS);`
`109`	`109`	`if (sleep_time_ms <= 0) {`
`110`	`110`	`break; // Passed end time.`
`111`		`- } else if (ml.wait(sleep_time_ms, Monitor::_no_safepoint_check_flag)) {`
	`111`	`+ } else if (ml.wait(sleep_time_ms)) {`
`112`	`112`	`break; // Timeout => reached end time.`
`113`	`113`	`}`
`114`	`114`	`// Other (possibly spurious) wakeup. Retry with updated sleep time.`