Commit 49837a80 authored May 28, 2010 by Al Viro

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 <viro@zeniv.linux.org.uk>

parent 4c9002de

fs/minix/dir.c

+1 −4

Original line number	Original line	Diff line number	Diff line
	@@ -72,11 +72,8 @@ static struct page * dir_get_page(struct inode *dir, unsigned long n)
	{		{
	struct address_space *mapping = dir->i_mapping;		struct address_space *mapping = dir->i_mapping;
	struct page *page = read_mapping_page(mapping, n, NULL);		struct page *page = read_mapping_page(mapping, n, NULL);
	if (!IS_ERR(page)) {		if (!IS_ERR(page))
	kmap(page);		kmap(page);
	if (!PageUptodate(page))
	goto fail;
	}
	return page;		return page;

	fail:		fail: