RFR: 8252888: Collapse G1MMUTracker class hierarchy [v2]
John Tortugo
github.com+2249648+JohnTortugo at openjdk.java.net
Mon Sep 28 17:04:43 UTC 2020
On Tue, 22 Sep 2020 12:41:38 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:
>> John Tortugo has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Rename MMUTrackerQueue[Element] to MMUTracker
>
> src/hotspot/share/gc/g1/g1MMUTracker.hpp line 55:
>
>> 53:
>> 54: // This implementation uses a fixed-size queue to keep track of all
>> 55: // recent pause times. Two major user controls over G1 behavior are
>
> I would move the first sentence down to right before the warning about seconds. I would also change it to something
> like: `The G1MMUTracker uses a fixed-size queue to keep track of all recent pause times. The pause time data is used to
> avoid breaking the MMU.`
done
-------------
PR: https://git.openjdk.java.net/jdk/pull/214
More information about the hotspot-gc-dev
mailing list