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

count phantom bytes as regular allocation

Browse Source 
The original idea was to count phantom bytes as "administrative
overhead", but issues discussed in #962 identified problems
with that idea. Finish shifting the accounting to treat
phantom bytes as payload allocation.
This commit is contained in:
Matthew Flatt 2015-09-18 12:26:32 -06:00
parent 4abedf63e8
commit 40f9467c07
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -5027,8 +5027,8 @@ static void garbage_collect(NewGC *gc, int force_full, int no_full, int switchin
int next_gc_full; int next_gc_full;
old_mem_use = gc->memory_in_use; old_mem_use = gc->memory_in_use; /* includes gc->phantom_count */
old_gen0 = gc->gen0.current_size; old_gen0 = gc->gen0.current_size + gc->gen0_phantom_count;
old_mem_allocated = mmu_memory_allocated(gc->mmu) + gc->phantom_count + gc->gen0_phantom_count; old_mem_allocated = mmu_memory_allocated(gc->mmu) + gc->phantom_count + gc->gen0_phantom_count;
TIME_DECLS(); TIME_DECLS();
@ -5272,8 +5272,8 @@ static void garbage_collect(NewGC *gc, int force_full, int no_full, int switchin
lmi->full = gc->gc_full, lmi->full = gc->gc_full,
lmi->pre_used = old_mem_use + old_gen0; lmi->pre_used = old_mem_use + old_gen0;
lmi->post_used = gc->memory_in_use; lmi->post_used = gc->memory_in_use;
lmi->pre_admin = old_mem_allocated+gc->phantom_count; lmi->pre_admin = old_mem_allocated;
lmi->post_admin = mmu_memory_allocated(gc->mmu); lmi->post_admin = mmu_memory_allocated(gc->mmu)+gc->phantom_count;
} }
GC_propagate_hierarchy_memory_use(); GC_propagate_hierarchy_memory_use();
#endif #endif