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 6B5E5C433EF for ; Fri, 11 Feb 2022 03:59:01 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id CB4E06B0074; Thu, 10 Feb 2022 22:59:00 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id C3CE96B0075; Thu, 10 Feb 2022 22:59:00 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id AB66F6B0078; Thu, 10 Feb 2022 22:59:00 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0246.hostedemail.com [216.40.44.246]) by kanga.kvack.org (Postfix) with ESMTP id 955F56B0074 for ; Thu, 10 Feb 2022 22:59:00 -0500 (EST) Received: from smtpin14.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 466C48249980 for ; Fri, 11 Feb 2022 03:59:00 +0000 (UTC) X-FDA: 79129143240.14.DD57EA1 Received: from gandalf.ozlabs.org (gandalf.ozlabs.org [150.107.74.76]) by imf17.hostedemail.com (Postfix) with ESMTP id EA9C040005 for ; Fri, 11 Feb 2022 03:58:58 +0000 (UTC) Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 4Jw0Gk6vqsz4xNn; Fri, 11 Feb 2022 14:58:54 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canb.auug.org.au; s=201702; t=1644551935; bh=tANgVZE3P47aF53XSfN41u2Y4MtV+aBH3rrzoHjf0Gk=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=TJc10X1humPxq54tTHXpi37q4W3JhMDlUmefID+CHV/iE11sjIcO8BYDjCeY11/iu N/8vUtnehArEWreU1PcHiqKssHxNjtDwoAQG/BFAA8zu2vmyXOw4WrK1Egfntjk/02 YR91FZMYgKjkif7/f9QoRButM9z/SHQlIv80u5Xq8C4FE2DKp/VImq4WoCq+UiZSZq JTU1jdLMWRy8hXfzKlw73+TRMBJg14Jgl4exgZsA0GibsuyRrZcQ1xj98VNZZONYTp g9R01QhjUtMDflCL35GymVwSzSQOvZV/hgxALOh8Tvt8THrPJdPuXnr8oi8iHuPzhq OKjIDpc26Kt0w== Date: Fri, 11 Feb 2022 14:58:53 +1100 From: Stephen Rothwell To: Hugh Dickins Cc: Andrew Morton , SeongJae Park , Geert Uytterhoeven , Naresh Kamboju , Linux MM , Linux Kernel Mailing List Subject: Re: [PATCH for-mm 1/2 v2] mm/internal: Implement no-op mlock_page_drain() for !CONFIG_MMU Message-ID: <20220211145853.329ff344@canb.auug.org.au> In-Reply-To: <8eae6026-098-befb-92d3-b9ad2ad57776@google.com> References: <20220209094158.21941-1-sj@kernel.org> <20220209094158.21941-2-sj@kernel.org> <715a8b2e-1048-c098-8b89-bcf3c13cbd75@google.com> <8eae6026-098-befb-92d3-b9ad2ad57776@google.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/uzV4b3Z7XYHDfaFovndMX.Y"; protocol="application/pgp-signature"; micalg=pgp-sha256 X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: EA9C040005 X-Stat-Signature: fuezie5z3nis4ccadigw8654zqnubu5y X-Rspam-User: Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=canb.auug.org.au header.s=201702 header.b=TJc10X1h; dmarc=none; spf=pass (imf17.hostedemail.com: domain of sfr@canb.auug.org.au designates 150.107.74.76 as permitted sender) smtp.mailfrom=sfr@canb.auug.org.au X-HE-Tag: 1644551938-10882 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: --Sig_/uzV4b3Z7XYHDfaFovndMX.Y Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Hugh, On Wed, 9 Feb 2022 20:44:02 -0800 (PST) Hugh Dickins wro= te: > > From: SeongJae Park >=20 > Commit 4b3b8bd6c8287 ("mm/munlock: mlock_page() munlock_page() batch by > pagevec") in -mm tree[1] implements 'mlock_page_drain()' under > CONFIG_MMU only, but the function is used by 'lru_add_drain_cpu()', > which defined outside of CONFIG_MMU. As a result, below build error > occurs. >=20 > /linux/mm/swap.c: In function 'lru_add_drain_cpu': > /linux/mm/swap.c:637:2: error: implicit declaration of function 'mloc= k_page_drain' [-Werror=3Dimplicit-function-declaration] > 637 | mlock_page_drain(cpu); > | ^~~~~~~~~~~~~~~~ > cc1: some warnings being treated as errors > /linux/scripts/Makefile.build:289: recipe for target 'mm/swap.o' fail= ed >=20 > This commit fixes it by implementing no-op 'mlock_page_drain()' for > !CONFIG_MMU case, similar to 'mlock_new_page()'. >=20 > [1] https://www.ozlabs.org/~akpm/mmotm/broken-out/mm-munlock-mlock_page-m= unlock_page-batch-by-pagevec.patch >=20 > [hughd: add need_mlock_page_drain() stub too] > Signed-off-by: SeongJae Park > Signed-off-by: Hugh Dickins > --- > Andrew, Stephen, please add as fix to > mm-munlock-mlock_page-munlock_page-batch-by-pagevec.patch > Thanks! >=20 > mm/internal.h | 2 ++ > 1 file changed, 2 insertions(+) >=20 > --- a/mm/internal.h > +++ b/mm/internal.h > @@ -508,6 +508,8 @@ static inline void mlock_vma_page(struct page *page, > static inline void munlock_vma_page(struct page *page, > struct vm_area_struct *vma, bool compound) { } > static inline void mlock_new_page(struct page *page) { } > +static inline bool need_mlock_page_drain(int cpu) { return false; } > +static inline void mlock_page_drain(int cpu) { } > static inline void vunmap_range_noflush(unsigned long start, unsigned lo= ng end) > { > } Added to linux-next from today. --=20 Cheers, Stephen Rothwell --Sig_/uzV4b3Z7XYHDfaFovndMX.Y Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAmIF3v0ACgkQAVBC80lX 0Gw6SAf/YdMw2mcK7OcCmFZERxK4VtcnUrURgP2yV67H8pWoh3aYufIiNrTX8QWK IL6uVdq7lXdjUD5JWcvlu7LRBIHicNfcbqSaHtBQJJlCYDbBc+Vnuk0EkFz8ERJC 3LwUxJnzrwYDz2C0uDma8tm5OGYnVzyAQYRmLXHtVRVtn3inOltpGo261fCuCzlU w3ojkVO09YGG1uF1tZoDQlx/vXSqyNLI7uvn8jAA1pgPDqM3KmypQn/ZINpwihT5 c8SUMvx9tkPUTkXsKBLsHFyvtXC0egOynMcRUs3BGjnZHNnCRWUxv0ZIRB3SyVUu 0xiKlfryatRYI9O1++ksAjlmlFa7IA== =n70x -----END PGP SIGNATURE----- --Sig_/uzV4b3Z7XYHDfaFovndMX.Y--