Skip to content

Commit ccb7e45

Browse files
Thomas SchatzlJuyeon Ma
authored andcommitted
Backport <d4d1fbc27a2382d8d9545b7bf672cdcc81654404>
1 parent 03abc5f commit ccb7e45

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/hotspot/share/gc/g1/g1Policy.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1299,7 +1299,6 @@ void G1Policy::calculate_old_collection_set_regions(G1CollectionSetCandidates* c
12991299
num_optional_regions = 0;
13001300
uint num_expensive_regions = 0;
13011301

1302-
double predicted_old_time_ms = 0.0;
13031302
double predicted_initial_time_ms = 0.0;
13041303
double predicted_optional_time_ms = 0.0;
13051304

@@ -1330,7 +1329,7 @@ void G1Policy::calculate_old_collection_set_regions(G1CollectionSetCandidates* c
13301329
time_remaining_ms = MAX2(time_remaining_ms - predicted_time_ms, 0.0);
13311330
// Add regions to old set until we reach the minimum amount
13321331
if (num_initial_regions < min_old_cset_length) {
1333-
predicted_old_time_ms += predicted_time_ms;
1332+
predicted_initial_time_ms += predicted_time_ms;
13341333
num_initial_regions++;
13351334
// Record the number of regions added with no time remaining
13361335
if (time_remaining_ms == 0.0) {
@@ -1344,7 +1343,7 @@ void G1Policy::calculate_old_collection_set_regions(G1CollectionSetCandidates* c
13441343
} else {
13451344
// Keep adding regions to old set until we reach the optional threshold
13461345
if (time_remaining_ms > optional_threshold_ms) {
1347-
predicted_old_time_ms += predicted_time_ms;
1346+
predicted_initial_time_ms += predicted_time_ms;
13481347
num_initial_regions++;
13491348
} else if (time_remaining_ms > 0) {
13501349
// Keep adding optional regions until time is up.
@@ -1368,7 +1367,7 @@ void G1Policy::calculate_old_collection_set_regions(G1CollectionSetCandidates* c
13681367
}
13691368

13701369
log_debug(gc, ergo, cset)("Finish choosing collection set old regions. Initial: %u, optional: %u, "
1371-
"predicted old time: %1.2fms, predicted optional time: %1.2fms, time remaining: %1.2f",
1370+
"predicted initial time: %1.2fms, predicted optional time: %1.2fms, time remaining: %1.2f",
13721371
num_initial_regions, num_optional_regions,
13731372
predicted_initial_time_ms, predicted_optional_time_ms, time_remaining_ms);
13741373
}

0 commit comments

Comments
 (0)