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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F1453C433F5 for ; Mon, 27 Sep 2021 02:16:23 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 727BB60F4F for ; Mon, 27 Sep 2021 02:16:23 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 727BB60F4F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=kvack.org Received: by kanga.kvack.org (Postfix) id 090B76B0071; Sun, 26 Sep 2021 22:16:23 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0403E6B0072; Sun, 26 Sep 2021 22:16:22 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E4AEC6B0073; Sun, 26 Sep 2021 22:16:22 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0096.hostedemail.com [216.40.44.96]) by kanga.kvack.org (Postfix) with ESMTP id D11EE6B0071 for ; Sun, 26 Sep 2021 22:16:22 -0400 (EDT) Received: from smtpin05.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id 8CE7222BEB for ; Mon, 27 Sep 2021 02:16:22 +0000 (UTC) X-FDA: 78631739004.05.AAED8F6 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf05.hostedemail.com (Postfix) with ESMTP id 003DF5065277 for ; Mon, 27 Sep 2021 02:16:21 +0000 (UTC) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id E0896D6E; Sun, 26 Sep 2021 19:16:20 -0700 (PDT) Received: from [10.163.73.35] (unknown [10.163.73.35]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 0E7C03F718; Sun, 26 Sep 2021 19:16:18 -0700 (PDT) Subject: Re: [PATCH V2] mm: debug_vm_pgtable: Don't use __P000 directly To: guoren@kernel.org, akpm@linux-foundation.org Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Guo Ren References: <20210924060821.1138281-1-guoren@kernel.org> From: Anshuman Khandual Message-ID: <30c10b5f-1b26-e0a8-8185-6fa3296d68dc@arm.com> Date: Mon, 27 Sep 2021 07:47:27 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210924060821.1138281-1-guoren@kernel.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 003DF5065277 X-Stat-Signature: ai94nwaxjixefrpur97pdzz4mckoine5 Authentication-Results: imf05.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf05.hostedemail.com: domain of anshuman.khandual@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=anshuman.khandual@arm.com X-Rspamd-Server: rspam06 X-HE-Tag: 1632708981-483761 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 9/24/21 11:38 AM, guoren@kernel.org wrote: > From: Guo Ren > > The __Pxxx/__Sxxx macros are only for protection_map[] init. All > usage of them in linux should come from protection_map array. > > Because a lot of architectures would re-initilize protection_map[] > array, eg: x86-mem_encrypt, m68k-motorola, mips, arm, sparc. > > Using __P000 is not rigorous. > > Signed-off-by: Guo Ren > Reviewed-by: Andrew Morton Reviewed-by: Anshuman Khandual > > --- > > Changes since V2: > - s/init protection_map[]/protection_map[] init/ > - s/Becasue/Because/ > - Remove unclear part > - Replace __P000 and __S000 with protection_map[0] and > protection_map[8] > --- > mm/debug_vm_pgtable.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/mm/debug_vm_pgtable.c b/mm/debug_vm_pgtable.c > index 1403639302e4..228e3954b90c 100644 > --- a/mm/debug_vm_pgtable.c > +++ b/mm/debug_vm_pgtable.c > @@ -1104,13 +1104,14 @@ static int __init init_args(struct pgtable_debug_args *args) > /* > * Initialize the debugging data. > * > - * __P000 (or even __S000) will help create page table entries with > - * PROT_NONE permission as required for pxx_protnone_tests(). > + * protection_map[0] (or even protection_map[8]) will help create > + * page table entries with PROT_NONE permission as required for > + * pxx_protnone_tests(). > */ > memset(args, 0, sizeof(*args)); > args->vaddr = get_random_vaddr(); > args->page_prot = vm_get_page_prot(VMFLAGS); > - args->page_prot_none = __P000; > + args->page_prot_none = protection_map[0]; > args->is_contiguous_page = false; > args->pud_pfn = ULONG_MAX; > args->pmd_pfn = ULONG_MAX; >