Skip to content

Commit

Permalink
8305892: G1: Fix G1MMUTracker::when_sec documentation
Browse files Browse the repository at this point in the history
Reviewed-by: iwalulya, tschatzl
  • Loading branch information
albertnetymk committed Apr 17, 2023
1 parent c0b4957 commit dbd2b5f
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions src/hotspot/share/gc/g1/g1MMUTracker.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -115,13 +115,13 @@ void G1MMUTracker::add_pause(double start, double end) {
// GC events / pause_time
// / | \ \ | / /
// -------------[----]-[---]--[--]---[---]------|[--]-----> Time
// | | |
// | | |
// |<- limit | |
// | |<- balance_timestamp |
// | ^ |
// | |
// |<-------- _time_slice ------>|
// | | |
// | | |
// |<- limit | |
// | |<- balance_timestamp |
// | ^ |
// | |
// |<-------- _time_slice --------->|
//
// The MMU constraint requires that we can spend up to `max_gc_time()` on GC
// pauses inside a window of `_time_slice` length. Therefore, we have a GC
Expand All @@ -134,7 +134,7 @@ void G1MMUTracker::add_pause(double start, double end) {
// time inside [balance_timestamp, current_timestamp] is equal to the budget.
// Next, return `balance_timestamp - limit`.
//
// When there are no enough GC events, i.e. we have a surplus buget, a new GC
// When there are not enough GC events, i.e. we have a surplus budget, a new GC
// pause can start right away, so return 0.
double G1MMUTracker::when_sec(double current_timestamp, double pause_time) {
assert(pause_time > 0.0, "precondition");
Expand Down

1 comment on commit dbd2b5f

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.