Descriptionnet: fix data races on deadline vars
Fixes issue 4434.
This proposal replaces the previous CL 6855110. Due to issue 599, 64-bit atomic operations should probably be avoided, so use a sync.Mutex instead.
Benchmark comparisons against 025b9d070a85 on linux/386:
CL 6855110:
benchmark old ns/op new ns/op delta
BenchmarkTCPOneShot 710024 727409 +2.45%
BenchmarkTCPOneShotTimeout 758178 768620 +1.38%
BenchmarkTCPPersistent 223464 228058 +2.06%
BenchmarkTCPPersistentTimeout 234494 242600 +3.46%
This proposal:
benchmark old ns/op new ns/op delta
BenchmarkTCPOneShot 710024 718492 +1.19%
BenchmarkTCPOneShotTimeout 758178 748783 -1.24%
BenchmarkTCPPersistent 223464 227628 +1.86%
BenchmarkTCPPersistentTimeout 234494 238321 +1.63%
Patch Set 1 #Patch Set 2 : diff -r 025b9d070a85 https://code.google.com/p/go #Patch Set 3 : diff -r 025b9d070a85 https://code.google.com/p/go #Patch Set 4 : diff -r 025b9d070a85 https://code.google.com/p/go #Patch Set 5 : diff -r 025b9d070a85 https://code.google.com/p/go #
Total comments: 2
Patch Set 6 : diff -r c8cb5d283270 https://code.google.com/p/go #Patch Set 7 : diff -r a32219a715c5 https://go.googlecode.com/hg/ #
MessagesTotal messages: 11
|