eventfs: Use eventfs_remove_events_dir()
authorSteven Rostedt (Google) <rostedt@goodmis.org>
Thu, 5 Oct 2023 13:13:48 +0000 (09:13 -0400)
committerSteven Rostedt (Google) <rostedt@goodmis.org>
Thu, 5 Oct 2023 14:49:32 +0000 (10:49 -0400)
commit2819f23ac12ce93ff79ca7a54597df9a4a1f6331
tree1b213d86cd17a02f30b063b4a4403b81484c97fc
parentf5d9e8e08f81c9e7c723de7abcce106808f0770c
eventfs: Use eventfs_remove_events_dir()

The update to removing the eventfs_file changed the way the events top
level directory was handled. Instead of returning a dentry, it now returns
the eventfs_inode. In this changed, the removing of the events top level
directory is not much different than removing any of the other
directories. Because of this, the removal just called eventfs_remove_dir()
instead of eventfs_remove_events_dir().

Although eventfs_remove_dir() does the clean up, it misses out on the
dget() of the ei->dentry done in eventfs_create_events_dir(). It makes
more sense to match eventfs_create_events_dir() with a specific function
eventfs_remove_events_dir() and this specific function can then perform
the dput() to the dentry that had the dget() when it was created.

Fixes: 5790b1fb3d67 ("eventfs: Remove eventfs_file and just use eventfs_inode")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202310051743.y9EobbUr-lkp@intel.com/
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
fs/tracefs/event_inode.c
include/linux/tracefs.h
kernel/trace/trace_events.c