From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id ACD3FC433F5 for ; Sat, 28 May 2022 05:56:56 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 442DE8D0003; Sat, 28 May 2022 01:56:56 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 3F27B8D0001; Sat, 28 May 2022 01:56:56 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2E03F8D0003; Sat, 28 May 2022 01:56:56 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 1FF448D0001 for ; Sat, 28 May 2022 01:56:56 -0400 (EDT) Received: from smtpin18.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay12.hostedemail.com (Postfix) with ESMTP id E2FA9120BD9 for ; Sat, 28 May 2022 05:56:55 +0000 (UTC) X-FDA: 79514093190.18.F1750FA Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) by imf04.hostedemail.com (Postfix) with ESMTP id 3FECB40046 for ; Sat, 28 May 2022 05:56:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=dQyv3eHL6CJ2dCoh+2jN3ynYnY2ODDo+v2cNgQYrw+U=; b=GU8o6f1zYwaqd1DdI8cPMthpT2 hOfT3xnUtclp5nOBzz5GPOIUpn+aARunUv5ukBnLZfLUlNHYZffDx5UNneUHNPO/cFH70CQk0x+EQ 7Flm49fs45Vwd78c4ok9wdVxi7XL4Oj9fWhBKv+ftnO34wfOt5ph1FjzYaa8rPWEtJSaZ4BzLQ7S7 pi0/kcZrv0mZGR8CXaYMz9bBAdPyPNkyWSwMnc4X6G4STl14+5LMdX6BeDFPoZmfrl2RvZPvQE5jq MamEWRvsWIgpWXLvgFRf0TKBjySahSoRqf++wvKx7ncsu0UDYwgbR+e27dbtkufyGAUjQ+uRqFoEi umhcVQdQ==; Received: from hch by bombadil.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1nupRU-001WQY-AK; Sat, 28 May 2022 05:56:52 +0000 Date: Fri, 27 May 2022 22:56:52 -0700 From: Christoph Hellwig To: "Matthew Wilcox (Oracle)" Cc: linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH 13/24] ufs: Remove checks for PageError Message-ID: References: <20220527155036.524743-1-willy@infradead.org> <20220527155036.524743-14-willy@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220527155036.524743-14-willy@infradead.org> X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html X-Rspamd-Queue-Id: 3FECB40046 X-Rspam-User: Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=infradead.org header.s=bombadil.20210309 header.b=GU8o6f1z; spf=none (imf04.hostedemail.com: domain of BATV+c5e11ba0f85895bf2855+6852+infradead.org+hch@bombadil.srs.infradead.org has no SPF policy when checking 198.137.202.133) smtp.mailfrom=BATV+c5e11ba0f85895bf2855+6852+infradead.org+hch@bombadil.srs.infradead.org; dmarc=none X-Stat-Signature: uzoga4drgp7fgzwgd13n6bb161y1ot6q X-Rspamd-Server: rspam05 X-HE-Tag: 1653717398-617214 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: > --- 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; > } Unrelated note: doing the PageChecked check inside of ufs_check_page wuld really help readability for the casual reader. > } > 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); > - } This looks good. But this code could use some more love nd a removal of the find_lock_page call by always just using read_mapping_page. Especially a the truncate protection should apply equally to cached pages and not just those freshly read off the disk. But I guess for now this looks good: Reviewed-by: Christoph Hellwig