Skip to content

Commit 6fdac58

Browse files
committed
tracing: Remove unused trace_event_file dir field
Now that eventfs structure is used to create the events directory via the eventfs dynamically allocate code, the "dir" field of the trace_event_file structure is no longer used. Remove it. Link: https://lkml.kernel.org/r/20230908022001.580400115@goodmis.org Cc: Masami Hiramatsu <mhiramat@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: Andrew Morton <akpm@linux-foundation.org> Cc: Ajay Kaher <akaher@vmware.com> Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
1 parent 1ef26d8 commit 6fdac58

File tree

2 files changed

+0
-14
lines changed

2 files changed

+0
-14
lines changed

include/linux/trace_events.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -650,7 +650,6 @@ struct trace_event_file {
650650
struct trace_event_call *event_call;
651651
struct event_filter __rcu *filter;
652652
struct eventfs_file *ef;
653-
struct dentry *dir;
654653
struct trace_array *tr;
655654
struct trace_subsystem_dir *system;
656655
struct list_head triggers;

kernel/trace/trace_events.c

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -992,19 +992,6 @@ static void remove_subsystem(struct trace_subsystem_dir *dir)
992992

993993
static void remove_event_file_dir(struct trace_event_file *file)
994994
{
995-
struct dentry *dir = file->dir;
996-
struct dentry *child;
997-
998-
if (dir) {
999-
spin_lock(&dir->d_lock); /* probably unneeded */
1000-
list_for_each_entry(child, &dir->d_subdirs, d_child) {
1001-
if (d_really_is_positive(child)) /* probably unneeded */
1002-
d_inode(child)->i_private = NULL;
1003-
}
1004-
spin_unlock(&dir->d_lock);
1005-
1006-
tracefs_remove(dir);
1007-
}
1008995
eventfs_remove(file->ef);
1009996
list_del(&file->list);
1010997
remove_subsystem(file->system);

0 commit comments

Comments
 (0)