• Austin Clements's avatar
    runtime: enable GC assists ASAP · 510fd135
    Austin Clements authored
    Currently the GC coordinator enables GC assists at the same time it
    enables background mark workers, after the concurrent scan phase is
    done. However, this means a rapidly allocating mutator has the entire
    scan phase during which to allocate beyond the heap trigger and
    potentially beyond the heap goal with no back-pressure from assists.
    This prevents the feedback system that's supposed to keep the heap
    size under the heap goal from doing its job.
    
    Fix this by enabling mutator assists during the scan phase. This is
    safe because the write barrier is already enabled and globally
    acknowledged at this point.
    
    There's still a very small window between when the heap size reaches
    the heap trigger and when the GC coordinator is able to stop the world
    during which the mutator can allocate unabated. This allows *very*
    rapidly allocator mutators like TestTraceStress to still occasionally
    exceed the heap goal by a small amount (~20 MB at most for
    TestTraceStress). However, this seems like a corner case.
    
    Fixes #11677.
    
    Change-Id: I0f80d949ec82341cd31ca1604a626efb7295a819
    Reviewed-on: https://go-review.googlesource.com/12674Reviewed-by: 's avatarRuss Cox <rsc@golang.org>
    510fd135
Name
Last commit
Last update
..
archive Loading commit data...
bufio Loading commit data...
builtin Loading commit data...
bytes Loading commit data...
cmd Loading commit data...
compress Loading commit data...
container Loading commit data...
crypto Loading commit data...
database/sql Loading commit data...
debug Loading commit data...
encoding Loading commit data...
errors Loading commit data...
expvar Loading commit data...
flag Loading commit data...
fmt Loading commit data...
go Loading commit data...
hash Loading commit data...
html Loading commit data...
image Loading commit data...
index/suffixarray Loading commit data...
internal Loading commit data...
io Loading commit data...
log Loading commit data...
math Loading commit data...
mime Loading commit data...
net Loading commit data...
os Loading commit data...
path Loading commit data...
reflect Loading commit data...
regexp Loading commit data...
runtime Loading commit data...
sort Loading commit data...
strconv Loading commit data...
strings Loading commit data...
sync Loading commit data...
syscall Loading commit data...
testing Loading commit data...
text Loading commit data...
time Loading commit data...
unicode Loading commit data...
unsafe Loading commit data...
Make.dist Loading commit data...
all.bash Loading commit data...
all.bat Loading commit data...
all.rc Loading commit data...
androidtest.bash Loading commit data...
bootstrap.bash Loading commit data...
buildall.bash Loading commit data...
clean.bash Loading commit data...
clean.bat Loading commit data...
clean.rc Loading commit data...
iostest.bash Loading commit data...
make.bash Loading commit data...
make.bat Loading commit data...
make.rc Loading commit data...
nacltest.bash Loading commit data...
race.bash Loading commit data...
race.bat Loading commit data...
run.bash Loading commit data...
run.bat Loading commit data...
run.rc Loading commit data...