lkml.org 
[lkml]   [2013]   [Jan]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 6/6] ext3: Convert ext3 to use mapping lock
Date
Convert the filesystem to use mapping lock when truncating files and doing
buffered writes. The rest is handled in the generic code.

Signed-off-by: Jan Kara <jack@suse.cz>
---
fs/ext3/inode.c | 17 +++++++++++++++++
1 files changed, 17 insertions(+), 0 deletions(-)

diff --git a/fs/ext3/inode.c b/fs/ext3/inode.c
index b176d42..1f5d5f1 100644
--- a/fs/ext3/inode.c
+++ b/fs/ext3/inode.c
@@ -1261,6 +1261,7 @@ static int ext3_write_begin(struct file *file, struct address_space *mapping,
index = pos >> PAGE_CACHE_SHIFT;
from = pos & (PAGE_CACHE_SIZE - 1);
to = from + len;
+ flags |= AOP_FLAG_LOCK_MAPPING;

retry:
page = grab_cache_page_write_begin(mapping, index, flags);
@@ -3273,6 +3274,8 @@ int ext3_setattr(struct dentry *dentry, struct iattr *attr)
struct inode *inode = dentry->d_inode;
int error, rc = 0;
const unsigned int ia_valid = attr->ia_valid;
+ struct range_lock mapping_lock;
+ int range_locked = 0;

error = inode_change_ok(inode, attr);
if (error)
@@ -3314,6 +3317,12 @@ int ext3_setattr(struct dentry *dentry, struct iattr *attr)
attr->ia_valid & ATTR_SIZE && attr->ia_size < inode->i_size) {
handle_t *handle;

+ range_lock_init(&mapping_lock,
+ attr->ia_size >> PAGE_CACHE_SHIFT,
+ ULONG_MAX);
+ range_lock(&inode->i_mapping->mapping_lock, &mapping_lock);
+ range_locked = 1;
+
handle = ext3_journal_start(inode, 3);
if (IS_ERR(handle)) {
error = PTR_ERR(handle);
@@ -3351,6 +3360,12 @@ int ext3_setattr(struct dentry *dentry, struct iattr *attr)
attr->ia_size != i_size_read(inode)) {
truncate_setsize(inode, attr->ia_size);
ext3_truncate(inode);
+
+ if (range_locked) {
+ range_unlock(&inode->i_mapping->mapping_lock,
+ &mapping_lock);
+ range_locked = 0;
+ }
}

setattr_copy(inode, attr);
@@ -3360,6 +3375,8 @@ int ext3_setattr(struct dentry *dentry, struct iattr *attr)
rc = ext3_acl_chmod(inode);

err_out:
+ if (range_locked)
+ range_unlock(&inode->i_mapping->mapping_lock, &mapping_lock);
ext3_std_error(inode->i_sb, error);
if (!error)
error = rc;
--
1.7.1


\
 
 \ /
  Last update: 2013-01-31 23:43    [W:0.194 / U:0.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site