diff options
| author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2022-05-17 18:12:25 -0400 |
|---|---|---|
| committer | Matthew Wilcox (Oracle) <willy@infradead.org> | 2022-06-29 08:51:07 -0400 |
| commit | 771075e15e39b30abcf38186592383f38d06a40f (patch) | |
| tree | f41448d4fb9c387c2cdd747f4ec20c289da1cd92 | |
| parent | b0c971e7b76887bfab44e6b891e90ebb4c637172 (diff) | |
| download | linux-771075e15e39b30abcf38186592383f38d06a40f.tar.gz linux-771075e15e39b30abcf38186592383f38d06a40f.tar.bz2 linux-771075e15e39b30abcf38186592383f38d06a40f.zip | |
ufs: Remove checks for PageError
If read_mapping_page() encounters an error, it returns an errno, not
a page with PageError set, or a page that is not Uptodate, so this is
dead code.
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
| -rw-r--r-- | fs/ufs/dir.c | 2 | ||||
| -rw-r--r-- | fs/ufs/util.c | 11 |
2 files changed, 1 insertions, 12 deletions
diff --git a/fs/ufs/dir.c b/fs/ufs/dir.c index b721d0bda5e5..391efaf1d528 100644 --- a/fs/ufs/dir.c +++ b/fs/ufs/dir.c @@ -193,7 +193,7 @@ static struct page *ufs_get_page(struct inode *dir, unsigned long n) if (!IS_ERR(page)) { kmap(page); if (unlikely(!PageChecked(page))) { - if (PageError(page) || !ufs_check_page(page)) + if (!ufs_check_page(page)) goto fail; } } diff --git a/fs/ufs/util.c b/fs/ufs/util.c index 4fa633f84274..08ddf41eaaad 100644 --- a/fs/ufs/util.c +++ b/fs/ufs/util.c @@ -264,17 +264,6 @@ struct page *ufs_get_locked_page(struct address_space *mapping, put_page(page); return NULL; } - - if (!PageUptodate(page) || PageError(page)) { - unlock_page(page); - put_page(page); - - printk(KERN_ERR "ufs_change_blocknr: " - "can not read page: ino %lu, index: %lu\n", - inode->i_ino, index); - - return ERR_PTR(-EIO); - } } if (!page_has_buffers(page)) create_empty_buffers(page, 1 << inode->i_blkbits, 0); |