The cramfs readpage implementation was already synchronous, so use
AOP_UPDATED_PAGE to avoid cycling the page lock.

Signed-off-by: Matthew Wilcox (Oracle) <wi...@infradead.org>
---
 fs/cramfs/inode.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/fs/cramfs/inode.c b/fs/cramfs/inode.c
index 912308600d39..7a642146c074 100644
--- a/fs/cramfs/inode.c
+++ b/fs/cramfs/inode.c
@@ -916,15 +916,14 @@ static int cramfs_readpage(struct file *file, struct page 
*page)
        flush_dcache_page(page);
        kunmap(page);
        SetPageUptodate(page);
-       unlock_page(page);
-       return 0;
+       return AOP_UPDATED_PAGE;
 
 err:
        kunmap(page);
        ClearPageUptodate(page);
        SetPageError(page);
        unlock_page(page);
-       return 0;
+       return -EIO;
 }
 
 static const struct address_space_operations cramfs_aops = {
-- 
2.28.0

Reply via email to