languages/go
languages/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-05 18:36:08 +00:00
Code Issues Projects Releases Packages Wiki Activity

runtime: fix riscv64 sigaction mask field offset

Browse source 
The Linux kernel for riscv64 does not include an sa_restorer field on
its sigaction struct, and expects sa_mask to come immediately after the
sa_flags field. Arrange the fields of the sigaction struct that are
known to the kernel so they appear at the correct byte offsets, and so
they agree with the output of "go tool cgo -godefs".

Follow the example set by the mips/mipsle port to leave the sa_restorer
field in place, but at an offset where it won't hurt anything.

Fixes #49709

Change-Id: I9bb0d7dbd7439d07e3a204461c7d790f33fd4977
Reviewed-on: https://go-review.googlesource.com/c/go/+/367635
Run-TryBot: Rhys Hiltner <rhys@justin.tv>
Reviewed-by: Cherry Mui <cherryyz@google.com>
Reviewed-by: Joel Sing <joel@sing.id.au>
Trust: Ian Lance Taylor <iant@golang.org>
This commit is contained in:
Rhys Hiltner 2021-11-29 16:48:34 -08:00 committed by Cherry Mui
parent 029dfbcc83
commit 2c5d2083e4
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/runtime/defs_linux_riscv64.go
View file

@ -122,10 +122,12 @@ func (tv *timeval) set_usec(x int32) {
}
type sigactiont struct {
sa_handler uintptr
sa_flags uint64
sa_handler uintptr
sa_flags uint64
sa_mask uint64
// Linux on riscv64 does not have the sa_restorer field, but the setsig
// function references it (for x86). Not much harm to include it at the end.
sa_restorer uintptr
sa_mask uint64
}
type siginfoFields struct {