suzanne.soy/racket
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix an incorrect simpliciation in the GC

Browse Source 
I had tried to simplify the "generation 0" allocation function to
always use `GEN0_PAGE_SIZE`, but "generation 0" is also used for place
messages, in which case a much smaller size should be used.

The "place-in-channel-fnl.rkt" test exposed this problem.
This commit is contained in:
Matthew Flatt 2015-07-17 09:47:22 -06:00
parent e8ca83ad78
commit 053cfc0ddb
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
racket/src/racket/gc2/newgc.c
View File

@ -1340,7 +1340,10 @@ inline static void gen0_sync_page_size_from_globals(NewGC *gc) {
} }
inline static void gen0_allocate_and_setup_new_page(NewGC *gc) { inline static void gen0_allocate_and_setup_new_page(NewGC *gc) {
mpage *new_mpage = gen0_create_new_nursery_mpage(gc, GEN0_PAGE_SIZE); /* Note that gc->gen0.page_alloc_size is not necessarily
GEN0_PAGE_SIZE. In the case of a place-message allocator, a
smaller size is used. */
mpage *new_mpage = gen0_create_new_nursery_mpage(gc, gc->gen0.page_alloc_size);
/* push page */ /* push page */
new_mpage->prev = gc->gen0.curr_alloc_page; new_mpage->prev = gc->gen0.curr_alloc_page;