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 EA548C52D11 for ; Thu, 26 Jan 2023 17:32:51 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4B2486B0072; Thu, 26 Jan 2023 12:32:51 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 4623A8E0002; Thu, 26 Jan 2023 12:32:51 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 329D68E0001; Thu, 26 Jan 2023 12:32:51 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 22ED56B0072 for ; Thu, 26 Jan 2023 12:32:51 -0500 (EST) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id EF15C120E39 for ; Thu, 26 Jan 2023 17:32:50 +0000 (UTC) X-FDA: 80397645300.12.1899B26 Received: from outbound-smtp58.blacknight.com (outbound-smtp58.blacknight.com [46.22.136.242]) by imf05.hostedemail.com (Postfix) with ESMTP id 0A350100011 for ; Thu, 26 Jan 2023 17:32:48 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf05.hostedemail.com: domain of mgorman@techsingularity.net designates 46.22.136.242 as permitted sender) smtp.mailfrom=mgorman@techsingularity.net ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1674754369; 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: in-reply-to:in-reply-to:references:references; bh=Qfe4Syc1bFiLpb2uUtxfuYWcGPoen6y+NH/BvG/7Z28=; b=dxaKgqhbxwm2ZxLTxoZwKWe1SobhIM08P8zsNl4eUCpdZ8rpSwVUXZToyhpurHHYjP2T3m og7cXbWXRFOIh/7fSMQYZmqBh5WXRsr+EImunaY7PyZbvby+217b47VEGCKF+QOwE32syG SgD+nvE3jCqNXAwVr/izlTZPmFOJURQ= ARC-Authentication-Results: i=1; imf05.hostedemail.com; dkim=none; dmarc=none; spf=pass (imf05.hostedemail.com: domain of mgorman@techsingularity.net designates 46.22.136.242 as permitted sender) smtp.mailfrom=mgorman@techsingularity.net ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1674754369; a=rsa-sha256; cv=none; b=r3fytKu5fluRIN8uo+OqkxuvzgtXDTfCmhdXoXp1M1dcg3TR7PYVa4P+rZ6/a3g3XhQhhC 2I/G5JM2qDKFtTCc/W1uHpW0AJ/EkDRfF8ImgPmvEFSQRd7ZBzsF1D+tFWP+YIwBIbhzOE VqxvqMAw0yooyaBJ4DQTY0SowDpB1Ic= Received: from mail.blacknight.com (pemlinmail04.blacknight.ie [81.17.254.17]) by outbound-smtp58.blacknight.com (Postfix) with ESMTPS id 38643FA7DA for ; Thu, 26 Jan 2023 17:32:47 +0000 (GMT) Received: (qmail 32726 invoked from network); 26 Jan 2023 17:32:46 -0000 Received: from unknown (HELO techsingularity.net) (mgorman@techsingularity.net@[84.203.198.246]) by 81.17.254.9 with ESMTPSA (AES256-SHA encrypted, authenticated); 26 Jan 2023 17:32:46 -0000 Date: Thu, 26 Jan 2023 17:32:45 +0000 From: Mel Gorman To: Suren Baghdasaryan Cc: akpm@linux-foundation.org, michel@lespinasse.org, jglisse@google.com, mhocko@suse.com, vbabka@suse.cz, hannes@cmpxchg.org, dave@stgolabs.net, willy@infradead.org, liam.howlett@oracle.com, peterz@infradead.org, ldufour@linux.ibm.com, paulmck@kernel.org, mingo@redhat.com, will@kernel.org, luto@kernel.org, songliubraving@fb.com, peterx@redhat.com, david@redhat.com, dhowells@redhat.com, hughd@google.com, bigeasy@linutronix.de, kent.overstreet@linux.dev, punit.agrawal@bytedance.com, lstoakes@gmail.com, peterjung1337@gmail.com, rientjes@google.com, axelrasmussen@google.com, joelaf@google.com, minchan@google.com, jannh@google.com, shakeelb@google.com, tatashin@google.com, edumazet@google.com, gthelen@google.com, gurua@google.com, arjunroy@google.com, soheil@google.com, hughlynch@google.com, leewalsh@google.com, posk@google.com, linux-mm@kvack.org, linux-arm-kernel@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, x86@kernel.org, linux-kernel@vger.kernel.org, kernel-team@android.com Subject: Re: [PATCH v3 6/7] mm: introduce mod_vm_flags_nolock and use it in untrack_pfn Message-ID: <20230126173245.cf3jcfw5s2a77s5v@techsingularity.net> References: <20230125233554.153109-1-surenb@google.com> <20230125233554.153109-7-surenb@google.com> <20230126154740.j3a3lu4x557c56yi@techsingularity.net> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 0A350100011 X-Rspam-User: X-Stat-Signature: n83ximycia6k397boxe1tn5ufkwpxuab X-HE-Tag: 1674754368-147256 X-HE-Meta: U2FsdGVkX1+Sfy3aIwwM1/QZDLEf4WSkBCgCRkuEJzqQC8jLlUF7+Azy+S7ICGqAYKBu5RalUWxffnka5NYtEFDzesOBY7ZD/27RBIaHXAd5c1Kj44sLkSjBmdiRvRNMyZciWoS9x8zyF5xZRcmRsKRE9U/DMzPf5qSzuMJiQh5peR1FLv8AFQ7ysO24B3IPxWkz/AaXIVNxDuDJQiyeDiW6a8zOUBtomNeJbTvYbQW7z/QXFZ5U0dk9C9eglK8f7XG/ftlsBb2aiwpgtli0QeL8c2I+Zu43fnrimffKuPz5vgCSPBbxIQKrfVN5DlXz9s++XIzEmc3iZcqRRAnTQTR1Chbj8cYMxDgEeLa2XACXihe+JG/hDPcXaVBez/MU1OXsP7gsuufNR7pT+3JLSJP6awoomAI6A8Ca+uRsqfBVyiLJnfePDPJe26bE3Mr0LfIN59FmEXYLOtIWOnzgDuw16JxjlykjJD954PLSllbza6xVVtfYYn/8GyVQPs7LEi4Xq1O0vS1fIc60Qzp7VkMO8s0mIukYVFmGlyG1hOi5aajoJ+6B/huDO7SwYzU3fDlFG9+A8ewMcp4Hpr58MCc5VjepLUKq8U2F4me7tJiItEtIokKOTgS+hOlW/KY38gS3jXAMIgxot9g16Fvm4V6pRrhxOAkdnKyvzTSYYwEGoU1ZX5FTf2LHnDbivHgy3OS0KBFYALdnscFjqQOUpNxdGIilgHfdWQj0hif7sDfjPrHu3AAjAwygpJG127GvYfSx819SaQ31ZI+ZoClQHtUD+nIo8k3L8CLpPD5RgSPhoQFUGH5xFt4lPvj8zAuEmIiR8P5mDIHVtBomX9IPSxNAccxJ9rluvbQo3gu/HVHwFSsWH1/9ZAf14YEqRjup24p/IHIP01PjJMzP/fLcm5bhGk740nqBz+VRQQniwJXZ3vju3cn5jpHAK0DlawDAhXVqg42ALvGDfn/JNBX l5vy9ccC l4vItRFfzH0K0Dp1ajZbwE4tM4590Pq1CgADaL/zqLed9o7Qf80UFiOltQdJ+ZfHPq64bl3NyMH2qXYECPHkmhq0Hl3hKDkOP3c9/vfTJiCCDb1els3YNwELWDR7MQ43mHIX4+gxLGvNoHRhlK058AlzBSdqwhqVsJMsjDCMBuP7Rggi4tQ0DhvX5TFOFA7C6B1AhwXK9ojmwcvHg9zcMHdxMN/6r+pZ32Nu9CZv8WCNOuU//0y+5Ns2P/lJAYSfYSv7xmsj6TdAqkL2SdOHBzhrEoRnJsqqhDpGHfR7kPsEOOFxonZADOiA3zProb/+m6bZP 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, Jan 26, 2023 at 08:18:31AM -0800, Suren Baghdasaryan wrote: > On Thu, Jan 26, 2023 at 7:47 AM Mel Gorman wrote: > > > > On Wed, Jan 25, 2023 at 03:35:53PM -0800, Suren Baghdasaryan wrote: > > > In cases when VMA flags are modified after VMA was isolated and mmap_lock > > > was downgraded, flags modifications would result in an assertion because > > > mmap write lock is not held. > > > > Add note that it's also used during exit when the locking of the VMAs > > becomes irrelevant (mm users is 0, should be no VMA modifications taking > > place other than zap). > > Ack. > > > > > The typical naming pattern when a caller either knows it holds the necessary > > lock or knows it does not matter is __mod_vm_flags() > > Ok. It sounds less explicit but plenty of examples, so I'm fine with > such rename. Will apply in the next version. > It might be a personal thing. nolock to me is ambigious because it might mean "lock is already held", "no lock is necessary" or "no lock is acquired" where as *for me*, calling foo vs __foo *usually* means "direct callers of __foo take care of the locking, memory ordering, per-cpu pinning details etc" depending on the context. Of course, this convention is not universally true. > > > Pass a hint to untrack_pfn to conditionally use mod_vm_flags_nolock for > > > flags modification and to avoid assertion. > > > > > > Signed-off-by: Suren Baghdasaryan > > > > Patch itself looks ok. It strays close to being "conditional locking" > > though which might attract some complaints. > > The description seems to accurately describe what's done here but I'm > open to better suggestions. I don't have alternative suggestions but if someone else reads the patch and says "this is conditional locking", you can at least claim that someone else considered "conditional locking" and didn't think it was a major problem in this specific patch. -- Mel Gorman SUSE Labs