1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Do not free too many pages.

Browse Source 
Sweep step checks `heap_pages_freeable_pages`, so compaction should do
the same.
This commit is contained in:
Aaron Patterson 2019-10-07 12:28:21 -07:00
parent 058db33c5e
commit 6abcd35762
No known key found for this signature in database
GPG Key ID: 953170BCB4FFAFC6
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gc.c
View File

@ -8409,10 +8409,12 @@ gc_compact_after_gc(rb_objspace_t *objspace, int use_toward_empty, int use_doubl
RMOVED(moved_list)->next = 0; RMOVED(moved_list)->next = 0;
page->free_slots++; page->free_slots++;
heap_page_add_freeobj(objspace, page, moved_list); heap_page_add_freeobj(objspace, page, moved_list);
if (page->free_slots == page->total_slots) { if (page->free_slots == page->total_slots && heap_pages_freeable_pages > 0) {
heap_pages_freeable_pages--; heap_pages_freeable_pages--;
heap_unlink_page(objspace, heap_eden, page); heap_unlink_page(objspace, heap_eden, page);
heap_add_page(objspace, heap_tomb, page); heap_add_page(objspace, heap_tomb, page);
} else if (page->free_slots == page->total_slots) {
page->free_next = NULL;
} }
objspace->profile.total_freed_objects++; objspace->profile.total_freed_objects++;
moved_list = next_moved; moved_list = next_moved;