Re: [PATCH v5 7/9] rust: file: add `Kuid` wrapper

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Trevor Gross <tmgross@xxxxxxxxx>
Subject: Re: [PATCH v5 7/9] rust: file: add `Kuid` wrapper
From: Alice Ryhl <aliceryhl@xxxxxxxxxx>
Date: Mon, 12 Feb 2024 11:04:47 +0100
Cc: Miguel Ojeda <ojeda@xxxxxxxxxx>, Alex Gaynor <alex.gaynor@xxxxxxxxx>, Wedson Almeida Filho <wedsonaf@xxxxxxxxx>, Boqun Feng <boqun.feng@xxxxxxxxx>, Gary Guo <gary@xxxxxxxxxxx>, Björn Roy Baron <bjorn3_gh@xxxxxxxxxxxxxx>, Benno Lossin <benno.lossin@xxxxxxxxx>, Andreas Hindborg <a.hindborg@xxxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>, Christian Brauner <brauner@xxxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Arve Hjønnevåg <arve@xxxxxxxxxxx>, Todd Kjos <tkjos@xxxxxxxxxxx>, Martijn Coenen <maco@xxxxxxxxxxx>, Joel Fernandes <joel@xxxxxxxxxxxxxxxxx>, Carlos Llamas <cmllamas@xxxxxxxxxx>, Suren Baghdasaryan <surenb@xxxxxxxxxx>, Dan Williams <dan.j.williams@xxxxxxxxx>, Kees Cook <keescook@xxxxxxxxxxxx>, Matthew Wilcox <willy@xxxxxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Daniel Xu <dxu@xxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, rust-for-linux@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, Martin Rodriguez Reboredo <yakoyoku@xxxxxxxxx>
In-reply-to: <CALNs47tTTbWL3T9rk_ismPT0Bwi8Kcm5aT9k8jfPsh=1wKvrPA@mail.gmail.com>

On Sat, Feb 10, 2024 at 8:43 AM Trevor Gross <tmgross@xxxxxxxxx> wrote:
>
> On Fri, Feb 9, 2024 at 5:22 AM Alice Ryhl <aliceryhl@xxxxxxxxxx> wrote:
> >
> > Of course, once a wrapper for rcu_read_lock is available, it is
> > preferable to use that over either of the two above approaches.
>
> Is this worth a FIXME?

Shrug. I think a patch to introduce rcu_read_lock would go through the
helpers as a matter of course either way. But it also doesn't hurt.

Alice

Follow-Ups:
- Re: [PATCH v5 7/9] rust: file: add `Kuid` wrapper
  - From: Boqun Feng

References:
- [PATCH v5 0/9] File abstractions needed by Rust Binder
  - From: Alice Ryhl
- [PATCH v5 7/9] rust: file: add `Kuid` wrapper
  - From: Alice Ryhl
- Re: [PATCH v5 7/9] rust: file: add `Kuid` wrapper
  - From: Trevor Gross

Prev by Date: Re: [PATCH] [RFC] fs: prefer kfree_rcu() in fasync_remove_entry()
Next by Date: Re: [RFC PATCH v3 00/26] ext4: use iomap for regular file's buffered IO path and enable large foilo
Previous by thread: Re: [PATCH v5 7/9] rust: file: add `Kuid` wrapper
Next by thread: Re: [PATCH v5 7/9] rust: file: add `Kuid` wrapper
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]