From: Dmitry Monakhov <dmonakhov@openvz.org>
Date: Mon, 8 Nov 2010 18:47:33 +0000 (-0500)
Subject: ext4: do not try to grab the s_umount semaphore in ext4_quota_off
X-Git-Tag: upstream/snapshot3+hdmi~12516^2~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=87009d86dc045d228e21242467a67a5f99347553;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

ext4: do not try to grab the s_umount semaphore in ext4_quota_off

It's not needed to sync the filesystem, and it fixes a lock_dep complaint.

Signed-off-by: Dmitry Monakhov <dmonakhov@gmail.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Reviewed-by: Jan Kara <jack@suse.cz>
---

diff --git a/fs/ext4/super.c b/fs/ext4/super.c
index 45653af..ee91e29d 100644
--- a/fs/ext4/super.c
+++ b/fs/ext4/super.c
@@ -4570,12 +4570,10 @@ static int ext4_quota_on(struct super_block *sb, int type, int format_id,
 
 static int ext4_quota_off(struct super_block *sb, int type)
 {
-	/* Force all delayed allocation blocks to be allocated */
-	if (test_opt(sb, DELALLOC)) {
-		down_read(&sb->s_umount);
+	/* Force all delayed allocation blocks to be allocated.
+	 * Caller already holds s_umount sem */
+	if (test_opt(sb, DELALLOC))
 		sync_filesystem(sb);
-		up_read(&sb->s_umount);
-	}
 
 	return dquot_quota_off(sb, type);
 }