diff options
| author | Matthew Dempsky <mdempsky@google.com> | 2015-02-25 14:41:21 +0900 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2015-02-26 08:59:24 +0000 |
| commit | 3c8a89daf3a05a1dd98075a733db7a20bef2dc5c (patch) | |
| tree | db477dc0bc924ff94ea7b334c5a9233df9c65a17 /src/runtime/signal_386.go | |
| parent | a32dd832530a176b5b45f06c3a97f52383227480 (diff) | |
| download | go-3c8a89daf3a05a1dd98075a733db7a20bef2dc5c.tar.gz go-3c8a89daf3a05a1dd98075a733db7a20bef2dc5c.zip | |
runtime: simplify CPU profiling code
This makes Go's CPU profiling code somewhat more idiomatic; e.g.,
using := instead of forward declaring variables, using "int" for
element counts instead of "uintptr", and slices instead of C-style
pointer+length. This makes the code easier to read and eliminates a
lot of type conversion clutter.
Additionally, in sigprof we can collect just maxCPUProfStack stack
frames, as cpuprof won't use more than that anyway.
Change-Id: I0235b5ae552191bcbb453b14add6d8c01381bd06
Reviewed-on: https://go-review.googlesource.com/6072
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dmitry Vyukov <dvyukov@google.com>
Diffstat (limited to 'src/runtime/signal_386.go')
| -rw-r--r-- | src/runtime/signal_386.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/signal_386.go b/src/runtime/signal_386.go index 5336a4330a..0ca593de02 100644 --- a/src/runtime/signal_386.go +++ b/src/runtime/signal_386.go @@ -29,7 +29,7 @@ func sighandler(sig uint32, info *siginfo, ctxt unsafe.Pointer, gp *g) { c := &sigctxt{info, ctxt} if sig == _SIGPROF { - sigprof((*byte)(unsafe.Pointer(uintptr(c.eip()))), (*byte)(unsafe.Pointer(uintptr(c.esp()))), nil, gp, _g_.m) + sigprof(uintptr(c.eip()), uintptr(c.esp()), 0, gp, _g_.m) return } |