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 X-Spam-Level: X-Spam-Status: No, score=-12.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1D0AFC2D0A3 for ; Tue, 10 Nov 2020 03:38:34 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 6BADF208FE for ; Tue, 10 Nov 2020 03:37:30 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="Wbs/sNmN" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6BADF208FE Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id A0DCF6B0073; Mon, 9 Nov 2020 22:37:15 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 993DA6B0074; Mon, 9 Nov 2020 22:37:15 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 7E4BA6B0075; Mon, 9 Nov 2020 22:37:15 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0242.hostedemail.com [216.40.44.242]) by kanga.kvack.org (Postfix) with ESMTP id 4EFD96B0073 for ; Mon, 9 Nov 2020 22:37:15 -0500 (EST) Received: from smtpin13.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id EFA108249980 for ; Tue, 10 Nov 2020 03:37:14 +0000 (UTC) X-FDA: 77467097988.13.skate04_0313a2b272f1 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin13.hostedemail.com (Postfix) with ESMTP id D2A1018140B69 for ; Tue, 10 Nov 2020 03:37:14 +0000 (UTC) X-HE-Tag: skate04_0313a2b272f1 X-Filterd-Recvd-Size: 4510 Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf30.hostedemail.com (Postfix) with ESMTP for ; Tue, 10 Nov 2020 03:37:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From:Sender:Reply-To: Content-Type:Content-ID:Content-Description; bh=m0eXNuUXEXWU5JN/qKJgJTRZ3NVya1wJfACcs8w99Cc=; b=Wbs/sNmNnggnxfXcilWsf7H743 7jEwOnD03obu5qVjlfrnpgS8XsFeNqxOgnKq/Og/NCPGllAGizz6LwYnEc1sT7ThykJiXKLbV904g xJeHKcVxr5LQXd3y7ETCYkYdQ1994T4ODuIA9pjov5SyG0l5YkgWl6cs6mRykHpDzPPIpSdZefD/X zCtLc02eG/3eHOlMg8Iv/vf4YU3GTKYPxwZu8P6KE9do0mg+6r/pJAOa2bj0a8LCyGEROZVmCVg0I KKJFqyV791OYpqL/jLQrbkpeE+GO3GC3GV3kF6WLeevpL/pmob3euFKaLFE1Wpb9tQKJ1gezno+zS UrsxPcIA==; Received: from willy by casper.infradead.org with local (Exim 4.92.3 #3 (Red Hat Linux)) id 1kcKT1-00065j-FG; Tue, 10 Nov 2020 03:37:11 +0000 From: "Matthew Wilcox (Oracle)" To: linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Cc: "Matthew Wilcox (Oracle)" , hch@lst.de, kent.overstreet@gmail.com Subject: [PATCH v3 10/18] mm/filemap: Change filemap_create_page calling conventions Date: Tue, 10 Nov 2020 03:36:55 +0000 Message-Id: <20201110033703.23261-11-willy@infradead.org> X-Mailer: git-send-email 2.21.3 In-Reply-To: <20201110033703.23261-1-willy@infradead.org> References: <20201110033703.23261-1-willy@infradead.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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: By moving the iocb flag checks to the caller, we can pass the file and the page index instead of the iocb. It never needed the iter. By passing the pagevec, we can return an errno (or AOP_TRUNCATED_PAGE) instead of an ERR_PTR. Signed-off-by: Matthew Wilcox (Oracle) Reviewed-by: Kent Overstreet Reviewed-by: Christoph Hellwig --- mm/filemap.c | 53 ++++++++++++++++++++++++++-------------------------- 1 file changed, 27 insertions(+), 26 deletions(-) diff --git a/mm/filemap.c b/mm/filemap.c index 52d606916477..ddc881c9d253 100644 --- a/mm/filemap.c +++ b/mm/filemap.c @@ -2288,32 +2288,33 @@ static struct page *filemap_update_page(struct ki= ocb *iocb, struct file *filp, return NULL; } =20 -static struct page *filemap_create_page(struct kiocb *iocb, - struct iov_iter *iter) +static int filemap_create_page(struct file *file, + struct address_space *mapping, pgoff_t index, + struct pagevec *pvec) { - struct file *filp =3D iocb->ki_filp; - struct address_space *mapping =3D filp->f_mapping; - pgoff_t index =3D iocb->ki_pos >> PAGE_SHIFT; struct page *page; int error; =20 - if (iocb->ki_flags & (IOCB_NOIO | IOCB_NOWAIT | IOCB_WAITQ)) - return ERR_PTR(-EAGAIN); - page =3D page_cache_alloc(mapping); if (!page) - return ERR_PTR(-ENOMEM); + return -ENOMEM; =20 error =3D add_to_page_cache_lru(page, mapping, index, - mapping_gfp_constraint(mapping, GFP_KERNEL)); - if (!error) - error =3D filemap_read_page(iocb->ki_filp, mapping, page); - if (!error) - return page; + mapping_gfp_constraint(mapping, GFP_KERNEL)); + if (error =3D=3D -EEXIST) + error =3D AOP_TRUNCATED_PAGE; + if (error) + goto error; + + error =3D filemap_read_page(file, mapping, page); + if (error) + goto error; + + pagevec_add(pvec, page); + return 0; +error: put_page(page); - if (error =3D=3D -EEXIST || error =3D=3D AOP_TRUNCATED_PAGE) - return NULL; - return ERR_PTR(error); + return error; } =20 static int filemap_get_pages(struct kiocb *iocb, struct iov_iter *iter, @@ -2341,15 +2342,15 @@ static int filemap_get_pages(struct kiocb *iocb, = struct iov_iter *iter, page_cache_sync_readahead(mapping, ra, filp, index, last_index - index)= ; =20 filemap_get_read_batch(mapping, index, last_index, pvec); - if (pvec->nr) - goto got_pages; - - pvec->pages[0] =3D filemap_create_page(iocb, iter); - err =3D PTR_ERR_OR_ZERO(pvec->pages[0]); - if (IS_ERR_OR_NULL(pvec->pages[0])) - goto err; - pvec->nr =3D 1; - return 0; + if (!pagevec_count(pvec)) { + if (iocb->ki_flags & (IOCB_NOWAIT | IOCB_WAITQ)) + return -EAGAIN; + err =3D filemap_create_page(filp, mapping, + iocb->ki_pos >> PAGE_SHIFT, pvec); + if (err =3D=3D AOP_TRUNCATED_PAGE) + goto find_page; + return err; + } got_pages: { struct page *page =3D pvec->pages[pvec->nr - 1]; --=20 2.28.0