From: Dmitry Vyukov Date: Tue, 10 Mar 2015 15:44:41 +0000 (+0300) Subject: runtime/pprof: fix trace test X-Git-Tag: go1.5beta1~1641 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=5471e02338becccdd5df76c96bfeb467c25f003c;p=gostls13.git runtime/pprof: fix trace test Some of the trace stacks are OS-dependent due to OS-specific code in net package. Check these stacks only on subset of OSes. Change-Id: If95e4485839f4120fd6395725374c3a2f8706dfc Reviewed-on: https://go-review.googlesource.com/7300 Run-TryBot: Dmitry Vyukov TryBot-Result: Gobot Gobot Reviewed-by: Dmitry Vyukov --- diff --git a/src/runtime/pprof/trace_stack_test.go b/src/runtime/pprof/trace_stack_test.go index 79ff4ed40d..ef55d52610 100644 --- a/src/runtime/pprof/trace_stack_test.go +++ b/src/runtime/pprof/trace_stack_test.go @@ -221,24 +221,29 @@ func TestTraceSymbolize(t *testing.T) { frame{"runtime/pprof_test.TestTraceSymbolize", 115}, frame{"testing.tRunner", 0}, }}, - eventDesc{trace.EvGoBlockNet, []frame{ - frame{"net.(*netFD).accept", 0}, - frame{"net.(*TCPListener).AcceptTCP", 0}, - frame{"net.(*TCPListener).Accept", 0}, - frame{"runtime/pprof_test.TestTraceSymbolize.func10", 86}, - }}, eventDesc{trace.EvGoSleep, []frame{ frame{"time.Sleep", 0}, frame{"runtime/pprof_test.TestTraceSymbolize", 106}, frame{"testing.tRunner", 0}, }}, - eventDesc{trace.EvGoSysCall, []frame{ - frame{"syscall.read", 0}, - frame{"syscall.Read", 0}, - frame{"os.(*File).read", 0}, - frame{"os.(*File).Read", 0}, - frame{"runtime/pprof_test.TestTraceSymbolize.func11", 100}, - }}, + } + // Stacks for the following events are OS-dependent due to OS-specific code in net package. + if runtime.GOOS != "windows" && runtime.GOOS != "plan9" { + want = append(want, []eventDesc{ + eventDesc{trace.EvGoBlockNet, []frame{ + frame{"net.(*netFD).accept", 0}, + frame{"net.(*TCPListener).AcceptTCP", 0}, + frame{"net.(*TCPListener).Accept", 0}, + frame{"runtime/pprof_test.TestTraceSymbolize.func10", 86}, + }}, + eventDesc{trace.EvGoSysCall, []frame{ + frame{"syscall.read", 0}, + frame{"syscall.Read", 0}, + frame{"os.(*File).read", 0}, + frame{"os.(*File).Read", 0}, + frame{"runtime/pprof_test.TestTraceSymbolize.func11", 100}, + }}, + }...) } matched := make([]bool, len(want)) for _, ev := range events {