From: Liang Chen <liangchen.li...@gmail.com>

mutex_destroy does nothing most of time, but it's better to call
it to make the code future proof and it also has some meaning
for like mutex debug.

Signed-off-by: Liang Chen <liangchen.li...@gmail.com>
Reviewed-by: Eric Wheeler <bca...@linux.ewheeler.net>
Cc: sta...@vger.kernel.org
---
 drivers/md/bcache/super.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache/super.c
index 48b8c20..1f84791 100644
--- a/drivers/md/bcache/super.c
+++ b/drivers/md/bcache/super.c
@@ -2089,6 +2089,7 @@ static void bcache_exit(void)
        if (bcache_major)
                unregister_blkdev(bcache_major, "bcache");
        unregister_reboot_notifier(&reboot);
+       mutex_destroy(&bch_register_lock);
 }
 
 static int __init bcache_init(void)
@@ -2106,6 +2107,7 @@ static int __init bcache_init(void)
 
        bcache_major = register_blkdev(0, "bcache");
        if (bcache_major < 0) {
+               mutex_destroy(&bch_register_lock);
                unregister_reboot_notifier(&reboot);
                return bcache_major;
        }
-- 
1.8.3.1

Reply via email to