]> Cypherpunks repositories - gostls13.git/commit
runtime: replace assist sleep loop with park/ready
authorAustin Clements <austin@google.com>
Thu, 15 Oct 2015 01:31:33 +0000 (21:31 -0400)
committerAustin Clements <austin@google.com>
Tue, 3 Nov 2015 15:53:25 +0000 (15:53 +0000)
commit15aa6bbd5aa2cc8d7f2de91255f6a331ec5c16d4
tree77354c5a649f7925e929830bc2acd09b07cf97d9
parent0ca4488cc1b91caef2667b6b58ff1562d44a2c74
runtime: replace assist sleep loop with park/ready

GC assists must block until the assist can be satisfied (either
through stealing credit or doing work) or the GC cycle ends.
Currently, this is implemented as a retry loop with a 100 µs delay.
This obviously isn't ideal, as it wastes CPU and delays mutator
execution. It also has the somewhat peculiar downside that sleeping a
G requires allocation, and this requires working around recursive
allocation.

Replace this timed delay with a proper scheduling queue. When an
assist can't be satisfied immediately, it adds the allocating G to a
queue and parks it. Any time background scan credit is flushed, it
consults this queue, directly satisfies the debt of queued assists,
and wakes up satisfied assists before flushing any remaining credit to
the background credit pool.

No effect on the go1 benchmarks. Slightly speeds up the garbage
benchmark.

name              old time/op  new time/op  delta
XBenchGarbage-12  5.81ms ± 1%  5.72ms ± 4%  -1.65%  (p=0.011 n=20+20)

Updates #12041.

Change-Id: I8ee3b6274dd097b12b10a8030796a958a4b0e7b7
Reviewed-on: https://go-review.googlesource.com/15890
Reviewed-by: Rick Hudson <rlh@golang.org>
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
src/runtime/mgc.go
src/runtime/mgcmark.go