diff mbox

block_dev : clean-up the code of thaw_bdev

Message ID 55473A41.5000506@windriver.com
State New, archived
Headers show

Commit Message

Yeon, JeHyeon (Tom) May 4, 2015, 9:22 a.m. UTC
the code in the "if (error) {" statment is duplicated with the last
statment at the end of function.
simply remove the mutex_unlock statement and return statement in there.

Signed-off-by: Tom(JeHyeon) Yeon <tom.yeon@windriver.com>
---
  fs/block_dev.c |    9 ++++-----
  1 file changed, 4 insertions(+), 5 deletions(-)
diff mbox

Patch

diff --git a/fs/block_dev.c b/fs/block_dev.c
index 975266b..01676ca 100644
--- a/fs/block_dev.c
+++ b/fs/block_dev.c
@@ -271,14 +271,13 @@  int thaw_bdev(struct block_device *bdev, struct 
super_block *sb)
  		error = sb->s_op->thaw_super(sb);
  	else
  		error = thaw_super(sb);
-	if (error) {
+
+	if (error)
  		bdev->bd_fsfreeze_count++;
-		mutex_unlock(&bdev->bd_fsfreeze_mutex);
-		return error;
-	}
+
  out:
  	mutex_unlock(&bdev->bd_fsfreeze_mutex);
-	return 0;
+	return error;
  }
  EXPORT_SYMBOL(thaw_bdev);