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.

Sign up for GitHub

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

Jump to bottom

cleaner kernel-user exchange area manipulation #102

Merged
merged 11 commits into from
Dec 13, 2024
Merged

cleaner kernel-user exchange area manipulation #102

merged 11 commits into from
Dec 13, 2024

Conversation

pthierry-ledger
Copy link
Member

@pthierry-ledger pthierry-ledger commented Dec 12, 2024 •
edited
Loading

Based on crate-local trait implementation for uapi types only.
Allows to properly define which (UAPI) type can be emited to or received from the kernel.
Make the exchange area interaction fully opaque

  • cleaning exchange and define trait for systypes
  • make exchange module private to the uapi crate
  • validate C usage of the ffi_c copy_from_user/copy_to_user interface with autotest
  • update uapi documentation accordingly

@pthierry-ledger pthierry-ledger added api impact the external API of the component enhancement request for code evolution lang:rust Pull requests that update Rust code labels Dec 12, 2024
@pthierry-ledger pthierry-ledger force-pushed the rust-exchange branch 2 times, most recently from 01b1608 to b1f125c Compare December 13, 2024 09:55
@pthierry-ledger pthierry-ledger marked this pull request as ready for review December 13, 2024 11:51
fvalette-ledger
fvalette-ledger approved these changes Dec 13, 2024
View reviewed changes
uapi/src/ffi_c.rs Show resolved Hide resolved
@pthierry-ledger pthierry-ledger merged commit 1c3e98c into outpost-os:main Dec 13, 2024
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fvalette-ledger fvalette-ledger fvalette-ledger approved these changes

Assignees
No one assigned
Labels
api impact the external API of the component enhancement request for code evolution lang:rust Pull requests that update Rust code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pthierry-ledger @fvalette-ledger