Skip to content

proposal: x/sys/unix: add MmapPtr/MunmapPtr #67669

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

Closed
ncruces opened this issue May 27, 2024 · 1 comment
Closed

proposal: x/sys/unix: add MmapPtr/MunmapPtr #67669

ncruces opened this issue May 27, 2024 · 1 comment
Labels
Milestone

Comments

@ncruces
Copy link
Contributor

ncruces commented May 27, 2024

Proposal Details

In github.com/ncruces/go-sqlite3 I'm go:linknaming mmap (I know, I'm sorry!) because I need unix.MAP_FIXED, which makes no sense for the version currently exported by either syscall or x/sys/unix.

Note that I'm currently link-naming syscall but could just as well use x/sys/unix (as I do for most everything else).

Link-naming is understandably discouraged, and on a path to being curtailed in #67401.

I know my package is pretty much irrelevant, but exposing the autogenerated bits that allow mmap to be called from a bunch of different Unixes would help. For example, I have no chance to test z/OS, but IBM is incentivized to add a portable version to x/sys/unix.

Hence, I'd like to propose x/sys/unix exports the uintptr version (or *byte?) of mmap, munmap and maybe mremap.

@gopherbot gopherbot added this to the Proposal milestone May 27, 2024
@ncruces
Copy link
Contributor Author

ncruces commented May 27, 2024

I managed to miss #56123, I'm sorry.

@ncruces ncruces closed this as completed May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants