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 40990C5474D for ; Thu, 29 Aug 2024 03:39:24 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 91A276B00B3; Wed, 28 Aug 2024 23:39:23 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 8C9196B00B5; Wed, 28 Aug 2024 23:39:23 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 7B7FA6B00CA; Wed, 28 Aug 2024 23:39:23 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 5E1076B00B3 for ; Wed, 28 Aug 2024 23:39:23 -0400 (EDT) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id E8489C02ED for ; Thu, 29 Aug 2024 03:39:22 +0000 (UTC) X-FDA: 82503877764.07.E84EA3A Received: from out-178.mta0.migadu.com (out-178.mta0.migadu.com [91.218.175.178]) by imf03.hostedemail.com (Postfix) with ESMTP id 01A102001F for ; Thu, 29 Aug 2024 03:39:20 +0000 (UTC) Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b="RKYCS/bs"; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf03.hostedemail.com: domain of muchun.song@linux.dev designates 91.218.175.178 as permitted sender) smtp.mailfrom=muchun.song@linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1724902662; 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=n+Lh2vHxUB+86yeUS0d5uwMJVWzCawU4cjiNowF9hs4=; b=7PxwG/oNYVG161AfolI8IeuUs1XfkHTx+ubuhS9I8/m/DTNojyZi8vyDTO/bssawvmrHnc lhlNvH3xhUyoPB1Rx6/VBrFXmRwWyt93y00awjpJWYYX4bP5TbHHdDNEkkpAE9NinZLRnB qFD8+SI4/u7FvO66voV9bV9ScLnN1+o= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1724902662; a=rsa-sha256; cv=none; b=QI7+BKDeflFeV9KpmneeIg0s/daaUxxE9YDsneAS6V9w6pFjRSaQdM/kULuRZwf8Hd1VNs XLb38nPlH3lUT7xhJgTtb9S3L/apP/Mb3bgyK/MiP159Nna3hy8nrEmi8tFoxDz2CRpoLc INh02FJsjesPTwWmAbdh6F/9O3nPbN0= ARC-Authentication-Results: i=1; imf03.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b="RKYCS/bs"; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf03.hostedemail.com: domain of muchun.song@linux.dev designates 91.218.175.178 as permitted sender) smtp.mailfrom=muchun.song@linux.dev Message-ID: <7c2237bd-c1eb-4cf6-9135-420a321ebd3b@linux.dev> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1724902758; h=from:from: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; bh=n+Lh2vHxUB+86yeUS0d5uwMJVWzCawU4cjiNowF9hs4=; b=RKYCS/bs9Fuf2a0LIc6oZEc0Qxnsqv4A/6jg2vqw9A1X/vYr6M0gINoQ8cYiZo68t/9AW/ 7hTk0h0Q3vTIPdKJW1u/PDgjvwFtRiTbiKd6SkCPS50ico497y8yK/zh+jlcuEuMRbqZOL GN5HEvWBmr2PiMol50Z18Xh9PnBmcMA= Date: Thu, 29 Aug 2024 11:39:09 +0800 MIME-Version: 1.0 Subject: Re: [PATCH v2 02/14] arm: adjust_pte() use pte_offset_map_rw_nolock() To: David Hildenbrand , Qi Zheng Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-arm-kernel@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, hughd@google.com, willy@infradead.org, vbabka@kernel.org, akpm@linux-foundation.org, rppt@kernel.org, vishal.moola@gmail.com, peterx@redhat.com, ryan.roberts@arm.com, christophe.leroy2@cs-soprasteria.com References: <7915acf5887e7bf0c5cc71ff30ad2fe8447d005d.1724310149.git.zhengqi.arch@bytedance.com> <77f3f5ce-dd12-4ca2-90cd-0f2226fe26ba@redhat.com> X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Muchun Song In-Reply-To: <77f3f5ce-dd12-4ca2-90cd-0f2226fe26ba@redhat.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Migadu-Flow: FLOW_OUT X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 01A102001F X-Stat-Signature: czt6zbzi1h48kuge95cxgo4kp6ifoya6 X-Rspam-User: X-HE-Tag: 1724902760-439977 X-HE-Meta: U2FsdGVkX189AU5HSjTbiAendWq5PtvnBO0iwPrrGLujpdAz6GxB6a/4do1nuSnGBS76c1Jz6HwfIzA3M2EVnwt568qQMw6gAJCrW8YcXBqauPpKnXIUsCHwHa8meXoOdy0Op2w+uwiS/FrzwH9ze4wbNt0T80MrCaQikuxa9RQHAqrTivSI7723rl76g743PanGl6jAvALFQx+2d9VgZ25U7l6OFijLqQ01xewOVkHCIzUmSYx7mYma0vw4f+v/6orWM782OnpNDKm1LmfUx7dMWf0dTAdxGYGFgIDWNwH1juV5mCNew123DqtyIKmkrOWDCvahioiONseHpHxKyiKLmmvqMgTdqDDf87TYiA4/vI0CM5RuFLvcsqbo8k9swKgjwEk5mi2/EgvBNrhWbc6s/Pov+gg6PyDiLv2uSJx7p6/o6JQUncn96YoeOPJTBRlrBM3/AYdekuFdRiQSDlZ+gx6lNJu+h1yJDJPcW78MMp6I1EG1c492OxH1mRFnSCRJYOSLfC9UArebtl8AMxLkz2AICV+q2LOb8VWNpemNji04FKGuO74Dp5UM/b4rBdIzqx+3hn7rOaLW4jIYc8ATgMn3NOs7CkfVK0+cGq1j/89qCVYZvBF0Bw0jS6DzXg9r2G27s2rNG+sJ/j9ka9yFArLvNoFqUggzIBHaUrbWyL2hbq5zXVFNJg+YXsXPZncajUuQcxLaCVx+cg/EfXD1aWj0DRztKtfzHLCmiTZZUs1wh+t+Fhi4UMc7MEWm91mm1/3kdE9pUbwwcdahqmO804xql0dapygEmgo06OhAgYFW4gYnKLGkIzElgHzw0g//CEgzi/06tuhoUVakpUuoIu0C/Y9PlSVHjjmzyQh4dcg5r21VOhHsfkCT8C2gvxHZLAG3209MEEgCgHf9jruam/usZTnSFLCGJoeW0rLvMNeWXSfOtzNYjDyJNqVTTolzZVQJ6Q+REBCjlsZ 22Dlgv2h LUGttRdaDMFws0Cx5BrUxC089i6zocDx67QS9Sc38h20DqSPkf73y4UD5hxLCikYJDXvpW37vFO9UR7SI/hsMOLwlHTlJpAnsWdH7950wYeLrTI498DmIAYmneI4hi5q8qBxzHq1xth+ff2OGzbT2XLSRQlau37t79ZM/p4bwQgd23LcLIrlKcxMCgMQR2hYjKGQ9SezpeJFwFh4lthadXDZfp/qZLMvtV2ix7htfg6LXn9EQF9YRgQY0NrN2PyMa79+k2LimbMT6RZHoJyqfzBsGlzMW+03WpAT6HgZKcaPBhIHltKJLM1nVqKlD9iJBqY1i+QabyU3g0VtSWSU0zGt7/9a8nDvT4WPMfYmv5T1M6O5ogl+9TBg6FSZ/bv6or+Pd 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: List-Subscribe: List-Unsubscribe: On 2024/8/26 23:26, David Hildenbrand wrote: > On 22.08.24 09:13, Qi Zheng wrote: >> In do_adjust_pte(), we may modify the pte entry. At this time, the write >> lock of mmap_lock is not held, and the pte_same() check is not performed >> after the PTL held. The corresponding pmd entry may have been modified >> concurrently. Therefore, in order to ensure the stability if pmd entry, >> use pte_offset_map_rw_nolock() to replace pte_offset_map_nolock(), >> and do >> pmd_same() check after holding the PTL. >> >> Signed-off-by: Qi Zheng Reviewed-by: Muchun Song >> --- >>   arch/arm/mm/fault-armv.c | 9 ++++++++- >>   1 file changed, 8 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/mm/fault-armv.c b/arch/arm/mm/fault-armv.c >> index 831793cd6ff94..de6c7d8a2ddfc 100644 >> --- a/arch/arm/mm/fault-armv.c >> +++ b/arch/arm/mm/fault-armv.c >> @@ -94,6 +94,7 @@ static int adjust_pte(struct vm_area_struct *vma, >> unsigned long address, >>       pud_t *pud; >>       pmd_t *pmd; >>       pte_t *pte; >> +    pmd_t pmdval; >>       int ret; >>         pgd = pgd_offset(vma->vm_mm, address); >> @@ -112,16 +113,22 @@ static int adjust_pte(struct vm_area_struct >> *vma, unsigned long address, >>       if (pmd_none_or_clear_bad(pmd)) >>           return 0; >>   +again: >>       /* >>        * This is called while another page table is mapped, so we >>        * must use the nested version.  This also means we need to >>        * open-code the spin-locking. >>        */ >> -    pte = pte_offset_map_nolock(vma->vm_mm, pmd, address, &ptl); >> +    pte = pte_offset_map_rw_nolock(vma->vm_mm, pmd, address, >> &pmdval, &ptl); >>       if (!pte) >>           return 0; >>         do_pte_lock(ptl); >> +    if (unlikely(!pmd_same(pmdval, pmdp_get_lockless(pmd)))) { >> +        do_pte_unlock(ptl); >> +        pte_unmap(pte); >> +        goto again; >> +    } >>         ret = do_adjust_pte(vma, address, pfn, pte); > > Looks correct to me, but I wonder why the missing pmd_same check is > not an issue so far ... any experts? THP on __LINUX_ARM_ARCH__ < 6 is > not really used/possible? I think it is because it does not support THP. TRANSPARENT_HUGEPAGE depends on HAVE_ARCH_TRANSPARENT_HUGEPAGE which depends on ARM_LPAE. However, the Kconfig says ARM_LPAE is only supported on ARMv7 processor. config ARM_LPAE          bool "Support for the Large Physical Address Extension"          depends on MMU && CPU_32v7 && !CPU_32v6 && !CPU_32v5 && \                  !CPU_32v4 && !CPU_32v3          select PHYS_ADDR_T_64BIT          select SWIOTLB          help            Say Y if you have an ARMv7 processor supporting the LPAE page            table format and you would like to access memory beyond the            4GB limit. The resulting kernel image will not run on            processors without the LPA extension.            If unsure, say N. Thanks. > > Acked-by: David Hildenbrand >