]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
fgraph: Remove coarse PID filtering from graph_entry()
authorShengming Hu <hu.shengming@zte.com.cn>
Wed, 26 Nov 2025 09:35:52 +0000 (17:35 +0800)
committerSteven Rostedt (Google) <rostedt@goodmis.org>
Wed, 26 Nov 2025 20:41:35 +0000 (15:41 -0500)
With PID filtering working via ftrace_pids_enabled() and fgraph_pid_func,
the coarse-grained ftrace_trace_task() check in graph_entry() is obsolete.

It was only a fallback for uninitialized op->private (now fixed), and its
removal ensures consistent PID filtering with standard function tracing.

Also remove unused ftrace_trace_task() definition from trace.h.

Cc: <wang.yaxin@zte.com.cn>
Cc: <mhiramat@kernel.org>
Cc: <mark.rutland@arm.com>
Cc: <mathieu.desnoyers@efficios.com>
Cc: <zhang.run@zte.com.cn>
Cc: <yang.yang29@zte.com.cn>
Link: https://patch.msgid.link/20251126173552333XoJZN20143fWbsdTEtWoU@zte.com.cn
Signed-off-by: Shengming Hu <hu.shengming@zte.com.cn>
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
kernel/trace/trace.h
kernel/trace/trace_functions_graph.c

index 85eabb454bee9fc7565f20dd495ceebe8a0887f9..b2b6a9775b44bc7671ef2c80517a011d71d127b9 100644 (file)
@@ -1154,11 +1154,6 @@ struct ftrace_func_command {
                                        char *params, int enable);
 };
 extern bool ftrace_filter_param __initdata;
-static inline int ftrace_trace_task(struct trace_array *tr)
-{
-       return this_cpu_read(tr->array_buffer.data->ftrace_ignore_pid) !=
-               FTRACE_PID_IGNORE;
-}
 extern int ftrace_is_dead(void);
 int ftrace_create_function_files(struct trace_array *tr,
                                 struct dentry *parent);
@@ -1176,10 +1171,6 @@ void ftrace_clear_pids(struct trace_array *tr);
 int init_function_trace(void);
 void ftrace_pid_follow_fork(struct trace_array *tr, bool enable);
 #else
-static inline int ftrace_trace_task(struct trace_array *tr)
-{
-       return 1;
-}
 static inline int ftrace_is_dead(void) { return 0; }
 static inline int
 ftrace_create_function_files(struct trace_array *tr,
index a7f4b9a47a71a0bc513ad16fd42b4686ab414985..75ef660e2ed51db45bc21bc39853f42611f61bc7 100644 (file)
@@ -232,9 +232,6 @@ static int graph_entry(struct ftrace_graph_ent *trace,
                return 1;
        }
 
-       if (!ftrace_trace_task(tr))
-               return 0;
-
        if (ftrace_graph_ignore_func(gops, trace))
                return 0;