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 2971DC433EF for ; Mon, 14 Feb 2022 06:52:19 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 9E74C6B0072; Mon, 14 Feb 2022 01:52:18 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 9708D6B0073; Mon, 14 Feb 2022 01:52:18 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 810E66B0078; Mon, 14 Feb 2022 01:52:18 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0204.hostedemail.com [216.40.44.204]) by kanga.kvack.org (Postfix) with ESMTP id 6C50F6B0072 for ; Mon, 14 Feb 2022 01:52:18 -0500 (EST) Received: from smtpin22.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 26A2818128B02 for ; Mon, 14 Feb 2022 06:52:18 +0000 (UTC) X-FDA: 79140466356.22.A431650 Received: from mail-qt1-f172.google.com (mail-qt1-f172.google.com [209.85.160.172]) by imf07.hostedemail.com (Postfix) with ESMTP id A5A4D40002 for ; Mon, 14 Feb 2022 06:52:17 +0000 (UTC) Received: by mail-qt1-f172.google.com with SMTP id y8so14660348qtn.8 for ; Sun, 13 Feb 2022 22:52:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=date:from:to:cc:subject:in-reply-to:message-id:references :mime-version; bh=EopoKbEevrqLLp9ZzPCZTcvF6J4LEYpvvNmknpmGcQ4=; b=BeUxMuEePib8qdIFCiAfvIKXi1J/Vzn+EWXyESq2TdUZwquqPO0vhwBhMJ8MjK7miy PkBPvyTAJAH90Vv86IleeL0YQwXS0y1cP56wVvt+ULsjqz4BcEPkakWJgCmVMcTsZv10 /4TzCZ6XClW/x0QAo9KPIydGL0HFjwXyxGPEXpLuJUOWCod5hl6iaSuthCcjk2zEpk3d svJLwQ/ldGIs6CLugsHgGp5oE+Fnp3rxvNHT7NuM3teT04YpRHg2q87F3qmH/bPhmGn9 1Yr7y0h9xR5cHvzCUBX+JQ0du4Hb5BKQ8h/wnBWBJsbdgAvpc6z6ifnPMWgqyPV2K6Bs PPKw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:in-reply-to:message-id :references:mime-version; bh=EopoKbEevrqLLp9ZzPCZTcvF6J4LEYpvvNmknpmGcQ4=; b=y39qffnqAI7Y0DiKurLix2iQ6h/sas5jGbVV95ZjlX3AicyZ0oOoo8wlJlEvyJU8rl wBfNHKrehBZ0I2pp9dXHWLdug5L87YhyYb+IfHtP9NiDdNPGFg27HP18GM9Z8iGUlUaW Rw0z3G8xrhRBSJcRnuy3qxFtApurUSM5xrqqISRv6h7WAiJWsE/mmTH+IXXEl4pzg1AC r7uV6Fin1dDJNegSYCmoaEnW44+NBbgp/ExOT37FC3Zm5obZFuFIViAkqRlBMu3oISuL zM5wRgwXRrUHJ2tqaRO22JmTrYuA3QmIYEhsu3z2lmAEI3/RdYDh4ouwmd0GdZO1PTsn 4nQQ== X-Gm-Message-State: AOAM533sGM3WCKSBjfJHRq4xZTnohYUGb3STly1gZNx1QnsUs/RD1qDn p+NNVlsm7dbU8lAM5AkhFUBW8A== X-Google-Smtp-Source: ABdhPJzCwHU4Op2qWLkszS0o2FCgyTE9JNDodLZ/xEayw3EgbBpqJvIQkE4GhbjDewxPDCglnhaWjg== X-Received: by 2002:a05:622a:15ce:: with SMTP id d14mr8711515qty.330.1644821536764; Sun, 13 Feb 2022 22:52:16 -0800 (PST) Received: from ripple.attlocal.net (172-10-233-147.lightspeed.sntcca.sbcglobal.net. [172.10.233.147]) by smtp.gmail.com with ESMTPSA id bl34sm12969452qkb.15.2022.02.13.22.52.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 13 Feb 2022 22:52:15 -0800 (PST) Date: Sun, 13 Feb 2022 22:52:13 -0800 (PST) From: Hugh Dickins X-X-Sender: hugh@ripple.anvils To: Vlastimil Babka cc: Hugh Dickins , Andrew Morton , Michal Hocko , "Kirill A. Shutemov" , Matthew Wilcox , David Hildenbrand , Alistair Popple , Johannes Weiner , Rik van Riel , Suren Baghdasaryan , Yu Zhao , Greg Thelen , Shakeel Butt , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH 01/13] mm/munlock: delete page_mlock() and all its works In-Reply-To: <957e2ea6-d01e-256f-51a0-d927a93b50a5@suse.cz> Message-ID: References: <8e4356d-9622-a7f0-b2c-f116b5f2efea@google.com> <5ed1f01-3e7e-7e26-cc1-2b7a574e2147@google.com> <4a5bc989-e59a-d421-faf4-8156f700ec99@suse.cz> <957e2ea6-d01e-256f-51a0-d927a93b50a5@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Stat-Signature: fh8t1xt3jc4imfmmc84k9wfuy3hsgwrj X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: A5A4D40002 Authentication-Results: imf07.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=BeUxMuEe; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf07.hostedemail.com: domain of hughd@google.com designates 209.85.160.172 as permitted sender) smtp.mailfrom=hughd@google.com X-Rspam-User: X-HE-Tag: 1644821537-112898 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, 10 Feb 2022, Vlastimil Babka wrote: > On 2/9/22 23:28, Hugh Dickins wrote: > > > > Though taking this apart and putting it back together brings its own > > dangers. That second suggestion probably won't fly very well, with > > 06/13 using mlock_count only while on the unevictable. I'm not sure > > how much rethinking the bisection possibility deserves. > > Right, if it's impractical to change for a potential and hopefully unlikely > bad bisection luck, just a note at the end of each patch's changelog > mentioning what temporarily doesn't work, should be enough. I'm adding a paragraph on that to the end of this 01/13's changelog. If you or akpm think it's better duplicated in 02, 03, 04, 05, 06 then I think it will be easiest if Andrew edits it into them, rather than me updating them one by one. > > As to VM_LOCKED: yes, you're right, at this stage of the series the > > munlock really ought to be clearing VM_LOCKED (even while it doesn't > > go on to do anything about the pages), as it claims in the comment above. > > I removed this line at a later stage (07/13), when changing it to > > mlock_vma_pages_range() serving both mlock and munlock according to > > whether VM_LOCKED is provided - and mistakenly folded back that deletion > > to this patch. End result the same, but better to restore that maskout > > in this patch, as you suggest. > > Great, thanks. That restores any effect on VM_LOCKONFAULT in any case as well. Yes, it turned out to be a mistake in my rebasing the series, the original had the VM_LOCKED_CLEAR_MASK masking still there (up until the interface changes in 07/13). Thanks for spotting that. Vlastimil, many thanks for your valiant effort reviewing this series: it's not at all easy, and I much appreciate the time you've put into it. I'm now going to send out v2 updates to 01, 04, 07, 10, 11 (only), still based on 5.17-rc2, but incorporating your and others' suggestions. Hugh