Skip to content

Commit ff5d417

Browse files
author
Thomas Schatzl
committedDec 21, 2021
8278893: Parallel: Remove GCWorkerDelayMillis
Reviewed-by: ayang, mli
1 parent 5179672 commit ff5d417

File tree

3 files changed

+0
-21
lines changed

3 files changed

+0
-21
lines changed
 

‎src/hotspot/share/gc/parallel/parallel_globals.hpp

-3
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,6 @@
5555
"limiter (a number between 0-100)") \
5656
range(0, 100) \
5757
\
58-
develop(uintx, GCWorkerDelayMillis, 0, \
59-
"Delay in scheduling GC workers (in milliseconds)") \
60-
\
6158
product(bool, PSChunkLargeArrays, true, \
6259
"Process large arrays in chunks")
6360

‎src/hotspot/share/gc/parallel/psCardTable.cpp

-11
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,6 @@ void PSCardTable::scavenge_contents_parallel(ObjectStartArray* start_array,
165165
uint stripe_number,
166166
uint stripe_total) {
167167
int ssize = 128; // Naked constant! Work unit = 64k.
168-
int dirty_card_count = 0;
169168

170169
// It is a waste to get here if empty.
171170
assert(sp->bottom() < sp->top(), "Should not be called if empty");
@@ -195,16 +194,6 @@ void PSCardTable::scavenge_contents_parallel(ObjectStartArray* start_array,
195194
HeapWord* slice_start = addr_for(worker_start_card);
196195
HeapWord* slice_end = MIN2((HeapWord*) sp_top, addr_for(worker_end_card));
197196

198-
#ifdef ASSERT
199-
if (GCWorkerDelayMillis > 0) {
200-
// Delay 1 worker so that it proceeds after all the work
201-
// has been completed.
202-
if (stripe_number < 2) {
203-
os::naked_sleep(GCWorkerDelayMillis);
204-
}
205-
}
206-
#endif
207-
208197
// If there are not objects starting within the chunk, skip it.
209198
if (!start_array->object_starts_in_range(slice_start, slice_end)) {
210199
continue;

‎src/hotspot/share/gc/parallel/psPromotionManager.inline.hpp

-7
Original file line numberDiff line numberDiff line change
@@ -219,13 +219,6 @@ inline oop PSPromotionManager::copy_unmarked_to_survivor_space(oop o,
219219

220220
HeapWord* lab_base = old_gen()->allocate(OldPLABSize);
221221
if(lab_base != NULL) {
222-
#ifdef ASSERT
223-
// Delay the initialization of the promotion lab (plab).
224-
// This exposes uninitialized plabs to card table processing.
225-
if (GCWorkerDelayMillis > 0) {
226-
os::naked_sleep(GCWorkerDelayMillis);
227-
}
228-
#endif
229222
_old_lab.initialize(MemRegion(lab_base, OldPLABSize));
230223
// Try the old lab allocation again.
231224
new_obj = cast_to_oop(_old_lab.allocate(new_obj_size));

0 commit comments

Comments
 (0)
Please sign in to comment.