-
Notifications
You must be signed in to change notification settings - Fork 710
System V shared memory APIs (2nd draft) #2314
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
Open
IzawGithub
wants to merge
58
commits into
nix-rust:master
Choose a base branch
from
IzawGithub:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 39 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
096cb21
First try at implementing SystemV
IzawGithub 072352c
Add permission struct
IzawGithub adeaaa2
Add crude semaphore support
IzawGithub 135488e
Add documentation that was missing
IzawGithub ac357e2
Add PR changelog file
IzawGithub 3f6a4c3
Move SystemV to it's own file
IzawGithub 5047036
Fix not compiling on Musl because of missing struct
IzawGithub b428de0
Fix clippy unsafe ptr arg dereference lint
IzawGithub 07a6f33
Formatting parse
IzawGithub ac33ca0
Add missing import that I forgot to commit
IzawGithub 11aad71
Update doctest path that changed
IzawGithub d8db9fa
Add basic IPC test.
IzawGithub bd938e6
Fix using latest Rust feature that was unstable before
IzawGithub b9a5905
Forgot a formatting parse
IzawGithub 50916a4
Enable SystemV only for linux, but not Android
IzawGithub f3e4771
Update conditional compilation for GNU extension
IzawGithub 9c71d97
Fix macro not compiling for NetBSD
IzawGithub 5d0d37c
Fix moving shmid_ds to GNU extension when it's not
IzawGithub 315eb31
Fix mistake in conditional compilation that didn't disable SystemV fo…
IzawGithub 9374dcf
Add back cfg that disable some flags for BSD
IzawGithub bb07d5b
Final (I hope) formatting pass
IzawGithub 19c713d
Disable test for non systemV platform
IzawGithub 17561e6
Fix wrong changelog number and content
IzawGithub 2b76711
Remove first try at SystemV, replacing with refractor
IzawGithub 94e4e11
First try at refractoring SystemV to be more Rusty
IzawGithub aea8588
Add a feature gate
IzawGithub 02b0de2
Add missing doc
IzawGithub ed99763
Fix doctest copy and paste error
IzawGithub b614d81
Remove Clone derive, wasn't working correctly
IzawGithub 2629192
Update test to create an entirely new SharedMemory object
IzawGithub b7b8818
Misc formatting
IzawGithub 930f352
Remove unused static mutex for system_v
IzawGithub 4bd41cf
Add `Shm` wrapper for creating a shared memory segment
IzawGithub 2c714c2
Remove function that create a memory segment from `SharedMemory`
IzawGithub 6877712
Make `buf` parameter pass by ref
IzawGithub b7f6a09
Remove the `shmat` enum value that work with incompatible void ptr
IzawGithub d07c9b8
Update unit test
IzawGithub 5697ffd
Update imported function
IzawGithub 3d70ae2
Remove stable ptr_from_ref feature that isn't stable on Rust 1.69
IzawGithub 238fe62
Remove unused comment mark
IzawGithub 0ff180a
Add back `shmat` address. Use a raw pointer.
IzawGithub f5f2f59
Fix mismatched target os
IzawGithub 2803e31
Fix wrong comment formatting
IzawGithub 35c2f4e
Move `shmctl` function to `Shm`.
IzawGithub d05db61
Update doctest
IzawGithub ef1a839
Remove `id` from `SharedMemory` don't need it now that `shmctl` isn't…
IzawGithub e91efa3
Move test to independent file
IzawGithub d96d5ee
Follow Nix module convention
IzawGithub 6aa8740
Update independent test import
IzawGithub fd108e4
Rename `system_v` feature to `sysvipc`, like Linux
IzawGithub 2fde2c5
Update doctest import path
IzawGithub e531e8b
Remove `IPC_PRIVATE` value from `ShmgetFlag`, because it wasn't actua…
IzawGithub fee5acb
Formatting parse
IzawGithub 52473ce
Update comment of function that moved
IzawGithub d526780
Fix typo `test_smh` to `test_shm`
IzawGithub 400f71e
Add a test for unsafe `shmget`. Branch coverage is now 89.29%
IzawGithub d619c48
Improve code comment
IzawGithub ca0c5f9
Formatting parse
IzawGithub File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Add SystemV semaphore set and shared memory segments support. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
//! Interact with SystemV inter-process communication API | ||
|
||
#[cfg(any(bsd, target_os = "linux"))] | ||
pub mod sem; | ||
#[cfg(any(bsd, target_os = "linux"))] | ||
pub mod shm; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
//! Safe wrapper around a SystemV semaphore | ||
//! |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.