Re: [PATCH v2 1/3] locking/csd-lock: Pack csd_lock_wait_toolong() state into a struct

[Dmitry Ilvokhin]

On Mon, Aug 10, 2026 at 04:29:24AM -0700, Breno Leitao wrote:
> csd_lock_wait_toolong() has some fields and they are being expanded now,
> separate them into a structure, that can be easily digestible.
> 
> This simplify the function aslo, given the fields were passed by
> reference, and the ts0/ts1 names say nothing about what the two
> timestamps hold.
> 
> Pack them into struct csd_wait_state and name the timestamps for what
> they store, ts_start and ts_report.  The local ts2 becomes ts_now.
> 
> Reporting a further timestamp, such as next patch, then costs a struct
> member rather than another argument.
> 
> No functional change.
> 
> Suggested-by: Dmitry Ilvokhin <d@xxxxxxxxxxxx>
> Signed-off-by: Breno Leitao <leitao@xxxxxxxxxx>

Reviewed-by: Dmitry Ilvokhin <d@xxxxxxxxxxxx>