diff options
author | Koichi Sasada <ko1@atdot.net> | 2020-12-10 10:38:30 +0900 |
---|---|---|
committer | Koichi Sasada <ko1@atdot.net> | 2020-12-10 13:05:43 +0900 |
commit | 554c0949777cd495e5a1296bd6719fcf508a70d0 (patch) | |
tree | 2a859f6ad1ccd095e2960c5c9106e54ba9cc1548 /gc.c | |
parent | eafe000af3d363f6cb65be99edcfccedf35d801c (diff) |
set min/maximum free slots relative to ractor cnt
A program with multiple ractors can consume more objects per
unit time, so this patch set minimum/maximum free_slots to
relative to ractors count (upto 8).
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/3875
Diffstat (limited to 'gc.c')
-rw-r--r-- | gc.c | 12 |
1 files changed, 9 insertions, 3 deletions
@@ -7298,11 +7298,15 @@ gc_marks_finish(rb_objspace_t *objspace) size_t max_free_slots = (size_t)(total_slots * gc_params.heap_free_slots_max_ratio); size_t min_free_slots = (size_t)(total_slots * gc_params.heap_free_slots_min_ratio); int full_marking = is_full_marking(objspace); + const int r_cnt = GET_VM()->ractor.cnt; + const int r_mul = r_cnt > 8 ? 8 : r_cnt; // upto 8 GC_ASSERT(heap->total_slots >= objspace->marked_slots); /* setup free-able page counts */ - if (max_free_slots < gc_params.heap_init_slots) max_free_slots = gc_params.heap_init_slots; + if (max_free_slots < gc_params.heap_init_slots * r_mul) { + max_free_slots = gc_params.heap_init_slots * r_mul; + } if (sweep_slots > max_free_slots) { heap_pages_freeable_pages = (sweep_slots - max_free_slots) / HEAP_PAGE_OBJ_LIMIT; @@ -7311,8 +7315,10 @@ gc_marks_finish(rb_objspace_t *objspace) heap_pages_freeable_pages = 0; } - /* check free_min */ - if (min_free_slots < gc_params.heap_free_slots) min_free_slots = gc_params.heap_free_slots; + /* check free_min */ + if (min_free_slots < gc_params.heap_free_slots * r_mul) { + min_free_slots = gc_params.heap_free_slots * r_mul; + } if (sweep_slots < min_free_slots) { if (!full_marking) { |