diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2023-01-19 10:47:00 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-10-06 14:56:44 +0200 |
| commit | a84ac8995ac70728fc599501f4d781327f0e7f27 (patch) | |
| tree | 43497d1dfc4d3a516d90e427080bab3977e427f6 /fs/f2fs | |
| parent | 8b3b859bf8bed573c7abcc7b17f638b56a0ee287 (diff) | |
| download | linux-stable-a84ac8995ac70728fc599501f4d781327f0e7f27.tar.gz linux-stable-a84ac8995ac70728fc599501f4d781327f0e7f27.tar.bz2 linux-stable-a84ac8995ac70728fc599501f4d781327f0e7f27.zip | |
f2fs: get out of a repeat loop when getting a locked data page
[ Upstream commit d2d9bb3b6d2fbccb5b33d3a85a2830971625a4ea ]
https://bugzilla.kernel.org/show_bug.cgi?id=216050
Somehow we're getting a page which has a different mapping.
Let's avoid the infinite loop.
Cc: <stable@vger.kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/data.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index 0faed0575f8a..a982f91b71eb 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -1329,18 +1329,14 @@ struct page *f2fs_get_lock_data_page(struct inode *inode, pgoff_t index, { struct address_space *mapping = inode->i_mapping; struct page *page; -repeat: + page = f2fs_get_read_data_page(inode, index, 0, for_write, NULL); if (IS_ERR(page)) return page; /* wait for read completion */ lock_page(page); - if (unlikely(page->mapping != mapping)) { - f2fs_put_page(page, 1); - goto repeat; - } - if (unlikely(!PageUptodate(page))) { + if (unlikely(page->mapping != mapping || !PageUptodate(page))) { f2fs_put_page(page, 1); return ERR_PTR(-EIO); } |