bcache: do not check if debug dentry is ERR or NULL explicitly on remove

debugfs_remove and debugfs_remove_recursive will check if the dentry
pointer is NULL or ERR, and will do nothing in that case.

Remove the check in cache_set_free and bch_debug_init.

Signed-off-by: Shenghui Wang <shhuiw@foxmail.com>
Signed-off-by: Coly Li <colyli@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/drivers/md/bcache/debug.c b/drivers/md/bcache/debug.c
index 8f448b9c96a14db9954d830bd27a7444c13ce83e..8b123be05254d39bcc4ca33caa6af01bf5056cfb 100644 (file)
--- a/drivers/md/bcache/debug.c
+++ b/drivers/md/bcache/debug.c
@@ -249,8 +249,7 @@ void bch_debug_init_cache_set(struct cache_set *c)
 
 void bch_debug_exit(void)
 {
-       if (!IS_ERR_OR_NULL(bcache_debug))
-               debugfs_remove_recursive(bcache_debug);
+       debugfs_remove_recursive(bcache_debug);
 }
 
 void __init bch_debug_init(void)

diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache/super.c
index 7bbd670a5a84841206405cbb1ff397abb72f412b..5b59d44656c0de8e1045807f460adda1090a1947 100644 (file)
--- a/drivers/md/bcache/super.c
+++ b/drivers/md/bcache/super.c
@@ -1510,8 +1510,7 @@ static void cache_set_free(struct closure *cl)
        struct cache *ca;
        unsigned int i;
 
-       if (!IS_ERR_OR_NULL(c->debug))
-               debugfs_remove(c->debug);
+       debugfs_remove(c->debug);
 
        bch_open_buckets_free(c);
        bch_btree_cache_free(c);