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 62A0AC4332F for ; Tue, 13 Dec 2022 14:28:31 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 85FE88E0003; Tue, 13 Dec 2022 09:28:30 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 80F558E0002; Tue, 13 Dec 2022 09:28:30 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6B04F8E0003; Tue, 13 Dec 2022 09:28:30 -0500 (EST) 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 5B8848E0002 for ; Tue, 13 Dec 2022 09:28:30 -0500 (EST) Received: from smtpin06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id B156680CC9 for ; Tue, 13 Dec 2022 14:28:29 +0000 (UTC) X-FDA: 80237513538.06.5FDDF78 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by imf05.hostedemail.com (Postfix) with ESMTP id 7F2B910000F for ; Tue, 13 Dec 2022 14:28:26 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=korg header.b=LNENHUPh; spf=pass (imf05.hostedemail.com: domain of gregkh@linuxfoundation.org designates 145.40.68.75 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org; dmarc=pass (policy=none) header.from=linuxfoundation.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1670941707; a=rsa-sha256; cv=none; b=c7JJACZp0VCJJ7y7tkK+BnmWGwO8KzmdiM5myWa3pSu9G8Q+GGnf/Lgb5IseYpmwR1HtNC 99WvnEMmdz42GRFnevDce8fM2HbjtP/S32cf8WjP0VPVYZGhwjccqrbVu8qwGQQsJUYPe6 Ce5zLTyTe0IBavCaXDvmQReRF4iGxKM= ARC-Authentication-Results: i=1; imf05.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=korg header.b=LNENHUPh; spf=pass (imf05.hostedemail.com: domain of gregkh@linuxfoundation.org designates 145.40.68.75 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org; dmarc=pass (policy=none) header.from=linuxfoundation.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1670941707; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=PXivUe4SlT51H+XkbxzD8tHS8O051eqsLitHVQp2Yh4=; b=KUcgf9K1od4+pAiHS5DAwbx8PxF2RUNti0DEpFkHNEgWj78jW414Z1+8ecqpy8zyygf5Kj 2IatsZ/mwKXO7L/MLw8tloTaFEGGpNx1nsqMPjzKy8KBZoWnL21DGH/g6V6ezKUDCgc5/z 1+oiikotB0fqdP28zsaQAdqKTOc0zi0= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 582D4B81201; Tue, 13 Dec 2022 14:28:24 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9B8F5C433EF; Tue, 13 Dec 2022 14:28:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1670941703; bh=rrzaJDPWmltpz/V6l3a6Gkl9FuLfJ97cFLrnX00SHfg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=LNENHUPhAT8Ebw6l5tkFIzGYdcaEZijTDBiQnLEAvwbdxNmGGQPxzp25kl2SH3ZHK F3FtyStongOr3rgWMLRImm2NUtrsQEY3O8jmFFJTRZ3dqO3dhN0vB0xE1TkETSt9gg Gaxp5+NPXEBhXMmg1zbpLP4S2KL3qykFVNP/3q9U= Date: Tue, 13 Dec 2022 15:28:19 +0100 From: Greg Kroah-Hartman To: Hugh Dickins Cc: stable@vger.kernel.org, patches@lists.linux.dev, Alex Shi , Johannes Weiner , Vlastimil Babka , "Kirill A. Shutemov" , Alexander Duyck , Andrea Arcangeli , Andrey Ryabinin , "Chen, Rong A" , Daniel Jordan , "Huang, Ying" , Jann Horn , Joonsoo Kim , "Kirill A. Shutemov" , Konstantin Khlebnikov , "Matthew Wilcox (Oracle)" , Mel Gorman , Michal Hocko , Michal Hocko , Mika =?iso-8859-1?Q?Penttil=E4?= , Minchan Kim , Shakeel Butt , Tejun Heo , Thomas Gleixner , Vladimir Davydov , Wei Yang , Yang Shi , Andrew Morton , Linus Torvalds , Sasha Levin , Gavin Shan , Zhenyu Zhang , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH 5.10 001/106] mm/mlock: remove lru_lock on TestClearPageMlocked Message-ID: References: <20221212130924.863767275@linuxfoundation.org> <20221212130924.929782499@linuxfoundation.org> <8ad6ed6-5f7c-f1cd-8693-caf88bfca73a@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <8ad6ed6-5f7c-f1cd-8693-caf88bfca73a@google.com> X-Rspam-User: X-Rspamd-Queue-Id: 7F2B910000F X-Rspamd-Server: rspam01 X-Stat-Signature: n83w6qk4hk1y7y16kr8nd95ximtrxgax X-HE-Tag: 1670941706-348283 X-HE-Meta: U2FsdGVkX18Vo0pjI30DQJFvd29QJ3t5LRSSJqvBTb953pqUOmOsGFAP9K+AAAu6OXXZMcBCZfIlG4Yjp33NYuyJ/kYRsw86PpyqYZkqINsYzrGuRvF8TxvquDYLdB1mrKz/g7dENsmXRlU8YareaG6BClF8/wWZBLFHN6DPhdyRaOtdTJfviUzT42YvlpVyWoqE5Cl9YEUamCwKgVNSANWi7RDSNfgfqfuXxfAnteQlOJ428B9yXe7hMFkpKWNex3gj2MtHcroDHfmLvujvTnQIbz+CGWX9Vc61BfsZRlCjet5VHcgVFK14uYusYnlpu7QZfHxGlKuTzGz+OL+XJIgYJSws/81aRds0o/dJTK0+hHh2qVKZGrvVvW8M/wt6Z6O/PF5SzoELyR9VdF6GT82LxEc6EOopq5caHAjNOo90lM8xvUnKTeRzvBQcnjQ0iMV+pjszyy6oXBIFhd/Xg2+TBjaUvP35ZxOqK4st2969LK/0yv7AhZobN5tzCwf8vQFi7klYcgoedbFFl7nvEQuGCJ+JQdznAfYkK47L5GijaSft9rbsLfheN0yQEurpSXGBzziqagoj0FD1kI+viNT3y6hC9zxMn5XH0YlRzpyGr5mTV0JKkNNjaEC/V82jnG8F5pt8+MOYBjvvHklX91LovgIlI4zfBtlpBdfn9yYKggf9P1KDwZaohq4Hw/+lZlhJ/NbNijIvLOxPh2lqx3Iq3fmxKyaYhN6XAcZ/bP5mSltOtublT9RF3OgYasCYiISDqjnJhde2EckYsLZNb8VTq8pzeoIREbqqZchz8JPeC2gllDTci9yapelVYsbgaZuiiBCv8fasE078noQSO/x5OpiDuqyX6z6ULkCggt3v5mqQk/BLVeRV+XcHwEOEiHwmrh5TWAIhlr8i4l19PsEsrgqyn6/9H3PFVTKSnY7KX7aeNrhoKAgzo1Mt27ePptYildIVL9Db3d+B4Oo /XxrwRsp Wuima7tHTLCbcYb0OwwmV0sxHsPytkG7eEy6i 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 Mon, Dec 12, 2022 at 12:35:57PM -0800, Hugh Dickins wrote: > On Mon, 12 Dec 2022, Greg Kroah-Hartman wrote: > > > From: Alex Shi > > > > [ Upstream commit 3db19aa39bac33f2e850fa1ddd67be29b192e51f ] > > > > In the func munlock_vma_page, comments mentained lru_lock needed for > > serialization with split_huge_pages. But the page must be PageLocked as > > well as pages in split_huge_page series funcs. Thus the PageLocked is > > enough to serialize both funcs. > > > > Further more, Hugh Dickins pointed: before splitting in > > split_huge_page_to_list, the page was unmap_page() to remove pmd/ptes > > which protect the page from munlock. Thus, no needs to guard > > __split_huge_page_tail for mlock clean, just keep the lru_lock there for > > isolation purpose. > > > > LKP found a preempt issue on __mod_zone_page_state which need change to > > mod_zone_page_state. Thanks! > > > > Link: https://lkml.kernel.org/r/1604566549-62481-13-git-send-email-alex.shi@linux.alibaba.com > > Signed-off-by: Alex Shi > > Acked-by: Hugh Dickins > > Acked-by: Johannes Weiner > > Acked-by: Vlastimil Babka > > Cc: Kirill A. Shutemov > > Cc: Alexander Duyck > > Cc: Andrea Arcangeli > > Cc: Andrey Ryabinin > > Cc: "Chen, Rong A" > > Cc: Daniel Jordan > > Cc: "Huang, Ying" > > Cc: Jann Horn > > Cc: Joonsoo Kim > > Cc: Kirill A. Shutemov > > Cc: Konstantin Khlebnikov > > Cc: Matthew Wilcox (Oracle) > > Cc: Mel Gorman > > Cc: Michal Hocko > > Cc: Michal Hocko > > Cc: Mika Penttilä > > Cc: Minchan Kim > > Cc: Shakeel Butt > > Cc: Tejun Heo > > Cc: Thomas Gleixner > > Cc: Vladimir Davydov > > Cc: Wei Yang > > Cc: Yang Shi > > Signed-off-by: Andrew Morton > > Signed-off-by: Linus Torvalds > > Stable-dep-of: 829ae0f81ce0 ("mm: migrate: fix THP's mapcount on isolation") > > Signed-off-by: Sasha Levin > > NAK from me to patches 001 through 007 here: 001 through 006 are a > risky subset of patches and followups to a per-memcg per-node lru_lock > series from Alex Shi, which made subtle changes to locking, memcg > charging, lru management, page migration etc. > > The whole series could be backported to 5.10 (I did so myself for > internal usage), but cherry-picking parts of it into 5.10-stable is > misguided and contrary to stable principles. > > Maybe there is in fact nothing wrong with the selection made: > but then give linux-mm guys two or three weeks to review and > test and give the thumbs up to that selection. > > Much easier, quicker and safer would be to adjust 007 (I presume > the reason behind 001 through 006) to fit the 5.10-stable tree: > I can do that myself if you ask, but not until later this week. All now dropped, thanks. greg k-h