• Steven Rostedt (Google)'s avatar
    eventfs: Do not treat events directory different than other directories · 22e61e15
    Steven Rostedt (Google) authored
    Treat the events directory the same as other directories when it comes to
    permissions. The events directory was considered different because it's
    dentry is persistent, whereas the other directory dentries are created
    when accessed. But the way tracefs now does its ownership by using the
    root dentry's permissions as the default permissions, the events directory
    can get out of sync when a remount is performed setting the group and user
    permissions.
    
    Remove the special case for the events directory on setting the
    attributes. This allows the updates caused by remount to work properly as
    well as simplifies the code.
    
    Link: https://lore.kernel.org/linux-trace-kernel/20240502200906.002923579@goodmis.org
    
    Cc: stable@vger.kernel.org
    Cc: Masami Hiramatsu <mhiramat@kernel.org>
    Cc: Mark Rutland <mark.rutland@arm.com>
    Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
    Cc: Andrew Morton <akpm@linux-foundation.org>
    Fixes: 8186fff7 ("tracefs/eventfs: Use root and instance inodes as default ownership")
    Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
    22e61e15
event_inode.c 23.9 KB