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=-13.6 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 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 4B397C433B4 for ; Fri, 2 Apr 2021 00:17:09 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 90D1761107 for ; Fri, 2 Apr 2021 00:17:08 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 90D1761107 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linux-foundation.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 27B036B0078; Thu, 1 Apr 2021 20:17:08 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 202D56B0081; Thu, 1 Apr 2021 20:17:08 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0CA6D6B0082; Thu, 1 Apr 2021 20:17:08 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0082.hostedemail.com [216.40.44.82]) by kanga.kvack.org (Postfix) with ESMTP id E23126B0078 for ; Thu, 1 Apr 2021 20:17:07 -0400 (EDT) Received: from smtpin07.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 9C6F38270ADA for ; Fri, 2 Apr 2021 00:17:07 +0000 (UTC) X-FDA: 77985512094.07.BF8F68C Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf24.hostedemail.com (Postfix) with ESMTP id 774CAA0000FA for ; Fri, 2 Apr 2021 00:17:05 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 0D0C461108; Fri, 2 Apr 2021 00:17:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linux-foundation.org; s=korg; t=1617322626; bh=yvOC7g6hjva6PwptF3HloL9AiBm6EDs0+4B8McV/PdQ=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=QyIFpMJshAUvLZYwHLCOhdXVDtK9dfFvBySkMkt94udWjnFtjDKE/iw3Bb4Hq2IDN W6avxIyQMFTFn8vlx/btxDZsAsvjcPHWWPVmWYXJ6vmYSVPZJDwo+CYh/Lewu2h5Z0 3bEb+Jjyc+4XdaOInmnCd17pw5WB7Ewm7orWGssE= Date: Thu, 1 Apr 2021 17:17:05 -0700 From: Andrew Morton To: Pavel Tatashin Cc: Mike Rapoport , Mike Kravetz , Mike Rapoport , linux-mm , LKML Subject: Re: [PATCH] mm/hugeltb: fix renaming of PF_MEMALLOC_NOCMA to PF_MEMALLOC_PIN Message-Id: <20210401171705.9bd350411cdee0db91332c92@linux-foundation.org> In-Reply-To: References: <20210331163816.11517-1-rppt@kernel.org> X-Mailer: Sylpheed 3.5.1 (GTK+ 2.24.31; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-Rspamd-Queue-Id: 774CAA0000FA X-Stat-Signature: 8457ix3qs579djkg73oaq9dztsday5s9 X-Rspamd-Server: rspam02 Received-SPF: none (linux-foundation.org>: No applicable sender policy available) receiver=imf24; identity=mailfrom; envelope-from=""; helo=mail.kernel.org; client-ip=198.145.29.99 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1617322625-770027 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 Thu, 1 Apr 2021 10:26:36 -0400 Pavel Tatashin wrote: > On Wed, Mar 31, 2021 at 12:38 PM Mike Rapoport wrote: > > > > From: Mike Rapoport > > > > The renaming of PF_MEMALLOC_NOCMA to PF_MEMALLOC_PIN missed one occur= rence > > in mm/hugetlb.c which causes build error: > > > > CC mm/hugetlb.o > > mm/hugetlb.c: In function =E2=80=98dequeue_huge_page_node_exact=E2=80= =99: > > mm/hugetlb.c:1081:33: error: =E2=80=98PF_MEMALLOC_NOCMA=E2=80=99 unde= clared (first use in this function); did you mean =E2=80=98PF_MEMALLOC_NO= FS=E2=80=99? > > bool pin =3D !!(current->flags & PF_MEMALLOC_NOCMA); > > ^~~~~~~~~~~~~~~~~ > > PF_MEMALLOC_NOFS > > mm/hugetlb.c:1081:33: note: each undeclared identifier is reported on= ly once for each function it appears in > > scripts/Makefile.build:273: recipe for target 'mm/hugetlb.o' failed > > make[2]: *** [mm/hugetlb.o] Error 1 > > > > Signed-off-by: Mike Rapoport > > --- > > mm/hugetlb.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/mm/hugetlb.c b/mm/hugetlb.c > > index a5236c2f7bb2..c22111f3da20 100644 > > --- a/mm/hugetlb.c > > +++ b/mm/hugetlb.c > > @@ -1078,7 +1078,7 @@ static void enqueue_huge_page(struct hstate *h,= struct page *page) > > static struct page *dequeue_huge_page_node_exact(struct hstate *h, i= nt nid) > > { > > struct page *page; > > - bool pin =3D !!(current->flags & PF_MEMALLOC_NOCMA); > > + bool pin =3D !!(current->flags & PF_MEMALLOC_PIN); >=20 > Thank you Mike! >=20 > Andrew, since "mm cma: rename PF_MEMALLOC_NOCMA to PF_MEMALLOC_PIN" is > not yet in the mainline, should I send a new version of this patch so > we won't have bisecting problems in the future? I've already added Mike's fix, as mm-cma-rename-pf_memalloc_nocma-to-pf_memalloc_pin-fix.patch. It shall fold it into mm-cma-rename-pf_memalloc_nocma-to-pf_memalloc_pin.patch prior to upstreaming, so no bisection issue.