Now that read_mapping_page() does error checking internally, there is no need to check PageError here. Signed-off-by: Nate Diller <nate.diller@xxxxxxxxx> --- diff -urpN -X dontdiff linux-2.6.21-rc6-mm1/fs/hfsplus/bnode.c linux-2.6.21-rc6-mm1-test/fs/hfsplus/bnode.c --- linux-2.6.21-rc6-mm1/fs/hfsplus/bnode.c 2007-04-09 17:20:13.000000000 -0700 +++ linux-2.6.21-rc6-mm1-test/fs/hfsplus/bnode.c 2007-04-10 21:28:45.000000000 -0700 @@ -442,10 +442,6 @@ static struct hfs_bnode *__hfs_bnode_cre page = read_mapping_page(mapping, block, NULL); if (IS_ERR(page)) goto fail; - if (PageError(page)) { - page_cache_release(page); - goto fail; - } page_cache_release(page); node->page[i] = page; } - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html