On 4/23/2025 9:09 PM, Johannes Berg wrote: > On Fri, 2025-03-28 at 17:55 +0530, Roopni Devanathan wrote: >> >> +TRACE_EVENT(drv_set_rts_threshold, >> + TP_PROTO(struct ieee80211_local *local, s8 radio_id, u32 value), >> + >> + TP_ARGS(local, radio_id, value), >> + >> + TP_STRUCT__entry( >> + LOCAL_ENTRY >> + __field(s8, radio_id) >> + __field(u32, value) >> + ), >> + >> + TP_fast_assign( >> + LOCAL_ASSIGN; >> + __entry->radio_id = radio_id; >> + __entry->value = value; >> + ), >> + >> + TP_printk( >> + LOCAL_PR_FMT " value:%d", >> + LOCAL_PR_ARG, __entry->value >> + ) >> > > I think it'd make sense (and be trivial) to have the radio ID here. > > Same in the first patch, but I missed it because the print was hidden > outside the context :) > Got it. Will add this in next version. > johannes
