From 49837a80b38b79a7c06217b2c40842aeb6fa13b9 Mon Sep 17 00:00:00 2001 From: Al Viro Date: Fri, 28 May 2010 11:34:50 -0400 Subject: [PATCH] remove detritus left by "mm: make read_cache_page synchronous" gets minix get_dir_page() in sync with its analogs; back in 2007 Nick has switched read_cache_page() and friends to sync behaviour (i.e. they wait for the page to get unlocked, check if it's uptodate and if it isn't return ERR_PTR(-EIO) instead) and removed the duplicate logics from the callers. In case of fs/minix/dir.c he'd removed only half of that... Signed-off-by: Al Viro --- fs/minix/dir.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/fs/minix/dir.c b/fs/minix/dir.c index aaab97ec5bc0..91969589131c 100644 --- a/fs/minix/dir.c +++ b/fs/minix/dir.c @@ -72,11 +72,8 @@ static struct page * dir_get_page(struct inode *dir, unsigned long n) { struct address_space *mapping = dir->i_mapping; struct page *page = read_mapping_page(mapping, n, NULL); - if (!IS_ERR(page)) { + if (!IS_ERR(page)) kmap(page); - if (!PageUptodate(page)) - goto fail; - } return page; fail: