Skip to content

Commit 2aef675

Browse files
committed
runtime: ignore sigaction error on Linux if it is for SIGRTMAX
The Go runtime registers a handler for every signal. This prevents Go binaries from working on QEMU in user-emulation mode, since the hacky way QEMU implements signals on Linux assumes that no-one uses signal 64 (SIGRTMAX). In the past, we had a workaround in the runtime to prevent crashes on start-up when running on QEMU: golang.org/cl/124900043 golang.org/cl/16853 but it went lost during the 1.11 dev cycle. More precisely, the test for SIGRTMAX was dropped in CL 18150 when we stopped testing the result of sigaction in the Linux implementation of setsig. That change was made to avoid a stack split overflow because code started calling setsig from nosplit functions. Then in CL 99077 we started testing the result of sigaction again, this time using systemstack to avoid to stack split overflow. When this test was added back, we did not bring back the test of SIGRTMAX. As a result, Go1.10 binaries work on QEMU, while 1.11 binaries immediately crash on startup. This change restores the QEMU workaround. Updates #24656 Change-Id: I46380b1e1b4bf47db7bc7b3d313f00c4e4c11ea3 Reviewed-on: https://go-review.googlesource.com/111176 Reviewed-by: Ian Lance Taylor <iant@golang.org>
1 parent eed79f4 commit 2aef675

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/runtime/os_linux.go

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -415,10 +415,14 @@ func (c *sigctxt) fixsigcode(sig uint32) {
415415
//go:nosplit
416416
func sysSigaction(sig uint32, new, old *sigactiont) {
417417
if rt_sigaction(uintptr(sig), new, old, unsafe.Sizeof(sigactiont{}.sa_mask)) != 0 {
418-
// Use system stack to avoid split stack overflow on ppc64/ppc64le.
419-
systemstack(func() {
420-
throw("sigaction failed")
421-
})
418+
// Workaround for bug in Qemu user mode emulation. (qemu
419+
// rejects rt_sigaction of signal 64, SIGRTMAX).
420+
if sig != 64 {
421+
// Use system stack to avoid split stack overflow on ppc64/ppc64le.
422+
systemstack(func() {
423+
throw("sigaction failed")
424+
})
425+
}
422426
}
423427
}
424428

0 commit comments

Comments
 (0)