Skip to content

Add SHM_ANON for FreeBSD #837

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

Merged
merged 1 commit into from
Nov 6, 2017
Merged

Conversation

valpackett
Copy link
Contributor

This is a constant for shm_open that is used instead of the address to create an anonymous shared memory mapping.

https://github.com/freebsd/freebsd/blob/a9656e45a9d1cc0fc5343dd941106e0b823ff770/sys/sys/mman.h#L168

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Nov 6, 2017

📌 Commit 715ed8c has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Nov 6, 2017

⌛ Testing commit 715ed8c with merge 17e0106...

bors added a commit that referenced this pull request Nov 6, 2017
Add SHM_ANON for FreeBSD

This is a constant for `shm_open` that is used instead of the address to create an anonymous shared memory mapping.

https://github.com/freebsd/freebsd/blob/a9656e45a9d1cc0fc5343dd941106e0b823ff770/sys/sys/mman.h#L168
@bors
Copy link
Contributor

bors commented Nov 6, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 17e0106 to master...

@bors bors merged commit 715ed8c into rust-lang:master Nov 6, 2017
danielverkamp pushed a commit to danielverkamp/libc that referenced this pull request Apr 28, 2020
* Fix race condition in feature cache on 32 platforms

If we observe that the second word is initialized, we can't really
assume that the first is initialized as well. So check each word
separately.

* Use stronger atomic ordering

Better SeqCst than sorry!

* Use two caches on x64 for simplicity
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants