Commit 78c70513 authored by Steven Rostedt's avatar Steven Rostedt Committed by Steven Rostedt (Google)
Browse files

tracing: Remove unneeded event_mutex lock in event_trigger_regex_release() 

In event_trigger_regex_release(), the only code is:

	mutex_lock(&event_mutex);
	if (file->f_mode & FMODE_READ)
		seq_release(inode, file);
	mutex_unlock(&event_mutex);

	return 0;

There's nothing special about the file->f_mode or the seq_release() that
requires any locking. Remove the unnecessary locks.

Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Tom Zanussi <zanussi@kernel.org>
Link: https://patch.msgid.link/20251125214031.975879283@kernel.org


Acked-by: default avatarMasami Hiramatsu (Google) <mhiramat@kernel.org>
Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
parent b052d70f

kernel/trace/trace_events_trigger.c

+0 −4
Original line number Diff line number Diff line
@@ -314,13 +314,9 @@ static ssize_t event_trigger_regex_write(struct file *file, 


static int event_trigger_regex_release(struct inode *inode, struct file *file)

{

	mutex_lock(&event_mutex);



	if (file->f_mode & FMODE_READ)

		seq_release(inode, file);



	mutex_unlock(&event_mutex);



	return 0;

}