diff options
| author | Michael Matloob <matloob@golang.org> | 2017-09-12 12:22:22 -0400 |
|---|---|---|
| committer | Michael Matloob <matloob@golang.org> | 2017-09-12 16:54:29 +0000 |
| commit | 96b1eff087bd29c43d60b27aad6e53c2ca7e67c7 (patch) | |
| tree | 21129d2978ab2768a873ec7d78a3e15d25befbaa /src/runtime/cpuprof.go | |
| parent | 76f4fd8a5251b4f63ea14a3c1e2fe2e78eb74f81 (diff) | |
| download | go-96b1eff087bd29c43d60b27aad6e53c2ca7e67c7.tar.gz go-96b1eff087bd29c43d60b27aad6e53c2ca7e67c7.zip | |
runtime: in cpuProfile.addExtra, set p.lostExtra to 0 after flush
After the number of lost extra events are written to the the cpuprof log,
the number of lost extra events should be set to zero, or else, the next
time time addExtra is logged, lostExtra will be overcounted. This change
resets lostExtra after its value is written to the log.
Fixes #21836
Change-Id: I8a6ac9c61e579e7a5ca7bdb0f3463f8ae8b9f863
Reviewed-on: https://go-review.googlesource.com/63270
Reviewed-by: Austin Clements <austin@google.com>
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/runtime/cpuprof.go')
| -rw-r--r-- | src/runtime/cpuprof.go | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/runtime/cpuprof.go b/src/runtime/cpuprof.go index fb841a9f3d..e00dcb1bbd 100644 --- a/src/runtime/cpuprof.go +++ b/src/runtime/cpuprof.go @@ -160,6 +160,7 @@ func (p *cpuProfile) addExtra() { funcPC(_ExternalCode) + sys.PCQuantum, } cpuprof.log.write(nil, 0, hdr[:], lostStk[:]) + p.lostExtra = 0 } } |