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=-15.1 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 A96E6C433ED for ; Mon, 12 Apr 2021 15:13:27 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 1F85960FD7 for ; Mon, 12 Apr 2021 15:13:27 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1F85960FD7 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 796316B0036; Mon, 12 Apr 2021 11:13:26 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 746516B006E; Mon, 12 Apr 2021 11:13:26 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 635C38D0001; Mon, 12 Apr 2021 11:13:26 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0073.hostedemail.com [216.40.44.73]) by kanga.kvack.org (Postfix) with ESMTP id 478A06B0036 for ; Mon, 12 Apr 2021 11:13:26 -0400 (EDT) Received: from smtpin02.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id EF8BD824934B for ; Mon, 12 Apr 2021 15:13:25 +0000 (UTC) X-FDA: 78024058770.02.983B8B7 Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) by imf21.hostedemail.com (Postfix) with ESMTP id AC646E000126 for ; Mon, 12 Apr 2021 15:13:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=Content-Transfer-Encoding:Content-Type :In-Reply-To:MIME-Version:Date:Message-ID:From:References:To:Subject:Sender: Reply-To:Cc:Content-ID:Content-Description; bh=K3QQt32e+Ihaan+cSsO36hLhVbEl1gWUBbtc6CSo5/4=; b=SgG2eZxQDMaYmy1kiGQruMdsRS Uh8yCsBn5X9yhqiVAzJskKImfSsMHYxd1nOtKuAwtTRPVWK9SeocL4XGiSPdiaE9JG9kG3ejlSuMs 3akFFQX8EPNxUiplBfe6v2KWq70xhlxlVJBOGqBFCa5Ylso5Oj856HzhK3UyaRja4x+Qh2Fwxj3uR Rc0I0KkMA+FY9at0wroT6gARQYbK4iy5k5o7Wr6hxZg5tEU4xxRd8eaS+xB0aCI74oSEp6XhAK3tX he2Nkr4Qc1gDzXeLFVdPRvI2pWmNf7o6EgGgom2rClzpr5KrHmstDMY/92Fjok5ZP9/7+ODhMV4G0 Jvt+yBAg==; Received: from [2601:1c0:6280:3f0::e0e1] by desiato.infradead.org with esmtpsa (Exim 4.94 #2 (Red Hat Linux)) id 1lVyFd-0072YN-3o; Mon, 12 Apr 2021 15:13:21 +0000 Subject: Re: mmotm 2021-04-11-20-47 uploaded (fs/io_uring.c) To: Jens Axboe , akpm@linux-foundation.org, broonie@kernel.org, mhocko@suse.cz, sfr@canb.auug.org.au, linux-next@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, mm-commits@vger.kernel.org References: <20210412034813.EK9k9%akpm@linux-foundation.org> <34ed89e1-683e-7c12-ceb0-f5b71148a8a7@infradead.org> <9533afdd-208e-c25d-2e11-cc7f2c9d147b@kernel.dk> From: Randy Dunlap Message-ID: <99c73206-8273-665a-0c66-2582873ffa48@infradead.org> Date: Mon, 12 Apr 2021 08:13:15 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.0 MIME-Version: 1.0 In-Reply-To: <9533afdd-208e-c25d-2e11-cc7f2c9d147b@kernel.dk> Content-Type: text/plain; charset=utf-8 Content-Language: en-US X-Rspamd-Queue-Id: AC646E000126 X-Stat-Signature: a99ug8hd8ggx5ucujwpedn35drtgrz17 X-Rspamd-Server: rspam02 Received-SPF: none (infradead.org>: No applicable sender policy available) receiver=imf21; identity=mailfrom; envelope-from=""; helo=desiato.infradead.org; client-ip=90.155.92.199 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1618240403-902040 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: On 4/12/21 5:39 AM, Jens Axboe wrote: > On 4/12/21 1:21 AM, Randy Dunlap wrote: >> On 4/11/21 8:48 PM, akpm@linux-foundation.org wrote: >>> The mm-of-the-moment snapshot 2021-04-11-20-47 has been uploaded to >>> >>> https://www.ozlabs.org/~akpm/mmotm/ >>> >>> mmotm-readme.txt says >>> >>> README for mm-of-the-moment: >>> >>> https://www.ozlabs.org/~akpm/mmotm/ >>> >>> This is a snapshot of my -mm patch queue. Uploaded at random hopeful= ly >>> more than once a week. >>> >>> You will need quilt to apply these patches to the latest Linus releas= e (5.x >>> or 5.x-rcY). The series file is in broken-out.tar.gz and is duplicat= ed in >>> https://ozlabs.org/~akpm/mmotm/series >>> >>> The file broken-out.tar.gz contains two datestamp files: .DATE and >>> .DATE-yyyy-mm-dd-hh-mm-ss. Both contain the string yyyy-mm-dd-hh-mm-= ss, >>> followed by the base kernel version against which this patch series i= s to >>> be applied. >>> >>> This tree is partially included in linux-next. To see which patches = are >>> included in linux-next, consult the `series' file. Only the patches >>> within the #NEXT_PATCHES_START/#NEXT_PATCHES_END markers are included= in >>> linux-next. >> >> on i386: >> # CONFIG_BLOCK is not set >> >> ../fs/io_uring.c: In function =E2=80=98kiocb_done=E2=80=99: >> ../fs/io_uring.c:2766:7: error: implicit declaration of function =E2=80= =98io_resubmit_prep=E2=80=99; did you mean =E2=80=98io_put_req=E2=80=99? = [-Werror=3Dimplicit-function-declaration] >> if (io_resubmit_prep(req)) { >=20 > I'll apply the below to take care of that. >=20 Acked-by: Randy Dunlap # build-tested Thanks. >=20 > diff --git a/fs/io_uring.c b/fs/io_uring.c > index 3a837d2b8331..aa29918944f6 100644 > --- a/fs/io_uring.c > +++ b/fs/io_uring.c > @@ -2464,6 +2464,10 @@ static bool io_rw_should_reissue(struct io_kiocb= *req) > return true; > } > #else > +static bool io_resubmit_prep(struct io_kiocb *req) > +{ > + return false; > +} > static bool io_rw_should_reissue(struct io_kiocb *req) > { > return false; > @@ -2504,14 +2508,8 @@ static void io_complete_rw_iopoll(struct kiocb *= kiocb, long res, long res2) > if (kiocb->ki_flags & IOCB_WRITE) > kiocb_end_write(req); > if (unlikely(res !=3D req->result)) { > - bool fail =3D true; > - > -#ifdef CONFIG_BLOCK > - if (res =3D=3D -EAGAIN && io_rw_should_reissue(req) && > - io_resubmit_prep(req)) > - fail =3D false; > -#endif > - if (fail) { > + if (!(res =3D=3D -EAGAIN && io_rw_should_reissue(req) && > + io_resubmit_prep(req))) { > req_set_fail_links(req); > req->flags |=3D REQ_F_DONT_REISSUE; > } >=20 --=20 ~Randy