rpc/stream/vc/reader: Remove defer statement.
Found an unecessary defer unlock statement, removing it seems to have
some small speedups :)
v23 go test -bench=Benchmark_100KB -cpu=1 -benchtime=20s v.io/x/ref/runtime/internal/rpc/benchmark
Benchmark_100KB 10000 2269162 ns/op 88.14 MB/s
--- Histogram (unit: ms)
Count: 10000 Min: 1 Max: 8 Avg: 1.58
------------------------------------------------------------
[ 1, 2) 6081 60.8% 60.8% ######
[ 2, 3) 2124 21.2% 82.0% ##
[ 3, 4) 1755 17.6% 99.6% ##
[ 4, 5) 32 0.3% 99.9%
[ 5, 6) 6 0.1% 100.0%
[ 6, 7) 1 0.0% 100.0%
[ 7, 9) 1 0.0% 100.0%
[ 9, inf) 0 0.0% 100.0%
ok v.io/x/ref/runtime/internal/rpc/benchmark 23.270s
Benchmark_100KB 10000 2242286 ns/op 89.19 MB/s
--- Histogram (unit: ms)
Count: 10000 Min: 1 Max: 4 Avg: 1.57
------------------------------------------------------------
[ 1, 2) 6182 61.8% 61.8% ######
[ 2, 3) 1992 19.9% 81.7% ##
[ 3, 4) 1814 18.1% 99.9% ##
[ 4, inf) 12 0.1% 100.0%
ok v.io/x/ref/runtime/internal/rpc/benchmark 23.035s
Change-Id: I4c910079e650282d100a8d0ef969abe88fc12d7c
diff --git a/runtime/internal/rpc/stream/vc/reader.go b/runtime/internal/rpc/stream/vc/reader.go
index 80f1c9b..127d451 100644
--- a/runtime/internal/rpc/stream/vc/reader.go
+++ b/runtime/internal/rpc/stream/vc/reader.go
@@ -102,8 +102,8 @@
func (r *reader) SetDeadline(deadline <-chan struct{}) {
r.mu.Lock()
- defer r.mu.Unlock()
r.deadline = deadline
+ r.mu.Unlock()
}
func (r *reader) BytesRead() uint32 {
