1
0
mirror of https://github.com/golang/go synced 2024-11-17 05:14:56 -07:00

runtime: skip known TestSegv failure mode on linux-mips64le-mengzhuo

Also adjust other skips to actually call t.Skip, so that the test
correctly shows as skipped instead of passing.

For #50605

Change-Id: Ied482f231a879224c5a92e2c47a6b21c1593a7da
Reviewed-on: https://go-review.googlesource.com/c/go/+/378554
Trust: Bryan Mills <bcmills@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
This commit is contained in:
Bryan C. Mills 2022-01-14 09:23:25 -05:00 committed by Bryan Mills
parent 17b2fb1b65
commit 9c6ecc49ca

View File

@ -622,17 +622,19 @@ func TestSegv(t *testing.T) {
// No runtime errors like "runtime: unknown pc".
switch runtime.GOOS {
case "darwin", "illumos", "solaris":
// TODO(golang.org/issue/49182): Skip, runtime
// throws while attempting to generate
// traceback.
return
// Runtime sometimes throws when generating the traceback.
testenv.SkipFlaky(t, 49182)
case "linux":
if runtime.GOARCH == "386" {
// TODO(golang.org/issue/50504): Skip,
// runtime throws while attempting to
// generate traceback from VDSO call
// via asmcgocall.
return
// Runtime throws when generating a traceback from
// a VDSO call via asmcgocall.
testenv.SkipFlaky(t, 50504)
}
if testenv.Builder() == "linux-mips64le-mengzhuo" && strings.Contains(got, "runtime: unknown pc") {
// Runtime sometimes throw "unknown pc" when generating the traceback.
// Curiously, that doesn't seem to happen on the linux-mips64le-rtrk
// builder.
testenv.SkipFlaky(t, 50605)
}
}
nowant := "runtime: "