Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

syscall: RawSyscall should panic on solaris #20833

Closed
binarycrusader opened this issue Jun 28, 2017 · 2 comments
Closed

syscall: RawSyscall should panic on solaris #20833

binarycrusader opened this issue Jun 28, 2017 · 2 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. OS-Solaris
Milestone

Comments

@binarycrusader
Copy link
Contributor

As noted in #20832, RawSyscall should panic on solaris as raw system calls are never actually possible on Solaris.

From syscall_solaris.go:

// This is syscall.RawSyscall, it exists to satisfy some build dependency,
// but it doesn't work correctly.
//
// DO NOT USE!
//
// TODO(aram): make this panic once we stop calling fcntl(2) in net using it.
func syscall_rawsyscall(trap, a1, a2, a3 uintptr) (r1, r2, err uintptr) {

From syscall/exec_solaris.go:

// We call hand-crafted syscalls, implemented in
// ../runtime/syscall_solaris.go, rather than generated libc wrappers
// because we need to avoid lazy-loading the functions (might malloc,
// split the stack, or acquire mutexes). We can't call RawSyscall
// because it's not safe even for BSD-subsystem calls.

Any existing use of RawSyscall should be excised first, and then it should be changed to panic on Solaris.

This is definitely for 1.10+

@bradfitz bradfitz added this to the Go1.10 milestone Jun 28, 2017
@bradfitz bradfitz added NeedsFix The path to resolution is known, but the work has not been done. OS-Solaris labels Jun 28, 2017
@davecb
Copy link

davecb commented Jun 29, 2017

I'm an old Solarii from the ABI team who "owned" the standard libraries, if I can help with this, loop me in.

@gopherbot
Copy link

Change https://golang.org/cl/82636 mentions this issue: runtime: make RawSyscall panic on Solaris

@golang golang locked and limited conversation to collaborators Dec 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. OS-Solaris
Projects
None yet
Development

No branches or pull requests

4 participants