]> Cypherpunks repositories - gostls13.git/commit
runtime: eagerly share GC work buffers
authorRick Hudson <rlh@golang.org>
Wed, 6 Jan 2016 22:07:58 +0000 (17:07 -0500)
committerRick Hudson <rlh@golang.org>
Mon, 11 Jan 2016 18:23:56 +0000 (18:23 +0000)
commit9439fa107829b6626eb6cef2a3549b1fc9a6f974
treeda43f436d46f4923405eab6b061441d183c95e50
parent109d54a32d15b805769d4c05e78367f126a8d7f0
runtime: eagerly share GC work buffers

Currently, due to an oversight, we only balance work buffers
in background and idle workers and not in assists. As a
result, in assist-heavy workloads, assists are likely to tie
up large work buffers in per-P caches increasing the
likelihood that the global list will be empty. This increases
the likelihood that other GC workers will exit and assists
will block, slowing down the system as a whole. Fix this by
eagerly balancing work buffers as soon as the assists notice
that the global buffers are empty. This makes it much more
likely that work will be immediately available to other
workers and assists.

This change reduces the garbage benchmark time by 39% and
fixes the regresssion seen at CL 15893 golang.org/cl/15893.

Garbage benchmark times before and after this CL.
Before GOPERF-METRIC:time=4427020
After  GOPERF-METRIC:time=2721645

Fixes #13827

Change-Id: I9cb531fb873bab4b69ce9c1617e30df6c49cdcfe
Reviewed-on: https://go-review.googlesource.com/18341
Reviewed-by: Austin Clements <austin@google.com>
src/runtime/mgcmark.go