Re: [PATCH] debugfs: remove inc_nlink in debugfs_create_automount

2018-12-23 Thread yangerkun
Greg KH wrote on 2018/12/22 19:25: On Sat, Dec 22, 2018 at 04:45:36PM +0800, yangerkun wrote: Remove inc_nlink in debugfs_create_automount, or this inode will never be free. Signed-off-by: yangerkun --- fs/debugfs/inode.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --gi

Re: [PATCH] debugfs: remove inc_nlink in debugfs_create_automount

2018-12-22 Thread Al Viro
On Sat, Dec 22, 2018 at 04:45:36PM +0800, yangerkun wrote: > Remove inc_nlink in debugfs_create_automount, or this inode will never > be free. Explain, please. What exactly would care about i_nlink in debugfs? It does *NOT* free any kind of backing store on inode eviction, for a good and simple r

[PATCH] debugfs: remove inc_nlink in debugfs_create_automount

2018-12-22 Thread yangerkun
Remove inc_nlink in debugfs_create_automount, or this inode will never be free. Signed-off-by: yangerkun --- fs/debugfs/inode.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/debugfs/inode.c b/fs/debugfs/inode.c index 13b01351dd1c..9294238e364f 100644 --- a/fs/debugfs/i

Re: [PATCH] debugfs: remove inc_nlink in debugfs_create_automount

2018-12-22 Thread Greg KH
On Sat, Dec 22, 2018 at 04:45:36PM +0800, yangerkun wrote: > Remove inc_nlink in debugfs_create_automount, or this inode will never > be free. > > Signed-off-by: yangerkun > --- > fs/debugfs/inode.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/debugfs/inode.c b/