On Thu, 2 Nov 2023 08:43:32 +0900
Masami Hiramatsu (Google) [off-list ref] wrote:

On Tue, 31 Oct 2023 18:33:30 -0400
Steven Rostedt [off-list ref] wrote:

quoted

From: "Steven Rostedt (Google)" <rostedt@goodmis.org>

Now that inodes and dentries are created on the fly, they are also
reclaimed on memory pressure. Since the ownership and file mode are saved
in the inode, if they are freed, any changes to the ownership and mode
will be lost.  

Do we (need to) allow to change the ownership and mode of the eventfs files?
I thought it was fixed on the files in tracefs...

Yes, it's the only way to allow non root users access to the tracing directories.

Otherwise, the code itself looks good to me.

Reviewed-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>

Thanks!

-- Steve