delay_to_keep_mmu passes wrong arguments to Monitor wait

XMLWordPrintable

    • Type: Bug
    • Resolution: Fixed
    • Priority: P4
    • 16
    • Affects Version/s: 16
    • Component/s: hotspot
    • gc
    • b23

      G1ConcurrentMarkThread::delay_to_keep_mmu locks the CGC_lock monitor using MonitorLocker, and does a wait on that MonitorLocker object. However, the arguments passed to wait include a second argument of Monitor::_no_safepoint_check_flag. Whether waits should be safepoint checking are determined by the MonitorLocker, not by the wait. The second argument to wait is bool as_suspend_equivalent, so this is actually a kind of type error that can't currently be detected by the compiler. The fix here is to just remove the second argument to wait.

            Assignee:
            Kim Barrett
            Reporter:
            Kim Barrett
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: