diff options
| author | Jan Kara <jack@suse.cz> | 2012-04-25 23:26:54 +0400 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2012-05-16 01:34:38 +0400 |
| commit | 0b7f7cefaea96ef25dbe54b8e16d2d61560d8b9d (patch) | |
| tree | 640e7107056b7cc5cf06b1c03ba569bd241b6c31 /fs/ext4 | |
| parent | 905c3937625d14fa4453a2643814b72b8a22d9ae (diff) | |
| download | linux-0b7f7cefaea96ef25dbe54b8e16d2d61560d8b9d.tar.xz | |
ext4: Remove i_mutex use from ext4_quota_write()
We don't need i_mutex in ext4_quota_write() because writes to quota file
are serialized by dqio_mutex anyway. Changes to quota files outside of quota
code are forbidded and enforced by NOATIME and IMMUTABLE bits.
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ext4')
| -rw-r--r-- | fs/ext4/super.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/fs/ext4/super.c b/fs/ext4/super.c index ceebaf853beb..97938db9f39d 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c @@ -4760,7 +4760,6 @@ static ssize_t ext4_quota_write(struct super_block *sb, int type, return -EIO; } - mutex_lock_nested(&inode->i_mutex, I_MUTEX_QUOTA); bh = ext4_bread(handle, inode, blk, 1, &err); if (!bh) goto out; @@ -4776,16 +4775,13 @@ static ssize_t ext4_quota_write(struct super_block *sb, int type, err = ext4_handle_dirty_metadata(handle, NULL, bh); brelse(bh); out: - if (err) { - mutex_unlock(&inode->i_mutex); + if (err) return err; - } if (inode->i_size < off + len) { i_size_write(inode, off + len); EXT4_I(inode)->i_disksize = inode->i_size; ext4_mark_inode_dirty(handle, inode); } - mutex_unlock(&inode->i_mutex); return len; } |