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 054E6C433EF for ; Thu, 3 Mar 2022 17:52:47 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 6F9BF8D0002; Thu, 3 Mar 2022 12:52:47 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 6A7FE8D0001; Thu, 3 Mar 2022 12:52:47 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5970E8D0002; Thu, 3 Mar 2022 12:52:47 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (relay.hostedemail.com [64.99.140.28]) by kanga.kvack.org (Postfix) with ESMTP id 463A58D0001 for ; Thu, 3 Mar 2022 12:52:47 -0500 (EST) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 00AD12215 for ; Thu, 3 Mar 2022 17:52:46 +0000 (UTC) X-FDA: 79203820374.01.F4BD516 Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf14.hostedemail.com (Postfix) with ESMTP id 09066100004 for ; Thu, 3 Mar 2022 17:52:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=FoSYhh+8GMMoPaKqHvmCVfeuwA9p77ql1l5blA4VkA8=; b=l/LIUKluOsswkZOwi66Ia8ud6h JiFS46+YAkHlkpNGRuyO3/h66eNunvvI8hfq09GahcVJE7W108zWJRh3OFSWs/1qhkUYm6wqA4/dD Nznvp01x64dBQJc47fkT5ouX2cNc5ZuhAOdsHpKEVuZ72T4q9OZB1yVWuuSbgeNeLt5UZXEgHfWNw yzSQbIS+DyA6Yvza98rOyiDDymiHnwQG1LaDVpQj+AuoztzeNvEy9scMiRSAEvIDdwONFtKJH9xkH GV2DzEJMpQ8mCWwqqXlfxWtXDlDmBo/DOeau7aQ8jOuTzIy8WKgI8KNukuz//DxlOYu0u2roJPKa1 jXkUAkww==; Received: from willy by casper.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1nPpd2-00BrH1-5q; Thu, 03 Mar 2022 17:52:40 +0000 Date: Thu, 3 Mar 2022 17:52:40 +0000 From: Matthew Wilcox To: Hugh Dickins Cc: Andrew Morton , Vlastimil Babka , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH mmotm] mm/munlock: mlock_vma_folio() check against VM_SPECIAL Message-ID: References: <9b95d366-1719-f8e2-a5a3-429f9e808288@google.com> <39f9e61b-c66c-a741-5118-997a48dd440@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <39f9e61b-c66c-a741-5118-997a48dd440@google.com> X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 09066100004 X-Stat-Signature: oas8rcstnwnxzx5rjr6w4pe6t9t6kqpb Authentication-Results: imf14.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b="l/LIUKlu"; spf=none (imf14.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org; dmarc=none X-Rspam-User: X-HE-Tag: 1646329965-361205 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, Mar 03, 2022 at 09:25:47AM -0800, Hugh Dickins wrote: > On Thu, 3 Mar 2022, Matthew Wilcox wrote: > > It looks like it fixes "mm/munlock: mlock_pte_range() when mlocking or > > munlocking", so I'll fold it into that patch? > > No and yes. > > That's great if you're prepared to move it back before the foliation. > I think that just involves editing every "folio" to "page", including > in the title - I very nearly sent it out with mlock_vma_page() in title. Thanks for the reminder to do the title and changelog ;-) > But I would prefer it to remain as a separate fix at the end of the > mm/munlock series: this case is too unusual, and only a "Bad page", > to mess with bisection prospects; and it's addressing an entirely > different issue from what the "mlock_pte_range()..." is dealing with. > Each of them needs its own explanation. > > So I would prefer it as a separate fix about "page"s, on top of the > mm/munlock series, and you then adjust your foliation commit accordingly. Done! Pushed out. > Thank you: this is what I really wanted, but was afraid to ask of you > (and of course, other fixes may turn out to be required, too late to > adjust across the page<->folio barrier in this way: so it's nice to > be able to do it this way, but rather beyond the call of duty). Yes, once it's landed in Linus' tree, it's too late to edit, but I'm willing to insert patches like this. It's a minor adjustment to my patch on top of it.