]> Cypherpunks repositories - gostls13.git/commit
runtime: use StorepNoWB instead of atomicstorep in netpoll
authorDmitry Vyukov <dvyukov@google.com>
Wed, 31 Oct 2018 18:36:21 +0000 (19:36 +0100)
committerDmitry Vyukov <dvyukov@google.com>
Fri, 2 Nov 2018 13:38:56 +0000 (13:38 +0000)
commit56421f26ef8526427e034639510a162f3b40d825
treeced86de8c5a70d40766b4a726179d2975b9a7800
parentcdad4080695f09d79c403fd3827ef0ba50c399a8
runtime: use StorepNoWB instead of atomicstorep in netpoll

We only need the memory barrier from these stores,
and we only store nil over nil or over a static function value.
The write barrier is unnecessary.

name                  old time/op  new time/op  delta
TCP4OneShotTimeout-6  17.0µs ± 0%  17.0µs ± 0%  -0.43%  (p=0.032 n=5+5)
SetReadDeadline-6      205ns ± 1%   205ns ± 1%    ~     (p=0.683 n=5+5)

Update #25729

Change-Id: I66c097a1db7188697ddfc381f31acec053dfed2c
Reviewed-on: https://go-review.googlesource.com/c/146345
Run-TryBot: Dmitry Vyukov <dvyukov@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/runtime/netpoll.go