Skip to content
This repository was archived by the owner on Mar 7, 2021. It is now read-only.

Add fsync support #276

Closed
wants to merge 1 commit into from
Closed

Add fsync support #276

wants to merge 1 commit into from

Conversation

adamrk
Copy link

@adamrk adamrk commented Oct 17, 2020

Hey, I took a stab at adding fsync to the file operations. I'm not really sure what the return value is supposed to be used for, so maybe it should just be ()?

@alex
Copy link
Member

alex commented Oct 17, 2020

Hey Adam, thanks for contributing!

We're slowly migrating ourselves to a plan for contributing this code to the upstream kernel. Would you be interested in migrating your patch to be against https://github.com/Rust-for-Linux/linux/ ?

@adamrk
Copy link
Author

adamrk commented Oct 17, 2020

Yeah, no problem - by the way, is there a chat or something for the Rust-for-Linux group? I've got some questions that don't really make sense to submit as issues.

For example, I'm not sure how to use the library as it is in https://github.com/Rust-for-Linux/linux/ for making a module that's not built-in.

@alex
Copy link
Member

alex commented Oct 17, 2020 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants