Commit b2bb112d authored by Gao Xiang's avatar Gao Xiang Committed by Greg Kroah-Hartman

staging: erofs: no need to take page lock in readdir

VFS will take inode_lock for readdir, therefore no need to
take page lock in readdir at all just as the majority of
other generic filesystems.
Signed-off-by: default avatarGao Xiang <gaoxiang25@huawei.com>
Reviewed-by: default avatarChao Yu <yuchao0@huawei.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 615cba35
...@@ -98,7 +98,6 @@ static int erofs_readdir(struct file *f, struct dir_context *ctx) ...@@ -98,7 +98,6 @@ static int erofs_readdir(struct file *f, struct dir_context *ctx)
if (IS_ERR(dentry_page)) if (IS_ERR(dentry_page))
continue; continue;
lock_page(dentry_page);
de = (struct erofs_dirent *)kmap(dentry_page); de = (struct erofs_dirent *)kmap(dentry_page);
nameoff = le16_to_cpu(de->nameoff); nameoff = le16_to_cpu(de->nameoff);
...@@ -128,7 +127,6 @@ static int erofs_readdir(struct file *f, struct dir_context *ctx) ...@@ -128,7 +127,6 @@ static int erofs_readdir(struct file *f, struct dir_context *ctx)
skip_this: skip_this:
kunmap(dentry_page); kunmap(dentry_page);
unlock_page(dentry_page);
put_page(dentry_page); put_page(dentry_page);
ctx->pos = blknr_to_addr(i) + ofs; ctx->pos = blknr_to_addr(i) + ofs;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment