[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANiq72kA8ZMf4ivQa4JTt0ZDmJ5bxWdpjgNb9bfW9n27HdTQ=A@mail.gmail.com>
Date: Wed, 22 Oct 2025 01:16:25 +0200
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Danilo Krummrich <dakr@...nel.org>
Cc: gregkh@...uxfoundation.org, rafael@...nel.org, ojeda@...nel.org,
alex.gaynor@...il.com, boqun.feng@...il.com, gary@...yguo.net,
bjorn3_gh@...tonmail.com, lossin@...nel.org, a.hindborg@...nel.org,
aliceryhl@...gle.com, tmgross@...ch.edu, mmaurer@...gle.com,
rust-for-linux@...r.kernel.org, linux-fsdevel@...r.kernel.org,
linux-kernel@...r.kernel.org, Alexander Viro <viro@...iv.linux.org.uk>,
Christian Brauner <brauner@...nel.org>, Jan Kara <jack@...e.cz>
Subject: Re: [PATCH v2 1/8] rust: fs: add file::Offset type alias
On Tue, Oct 21, 2025 at 7:34 PM Danilo Krummrich <dakr@...nel.org> wrote:
>
> However, I understand where you're coming from. Even though there's not a huge
> gain here, it would be good to set an example -- especially if it's something as
> cental as a file offset type.
>
> If this is what you have in mind, let me go ahead and do it right away (at least
> for the things needed by this patch series), because that's a very good reason I
> think.
Yeah, exactly, it should be fairly straightforward for at least the ones here.
Up to you, i.e. we can already start (the code you showed me offline
looks good) or wait for someone to send the change (or since you
started, they can still consider more operations and expand the type
etc.).
I will send a small coding guidelines note on type aliases -- I can
reference this example when added, since it is a good example I think.
:)
Thanks!
Cheers,
Miguel
Powered by blists - more mailing lists