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: sys_namespace syscall has been removed in pepper_42 #11961

Closed
davecheney opened this issue Jul 31, 2015 · 4 comments
Closed

syscall: sys_namespace syscall has been removed in pepper_42 #11961

davecheney opened this issue Jul 31, 2015 · 4 comments
Assignees
Milestone

Comments

@davecheney
Copy link
Contributor

@davecheney davecheney commented Jul 31, 2015

For GOOS=nacl, we use the srpc service to communicate with the nacl sandbox to get things like random data. This is implemented in a quasi syscall, called sys_namespace (syscall/tables_nacl.go).

This syscall is present in pepper_41, but missing in pepper_44, which the current stable pepper version.

For Go 1.6, we should find out where this service has gone, and if there is a replacement. In the interim, nacl users should stick to pepper_41 or lower.

@davecheney davecheney added the OS-NaCl label Jul 31, 2015
@davecheney davecheney added this to the Go1.6 milestone Jul 31, 2015
@davecheney
Copy link
Contributor Author

@davecheney davecheney commented Jul 31, 2015

The nacl release notes are obtuse on the fate of sRPC, https://developer.chrome.com/native-client/sdk/release-notes

@davecheney
Copy link
Contributor Author

@davecheney davecheney commented Jul 31, 2015

The SRPC name service was removed in https://codereview.chromium.org/891603002

The tracking bug is https://code.google.com/p/nativeclient/issues/detail?id=3864

We use SRPC to get random bytes, only, which is handy because there is a new syscall which provides just that service, https://codereview.chromium.org/537543003

For 1.6, we should remove syscall/srpc_nacl.go and adjust our use of secure random to call this new get random bytes syscall.

/cc @rsc @minux

@mikioh mikioh changed the title nacl: sys_namespace syscall has been removed in pepper_44 syscall: sys_namespace syscall has been removed in pepper_44 Aug 1, 2015
@davecheney davecheney self-assigned this Aug 2, 2015
@davecheney davecheney changed the title syscall: sys_namespace syscall has been removed in pepper_44 syscall: sys_namespace syscall has been removed in pepper_43 Aug 7, 2015
@davecheney davecheney changed the title syscall: sys_namespace syscall has been removed in pepper_43 syscall: sys_namespace syscall has been removed in pepper_42 Aug 7, 2015
@gopherbot
Copy link

@gopherbot gopherbot commented Aug 7, 2015

CL https://golang.org/cl/13341 mentions this issue.

davecheney added a commit that referenced this issue Aug 10, 2015
Fixes #12062
Updates #11961

The sRPC nameservice was removed in pepper 42. For Go 1.5 stipulate
that NaCl requires pepper 41 only.

Change-Id: Ic88ba342d41f673391efaa96fb581712fa10a0fd
Reviewed-on: https://go-review.googlesource.com/13341
Reviewed-by: Andrew Gerrand <adg@golang.org>
@gopherbot
Copy link

@gopherbot gopherbot commented Aug 27, 2015

CL https://golang.org/cl/13958 mentions this issue.

@golang golang locked and limited conversation to collaborators Sep 4, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.