Re: [PATCH 11/21] binder: remove pid param in binder_alloc_new_buf()

From: Alice Ryhl
Date: Tue Nov 07 2023 - 04:08:59 EST


Carlos Llamas <cmllamas@xxxxxxxxxx> writes:
> Binder attributes the buffer allocation to the current->tgid everytime.
> There is no need to pass this as a parameter so drop it.
>
> Also add a few touchups to follow the coding guidelines. No functional
> changes are introduced in this patch.
>
> Signed-off-by: Carlos Llamas <cmllamas@xxxxxxxxxx>

You could argue that this code would be cleaner without this change,
since it now relies on global state. However, I'm okay either way.

Reviewed-by: Alice Ryhl <aliceryhl@xxxxxxxxxx>