f2fs: no need to check return value of debugfs_create functions

When calling debugfs functions, there is no need to ever check the
return value.  The function can work or not, but the code logic should
never do something different based on this.

Cc: Jaegeuk Kim <jaegeuk@kernel.org>
Cc: Chao Yu <yuchao0@huawei.com>
Cc: linux-f2fs-devel@lists.sourceforge.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

diff --git a/fs/f2fs/debug.c b/fs/f2fs/debug.c
index 503fde8349e641eca6047a167fa85c96061be72c..99e9a5c37b7110eb2d20484c87ad78c0eb85f225 100644 (file)
--- a/fs/f2fs/debug.c
+++ b/fs/f2fs/debug.c
@@ -511,30 +511,16 @@ void f2fs_destroy_stats(struct f2fs_sb_info *sbi)
        kvfree(si);
 }
 
-int __init f2fs_create_root_stats(void)
+void __init f2fs_create_root_stats(void)
 {
-       struct dentry *file;
-
        f2fs_debugfs_root = debugfs_create_dir("f2fs", NULL);
-       if (!f2fs_debugfs_root)
-               return -ENOMEM;
 
-       file = debugfs_create_file("status", S_IRUGO, f2fs_debugfs_root,
-                       NULL, &stat_fops);
-       if (!file) {
-               debugfs_remove(f2fs_debugfs_root);
-               f2fs_debugfs_root = NULL;
-               return -ENOMEM;
-       }
-
-       return 0;
+       debugfs_create_file("status", S_IRUGO, f2fs_debugfs_root, NULL,
+                           &stat_fops);
 }
 
 void f2fs_destroy_root_stats(void)
 {
-       if (!f2fs_debugfs_root)
-               return;
-
        debugfs_remove_recursive(f2fs_debugfs_root);
        f2fs_debugfs_root = NULL;
 }

diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index 9286ec381453038634dd44b75be927f7d8478e61..7df41cd1eb35ebe3baec84aec93a17b768eb1adb 100644 (file)
--- a/fs/f2fs/f2fs.h
+++ b/fs/f2fs/f2fs.h
@@ -3329,7 +3329,7 @@ static inline struct f2fs_stat_info *F2FS_STAT(struct f2fs_sb_info *sbi)
 
 int f2fs_build_stats(struct f2fs_sb_info *sbi);
 void f2fs_destroy_stats(struct f2fs_sb_info *sbi);
-int __init f2fs_create_root_stats(void);
+void __init f2fs_create_root_stats(void);
 void f2fs_destroy_root_stats(void);
 #else
 #define stat_inc_cp_count(si)                          do { } while (0)
@@ -3367,7 +3367,7 @@ void f2fs_destroy_root_stats(void);
 
 static inline int f2fs_build_stats(struct f2fs_sb_info *sbi) { return 0; }
 static inline void f2fs_destroy_stats(struct f2fs_sb_info *sbi) { }
-static inline int __init f2fs_create_root_stats(void) { return 0; }
+static inline void __init f2fs_create_root_stats(void) { }
 static inline void f2fs_destroy_root_stats(void) { }
 #endif
 

diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index bc32a1035f656bc16bff98a264fce016b830c696..ea514acede36f09dc42ec82934832d787ae7510f 100644 (file)
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -3550,9 +3550,7 @@ static int __init init_f2fs_fs(void)
        err = register_filesystem(&f2fs_fs_type);
        if (err)
                goto free_shrinker;
-       err = f2fs_create_root_stats();
-       if (err)
-               goto free_filesystem;
+       f2fs_create_root_stats();
        err = f2fs_init_post_read_processing();
        if (err)
                goto free_root_stats;
@@ -3560,7 +3558,6 @@ static int __init init_f2fs_fs(void)
 
 free_root_stats:
        f2fs_destroy_root_stats();
-free_filesystem:
        unregister_filesystem(&f2fs_fs_type);
 free_shrinker:
        unregister_shrinker(&f2fs_shrinker_info);