Skip to content

Commit b025662

Browse files
rouming0day robot
authored andcommitted
debugfs: fix automount inode i_nlink references
Directory inodes should start off with i_nlink == 2 (for "." entry). Of course the same rule should be applied to automount dentries for child and parent inodes as well. Also now automount dentry does fsnotify_mkdir. Without this patch kernel complains when sees i_nlink == 0: [ 86.288070] WARNING: CPU: 1 PID: 3616 at fs/inode.c:273 drop_nlink+0x3e/0x50() [ 86.288461] Modules linked in: debugfs_example2(O-) [ 86.288745] CPU: 1 PID: 3616 Comm: rmmod Tainted: G O 4.4.0-rc3-next-20151207+ torvalds#135 [ 86.289197] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.8.2-20150617_082717-anatol 04/01/2014 [ 86.289696] ffffffff81be05c9 ffff8800b9e6fda0 ffffffff81352e2c 0000000000000000 [ 86.290110] ffff8800b9e6fdd8 ffffffff81065142 ffff8801399175e8 ffff8800bb78b240 [ 86.290507] ffff8801399175e8 ffff8800b73d7898 ffff8800b73d7840 ffff8800b9e6fde8 [ 86.290933] Call Trace: [ 86.291080] [<ffffffff81352e2c>] dump_stack+0x4e/0x82 [ 86.291340] [<ffffffff81065142>] warn_slowpath_common+0x82/0xc0 [ 86.291640] [<ffffffff8106523a>] warn_slowpath_null+0x1a/0x20 [ 86.291932] [<ffffffff811ae62e>] drop_nlink+0x3e/0x50 [ 86.292208] [<ffffffff811ba35b>] simple_unlink+0x4b/0x60 [ 86.292481] [<ffffffff811ba3a7>] simple_rmdir+0x37/0x50 [ 86.292748] [<ffffffff812d9808>] __debugfs_remove.part.16+0xa8/0xd0 [ 86.293082] [<ffffffff812d9a0b>] debugfs_remove_recursive+0xdb/0x1c0 [ 86.293406] [<ffffffffa00004dd>] cleanup_module+0x2d/0x3b [debugfs_example2] [ 86.293762] [<ffffffff810d959b>] SyS_delete_module+0x16b/0x220 [ 86.294077] [<ffffffff818ef857>] entry_SYSCALL_64_fastpath+0x12/0x6a [ 86.294405] ---[ end trace c9fc53353fe14a36 ]--- [ 86.294639] ------------[ cut here ]------------ [ 86.294871] WARNING: CPU: 1 PID: 3616 at fs/inode.c:273 drop_nlink+0x3e/0x50() [ 86.295249] Modules linked in: debugfs_example2(O-) [ 86.295510] CPU: 1 PID: 3616 Comm: rmmod Tainted: G W O 4.4.0-rc3-next-20151207+ torvalds#135 [ 86.295943] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.8.2-20150617_082717-anatol 04/01/2014 [ 86.296450] ffffffff81be05c9 ffff8800b9e6fda0 ffffffff81352e2c 0000000000000000 [ 86.296846] ffff8800b9e6fdd8 ffffffff81065142 ffff880139917810 ffff8800b73d7840 [ 86.297253] ffff880139917810 ffff8800b73d7898 ffff8800b73d7840 ffff8800b9e6fde8 [ 86.297650] Call Trace: [ 86.297777] [<ffffffff81352e2c>] dump_stack+0x4e/0x82 [ 86.298043] [<ffffffff81065142>] warn_slowpath_common+0x82/0xc0 [ 86.298344] [<ffffffff8106523a>] warn_slowpath_null+0x1a/0x20 [ 86.298636] [<ffffffff811ae62e>] drop_nlink+0x3e/0x50 [ 86.298893] [<ffffffff811ba35b>] simple_unlink+0x4b/0x60 [ 86.299174] [<ffffffff811ba3a7>] simple_rmdir+0x37/0x50 [ 86.299442] [<ffffffff812d9808>] __debugfs_remove.part.16+0xa8/0xd0 [ 86.299760] [<ffffffff812d9aaf>] debugfs_remove_recursive+0x17f/0x1c0 [ 86.300095] [<ffffffffa00004dd>] cleanup_module+0x2d/0x3b [debugfs_example2] [ 86.300453] [<ffffffff810d959b>] SyS_delete_module+0x16b/0x220 [ 86.300750] [<ffffffff818ef857>] entry_SYSCALL_64_fastpath+0x12/0x6a [ 86.301198] ---[ end trace c9fc53353fe14a37 ]--- Signed-off-by: Roman Pen <[email protected]> Cc: Greg Kroah-Hartman <[email protected]> Cc: [email protected]
1 parent 527e931 commit b025662

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

fs/debugfs/inode.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -461,7 +461,11 @@ struct dentry *debugfs_create_automount(const char *name,
461461
inode->i_flags |= S_AUTOMOUNT;
462462
inode->i_private = data;
463463
dentry->d_fsdata = (void *)f;
464+
/* directory inodes start off with i_nlink == 2 (for "." entry) */
465+
inc_nlink(inode);
464466
d_instantiate(dentry, inode);
467+
inc_nlink(d_inode(dentry->d_parent));
468+
fsnotify_mkdir(d_inode(dentry->d_parent), dentry);
465469
return end_creating(dentry);
466470
}
467471
EXPORT_SYMBOL(debugfs_create_automount);

0 commit comments

Comments
 (0)