Re: [PATCH v3 7/7] rust_binder: Implement BINDER_DEBUG_DEAD_TRANSACTION
From: Carlos Llamas
Date: Mon Jul 13 2026 - 17:04:17 EST
On Mon, Jul 13, 2026 at 12:35:29PM +0000, Jahnavi MN via B4 Relay wrote:
> From: Jahnavi MN <jahnavimn@xxxxxxxxxx>
>
> This adds dynamic debug logs for:
> - Releasing active transactions during thread stack unwinding.
> - Discarded transaction error codes when a thread exits.
> - Undelivered transaction acknowledgments (TRANSACTION_COMPLETE)
> upon thread exit.
> - Undelivered process death and freeze notifications when processes
> exit or die.
> - Undelivered transactions canceled due to target process death.
>
> We now store the process PID in `ThreadError`, `DeliverCode`, and
> `FreezeMessage` to ensure the correct PID is logged on cancellation.
> This is necessary because `cancel()` runs from background `kworkers`,
> which would otherwise print the wrong PID.
>
> Reviewed-by: Alice Ryhl <aliceryhl@xxxxxxxxxx>
> Signed-off-by: Jahnavi MN <jahnavimn@xxxxxxxxxx>
> ---
Thanks Jahnavi,
Reviewed-by: Carlos Llamas <cmllamas@xxxxxxxxxx>