stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8243301: Shenandoah: ditch ShenandoahAllowMixedAllocs

Browse Source 
Reviewed-by: rkennke
This commit is contained in:
Aleksey Shipilev 2020-04-22 15:19:52 +02:00
parent 8e21a2a1e4
commit 91d204780d
2 changed files with 4 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/hotspot/share/gc/shenandoah/shenandoahFreeSet.cpp
View File

@ -125,18 +125,10 @@ HeapWord* ShenandoahFreeSet::allocate_single(ShenandoahAllocRequest& req, bool&
}
}
// Try to mix the allocation into the mutator view:
if (ShenandoahAllowMixedAllocs) {
for (size_t c = _mutator_rightmost + 1; c > _mutator_leftmost; c--) {
size_t idx = c - 1;
if (is_mutator_free(idx)) {
HeapWord* result = try_allocate_in(_heap->get_region(idx), req, in_new_region);
if (result != NULL) {
return result;
}
}
}
}
// No dice. Do not try to mix mutator and GC allocations, because
// URWM moves due to GC allocations would expose unparsable mutator
// allocations.
break;
}
default:

4
src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp
View File

@ -211,10 +211,6 @@
diagnostic(bool, ShenandoahElasticTLAB, true, \
"Use Elastic TLABs with Shenandoah") \
\
diagnostic(bool, ShenandoahAllowMixedAllocs, true, \
"Allow mixing mutator and collector allocations into a single " \
"region. Some heuristics enable/disable it for their needs") \
\
experimental(uintx, ShenandoahEvacReserve, 5, \
"How much of heap to reserve for evacuations. Larger values make "\
"GC evacuate more live objects on every cycle, while leaving " \