On Mon, Feb 17, 2025 at 11:38:44AM -0500, Steven Rostedt wrote:

quoted

+void __do_trace_set_current_state(int state_value)
+{
+	__do_trace_sched_set_state_tp(current, current->__state, state_value);

And this should not be using the internal macros of a trace point. It should be:

	trace_sched_set_state_tp(current, state_value);

(I removed the current->__state as mentioned above).

But the static branch is already in the caller, no point duplicating
that.