Thread (8 messages) 8 messages, 3 authors, 28d ago

Re: [PATCH v4 0/2] Delete task_euid()

From: Alice Ryhl <aliceryhl@google.com>
Date: 2026-06-02 06:15:44
Also in: linux-doc, lkml, rust-for-linux

On Mon, Jun 01, 2026 at 07:13:37PM -0400, Paul Moore wrote:
On Fri, May 29, 2026 at 5:33 AM Alice Ryhl [off-list ref] wrote:
quoted
The task_euid() method is a very weird method, and Binder was the only
user. As of commit 65b672152289 ("binder: use current_euid() for
transaction sender identity") Binder doesn't use task_euid() anymore,
so we can delete this method.
Given the problems from last time, it seems like it might be prudent
to let the commit have some time to "breathe" in a proper release, I'd
suggest merging this not for the upcoming v7.2 merge window but
instead waiting for v7.3.
Sure, that makes sense. I'll resend after the merge window.
quoted
My suggestion would be to merge this through the LSM tree.
That's fine with me.  I'd also suggest updating the commit description
in patch 1/2 to indicate that binder is no longer using task_euid();
it currently reads like it is still being used.
I guess this occurred because when patch 1 was written, it really *was*
still being used. Perhaps we could pick up only patch 1 now since even
if we run into problems and Binder has to go back to using task_euid(),
clarifying the docs is still useful.

Alice
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help