Re: [PATCH 1/3] rust: add userspace pointers

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

To: Alice Ryhl <aliceryhl@xxxxxxxxxx>
Subject: Re: [PATCH 1/3] rust: add userspace pointers
From: Trevor Gross <tmgross@xxxxxxxxx>
Date: Sat, 10 Feb 2024 01:06:07 -0600
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>, Kees Cook <keescook@xxxxxxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, 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>, Arnd Bergmann <arnd@xxxxxxxx>, linux-mm@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, rust-for-linux@xxxxxxxxxxxxxxx, Christian Brauner <brauner@xxxxxxxxxx>
In-reply-to: <CAH5fLggdwWoq4wKv7VxZ-_VbWMV_Ui03rGOCMPbWn8=ewznmvA@mail.gmail.com>
References: <20240124-alice-mm-v1-0-d1abcec83c44@google.com> <20240124-alice-mm-v1-1-d1abcec83c44@google.com> <CALNs47uDaD05oD8TtZoSqUBc4SaRig80u2_1P0qMCXtE3H9_Vw@mail.gmail.com> <CAH5fLggdwWoq4wKv7VxZ-_VbWMV_Ui03rGOCMPbWn8=ewznmvA@mail.gmail.com>

On Thu, Feb 8, 2024 at 6:53 AM Alice Ryhl <aliceryhl@xxxxxxxxxx> wrote:
> Leaking pointer addresses in logs is frowned upon in the kernel, so I
> don't think we should include that.

Sounds like the very correct decision from what Greg and Kees said. If
we add an impl in the future (to be able to `#[derive(Debug)]` on
anything containing this type) it could just print
`0x****************`.

References:
- [PATCH 0/3] Memory management patches needed by Rust Binder
  - From: Alice Ryhl
- [PATCH 1/3] rust: add userspace pointers
  - From: Alice Ryhl
- Re: [PATCH 1/3] rust: add userspace pointers
  - From: Trevor Gross
- Re: [PATCH 1/3] rust: add userspace pointers
  - From: Alice Ryhl

Prev by Date: Re: [linux-next:master 3841/4021] kernel/workqueue.c:6798:32: error: passing 'const struct bus_type *' to parameter of type 'struct bus_type *' discards qualifiers
Next by Date: Re: [PATCH] selftests/mm: Don't needlessly use sudo to obtain root in run_vmtests.sh
Previous by thread: Re: [PATCH 1/3] rust: add userspace pointers
Next by thread: RE: [PATCH 1/3] rust: add userspace pointers
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]