Skip to content

Commit 16da5f2

Browse files
Revert "Double sweep byte budget to preserve heap 1 behavior"
This reverts commit c617c5e.
1 parent 7ddcaa4 commit 16da5f2

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

gc/default/default.c

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1053,12 +1053,11 @@ total_final_slots_count(rb_objspace_t *objspace)
10531053
* so larger slot pools (which are less heavily used) naturally get
10541054
* fewer slots swept per step.
10551055
*
1056-
* Baseline: preserves the historical sweep/pool slot counts (2048/1024)
1057-
* for the second heap (2 * RVALUE_SLOT_SIZE = 80 bytes). The smallest
1058-
* heap sweeps more per step (4096/2048 slots), larger heaps taper down.
1056+
* Baseline: 2048 slots * RVALUE_SLOT_SIZE = 2048 * 40 = 81920 bytes,
1057+
* preserving the historical behavior for the smallest heap.
10591058
*/
1060-
#define GC_INCREMENTAL_SWEEP_BYTES (2048 * RVALUE_SLOT_SIZE * 2)
1061-
#define GC_INCREMENTAL_SWEEP_POOL_BYTES (1024 * RVALUE_SLOT_SIZE * 2)
1059+
#define GC_INCREMENTAL_SWEEP_BYTES (2048 * RVALUE_SLOT_SIZE)
1060+
#define GC_INCREMENTAL_SWEEP_POOL_BYTES (1024 * RVALUE_SLOT_SIZE)
10621061
#define is_lazy_sweeping(objspace) (GC_ENABLE_LAZY_SWEEP && has_sweeping_pages(objspace))
10631062
/* In lazy sweeping or the previous incremental marking finished and did not yield a free page. */
10641063
#define needs_continue_sweeping(objspace, heap) \

0 commit comments

Comments
 (0)