Wrong iteration order of pause array in g1MMUTracker

XMLWordPrintable

    • Type: Bug
    • Resolution: Fixed
    • Priority: P4
    • 21
    • Affects Version/s: 20, 21
    • Component/s: hotspot
    • None
    • gc
    • b18

      G1 pause _array is iterated from oldest pause to newer pause in order to calculate the pause budget, but it should be iterated in the opposite direction (newest to oldest).

      In double G1MMUTracker::when_sec(double current_timestamp, double pause_time) code:

      int index = trim_index(_head_index + i);

      should be replaced with:

      int index = trim_index(_head_index - i);

            Assignee:
            Aleksei Voitylov
            Reporter:
            Aleksei Voitylov
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: