[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20231129-mitsingen-umweltschutz-c6f8d9569234@brauner>
Date: Wed, 29 Nov 2023 17:31:44 +0100
From: Christian Brauner <brauner@...nel.org>
To: Alice Ryhl <aliceryhl@...gle.com>
Cc: Miguel Ojeda <ojeda@...nel.org>,
Alex Gaynor <alex.gaynor@...il.com>,
Wedson Almeida Filho <wedsonaf@...il.com>,
Boqun Feng <boqun.feng@...il.com>, Gary Guo <gary@...yguo.net>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
Benno Lossin <benno.lossin@...ton.me>,
Andreas Hindborg <a.hindborg@...sung.com>,
Peter Zijlstra <peterz@...radead.org>,
Alexander Viro <viro@...iv.linux.org.uk>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Arve Hjønnevåg <arve@...roid.com>,
Todd Kjos <tkjos@...roid.com>,
Martijn Coenen <maco@...roid.com>,
Joel Fernandes <joel@...lfernandes.org>,
Carlos Llamas <cmllamas@...gle.com>,
Suren Baghdasaryan <surenb@...gle.com>,
Dan Williams <dan.j.williams@...el.com>,
Kees Cook <keescook@...omium.org>,
Matthew Wilcox <willy@...radead.org>,
Thomas Gleixner <tglx@...utronix.de>,
Daniel Xu <dxu@...uu.xyz>, linux-kernel@...r.kernel.org,
rust-for-linux@...r.kernel.org, linux-fsdevel@...r.kernel.org
Subject: Re: [PATCH 0/7] File abstractions needed by Rust Binder
On Wed, Nov 29, 2023 at 12:51:06PM +0000, Alice Ryhl wrote:
> This patchset contains the file abstractions needed by the Rust
> implementation of the Binder driver.
>
> Please see the Rust Binder RFC for usage examples:
> https://lore.kernel.org/rust-for-linux/20231101-rust-binder-v1-0-08ba9197f637@google.com/
>
> Users of "rust: file: add Rust abstraction for `struct file`":
> [PATCH RFC 02/20] rust_binder: add binderfs support to Rust binder
> [PATCH RFC 03/20] rust_binder: add threading support
>
> Users of "rust: cred: add Rust abstraction for `struct cred`":
> [PATCH RFC 05/20] rust_binder: add nodes and context managers
> [PATCH RFC 06/20] rust_binder: add oneway transactions
> [PATCH RFC 11/20] rust_binder: send nodes in transaction
> [PATCH RFC 13/20] rust_binder: add BINDER_TYPE_FD support
>
> Users of "rust: security: add abstraction for security_secid_to_secctx":
> [PATCH RFC 06/20] rust_binder: add oneway transactions
>
> Users of "rust: file: add `FileDescriptorReservation`":
> [PATCH RFC 13/20] rust_binder: add BINDER_TYPE_FD support
> [PATCH RFC 14/20] rust_binder: add BINDER_TYPE_FDA support
>
> Users of "rust: file: add kuid getters":
> [PATCH RFC 05/20] rust_binder: add nodes and context managers
> [PATCH RFC 06/20] rust_binder: add oneway transactions
>
> Users of "rust: file: add `DeferredFdCloser`":
> [PATCH RFC 14/20] rust_binder: add BINDER_TYPE_FDA support
>
> Users of "rust: file: add abstraction for `poll_table`":
> [PATCH RFC 07/20] rust_binder: add epoll support
>
> This patchset has some uses of read_volatile in place of READ_ONCE.
> Please see the following rfc for context on this:
> https://lore.kernel.org/all/20231025195339.1431894-1-boqun.feng@gmail.com/
>
> This was previously sent as an rfc:
> https://lore.kernel.org/all/20230720152820.3566078-1-aliceryhl@google.com/
>
> Signed-off-by: Alice Ryhl <aliceryhl@...gle.com>
> ---
> Alice Ryhl (4):
> rust: security: add abstraction for security_secid_to_secctx
> rust: file: add `Kuid` wrapper
> rust: file: add `DeferredFdCloser`
> rust: file: add abstraction for `poll_table`
>
> Wedson Almeida Filho (3):
> rust: file: add Rust abstraction for `struct file`
> rust: cred: add Rust abstraction for `struct cred`
> rust: file: add `FileDescriptorReservation`
>
> rust/bindings/bindings_helper.h | 9 ++
> rust/bindings/lib.rs | 1 +
> rust/helpers.c | 94 +++++++++++
> rust/kernel/cred.rs | 73 +++++++++
> rust/kernel/file.rs | 345 ++++++++++++++++++++++++++++++++++++++++
> rust/kernel/file/poll_table.rs | 97 +++++++++++
That's pretty far away from the subsystem these wrappers belong to. I
would prefer if wrappers such as this would live directly in fs/rust/
and so live within the subsystem they belong to. I think I mentioned
that before. Maybe I missed some sort of agreement here?
> rust/kernel/lib.rs | 3 +
> rust/kernel/security.rs | 78 +++++++++
> rust/kernel/sync/condvar.rs | 2 +-
> rust/kernel/task.rs | 71 ++++++++-
> 10 files changed, 771 insertions(+), 2 deletions(-)
> ---
> base-commit: 98b1cc82c4affc16f5598d4fa14b1858671b2263
> change-id: 20231123-alice-file-525b98e8a724
>
> Best regards,
> --
> Alice Ryhl <aliceryhl@...gle.com>
>
Powered by blists - more mailing lists