On Mon, 3 Nov 2025 at 20:27, Christian Brauner <[email protected]> wrote:
>
> /* Perform file operations on behalf of whoever enabled accounting */
> - cred = override_creds(file->f_cred);
> -
> + with_creds(file->f_cred);
I'd almost prefer if we *only* did "scoped_with_creds()" and didn't
have this version at all.
Most of the cases want that anyway, and the couple of plain
"with_creds()" cases look like they would only be cleaned up by making
the cred scoping more explicit.
What do you think?
Anyway, I approve of the whole series, obviously, I just suspect we
could narrow down the new interface a bit more.
Linus