mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/xiang/erofs.git
synced 2026-11-01 14:53:11 +01:00
eventfs: Have the inodes all for files and directories all be the same
The dentries and inodes are created in the readdir for the sole purpose of getting a consistent inode number. Linus stated that is unnecessary, and that all inodes can have the same inode number. For a virtual file system they are pretty meaningless. Instead use a single unique inode number for all files and one for all directories. Link: https://lore.kernel.org/all/[email protected]/ Link: https://lore.kernel.org/linux-trace-kernel/[email protected] Cc: Masami Hiramatsu <[email protected]> Cc: Mark Rutland <[email protected]> Cc: Mathieu Desnoyers <[email protected]> Cc: Christian Brauner <[email protected]> Cc: Al Viro <[email protected]> Cc: Ajay Kaher <[email protected]> Suggested-by: Linus Torvalds <[email protected]> Signed-off-by: Steven Rostedt (Google) <[email protected]>
This commit is contained in:
1 parent
1de94b52d5
commit
53c41052ba
1 file changed
+10
@@ -32,6 +32,10 @@
|
||||
*/
|
||||
static DEFINE_MUTEX(eventfs_mutex);
|
||||
|
||||
/* Choose something "unique" ;-) */
|
||||
#define EVENTFS_FILE_INODE_INO 0x12c4e37
|
||||
#define EVENTFS_DIR_INODE_INO 0x134b2f5
|
||||
|
||||
/*
|
||||
* The eventfs_inode (ei) itself is protected by SRCU. It is released from
|
||||
* its parent's list and will have is_freed set (under eventfs_mutex).
|
||||
@@ -352,6 +356,9 @@ static struct dentry *create_file(const char *name, umode_t mode,
|
||||
inode->i_fop = fop;
|
||||
inode->i_private = data;
|
||||
|
||||
/* All files will have the same inode number */
|
||||
inode->i_ino = EVENTFS_FILE_INODE_INO;
|
||||
|
||||
ti = get_tracefs(inode);
|
||||
ti->flags |= TRACEFS_EVENT_INODE;
|
||||
d_instantiate(dentry, inode);
|
||||
@@ -388,6 +395,9 @@ static struct dentry *create_dir(struct eventfs_inode *ei, struct dentry *parent
|
||||
inode->i_op = &eventfs_root_dir_inode_operations;
|
||||
inode->i_fop = &eventfs_file_operations;
|
||||
|
||||
/* All directories will have the same inode number */
|
||||
inode->i_ino = EVENTFS_DIR_INODE_INO;
|
||||
|
||||
ti = get_tracefs(inode);
|
||||
ti->flags |= TRACEFS_EVENT_INODE;
|
||||
|
||||
|
||||
Reference in new issue
Block a user