Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   hanchuanhua <hanchuanhua@oppo.com>
To:     paul.walmsley@sifive.com, palmer@dabbelt.com, aou@eecs.berkeley.edu
Cc:     jszhang@kernel.org, alexandre.ghiti@canonical.com,
        wangkefeng.wang@huawei.com, liushixin2@huawei.com,
        zhengqi.arch@bytedance.com, ebiederm@xmission.com,
        linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org,
        hanchuanhua <hanchuanhua@oppo.com>
Subject: [PATCH v2] riscv: mm: Remove the copy operation of pmd
Date:   Mon, 28 Feb 2022 20:34:07 +0800
Message-Id: <20220228123407.939-1-hanchuanhua@oppo.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?hrCpcUY2+6nn4hD1XnZBAhGIe83ZSMZkqJB7Blo7uy0Zst5CbJL4AZdbJLGx?=
 =?us-ascii?Q?PwU0hEgqhuguFA+DQk4PgPHXnlffBS6t8U8GRgmvbMthgSrwVYB0q92Jvn/k?=
 =?us-ascii?Q?rIBu+81v14VK0HNBKrEr72uvX1vwg8LJKkiPZm5ZtYxc4W4a8FCQf4fr5mSg?=
 =?us-ascii?Q?KrsI6YGTggjBpbXEtW5JYvFxYvh/aUd7j/K8kwKRwgJFwm3xd1NF1npiD3gM?=
 =?us-ascii?Q?wEl9FiXOnV6lKGKP9jBHWL3NXJBplIhjecoIS2c1ZGAQYgZBhNqy7e3+iUpt?=
 =?us-ascii?Q?qs9e4Mas0ZP46UAlcNX/EERmoK/CIp6u61pAzAAiQU4cbPC5TJlpJ64ftNr6?=
 =?us-ascii?Q?R/ZKGWROqXDPl7Da0Eu1RbDkF/D3yQgkH73qvstr1eaAvvWPRs0LcFwcsthd?=
 =?us-ascii?Q?fyPUiAzmygiUM/ZsAvYQ9/MQazqMK75p0aK/4HFXcTMVo9UdPQK1sO+25WJf?=
 =?us-ascii?Q?QbZ/0awd07pHhLWHToOkh9fmzvD2wTrlknb+vHPbYpDduwjVgGReEZjV7DvP?=
 =?us-ascii?Q?XKrN4upz5tNPG72+w3/FC+vAKWX3hHhGvOWI8RW8f9eVPf2fSKQbZEeYObp9?=
 =?us-ascii?Q?31zheZOrT/z5yRyBXAIiB4zmJ0Jt0ApHhvJq2WPGcmUg1tgg9N99fs7ohm1D?=
 =?us-ascii?Q?RNO8BFAsqTskvEk/UF22vRwH6OtP1C+AkfGGbU3EAoDEFO9kbALIaWEvbIIS?=
 =?us-ascii?Q?IbeqKJ1d6rpCkeU/zER7a62HDdujev1hZqJjtawHQCTFCFOAAgt83OOLNqAM?=
 =?us-ascii?Q?FKA05SsEViztmHU7KnPzyYGITIAJw6EpbFRzgoiuEz9g+zDi621xcBTdoSCM?=
 =?us-ascii?Q?1KfZh7yUtpv3dtPplm6btFJkvRpIghig69hKFQrZzFDIBLA8JF2UtPZDRnrI?=
 =?us-ascii?Q?gvU3vnFxzRaHywCfDL9MwkhDXtNAyn2KUceY7eHQng+oNg3qewVmasC+0tL4?=
 =?us-ascii?Q?EArQ3ELFmzPbgHAmy/XJfnl9D+aZk2qq9fP36z7FB2zRTJxwhK/y2q+7OIUs?=
 =?us-ascii?Q?seeQBoECJE4ADIRARXxUATV8zUM2lS22K5HjB87UeV4VOlU+rxDtF/SPLTQa?=
 =?us-ascii?Q?LsJyeF3x2KpsLUC2TDvJ6rtn7oF5CSC8ghMgk+0tuFZXQG/d7sPQNIerx96C?=
 =?us-ascii?Q?mVca7Ynh4cszgYaZ34yX+ue+en5QK1eeCRxAFq1o76hllQRpk1wIPwx8J7gq?=
 =?us-ascii?Q?cDbZNVtcG9Sr71L4A1Od73yTfJ59KZCjlq5hHoQuODigYkA2Uh9K5HyFUo1l?=
 =?us-ascii?Q?6lzcHov21pdkRKQGl4mT1ZK4Yzu3GQZ9QL6AdWKS8y4qNcoKcCqkRpIPW8t4?=
 =?us-ascii?Q?y5SLNEpPY1BJaYDPqp8Jpq+4Z9sIXN1SQuXfShcAeMbs7rv3Kg9iHTdSXqiz?=
 =?us-ascii?Q?LaTCV8QqodLZNgLWyTM3J5gjy28gKxJpliwcjnz7kzA5pB4i3mZY62gd1+cc?=
 =?us-ascii?Q?WT9MInhY4UtC/dQywtZv/NuEKM3ueX/hyWwplRgc21KkagSuMAFuQ6vGGjrK?=
 =?us-ascii?Q?q6s158YDJndt8SoFz5fNyoJrVUzUVPCC30tZR6XiOPlmfQXiyWoNm1dkJ3OY?=
 =?us-ascii?Q?TMWNN9X+iL/7UxuzPVaCaXgEKnhuWktmwYMOIPw4zj1RXDRGX6UAG3PmHIMs?=
 =?us-ascii?Q?E8I022fATbc7679s7Bvef7c=3D?=
X-OriginatorOrg: oppo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e89c3feb-d97c-4fef-fda1-08d9fab6b50f
X-MS-Exchange-CrossTenant-AuthSource: SG2PR02MB5379.apcprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 12:34:47.9445
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f1905eb1-c353-41c5-9516-62b4a54b5ee6
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wXYrD1WSogPUqYG+xlQnYAFCYT8ekjPvAjr2W8iMoiwWcYGeTfQj5qouDZbsFJJ+KdLjqWod4M3a/Q/WPS8nSg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SG2PR02MB3532
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Since all processes share the kernel address space,
we only need to copy pgd in case of a vmalloc page
fault exception, the other levels of page tables are
shared, so the operation of copying pmd is unnecessary.

Signed-off-by: hanchuanhua <hanchuanhua@oppo.com>
---
Changes in v2:
  -Delete incorrect default signatures.

 arch/riscv/mm/fault.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/arch/riscv/mm/fault.c b/arch/riscv/mm/fault.c
index 4e9efbe..40694f0 100644
--- a/arch/riscv/mm/fault.c
+++ b/arch/riscv/mm/fault.c
@@ -102,9 +102,9 @@ static inline void bad_area(struct pt_regs *regs, struct mm_struct *mm, int code
 static inline void vmalloc_fault(struct pt_regs *regs, int code, unsigned long addr)
 {
 	pgd_t *pgd, *pgd_k;
-	pud_t *pud, *pud_k;
-	p4d_t *p4d, *p4d_k;
-	pmd_t *pmd, *pmd_k;
+	pud_t *pud_k;
+	p4d_t *p4d_k;
+	pmd_t *pmd_k;
 	pte_t *pte_k;
 	int index;
 	unsigned long pfn;
@@ -132,14 +132,12 @@ static inline void vmalloc_fault(struct pt_regs *regs, int code, unsigned long a
 	}
 	set_pgd(pgd, *pgd_k);
 
-	p4d = p4d_offset(pgd, addr);
 	p4d_k = p4d_offset(pgd_k, addr);
 	if (!p4d_present(*p4d_k)) {
 		no_context(regs, addr);
 		return;
 	}
 
-	pud = pud_offset(p4d, addr);
 	pud_k = pud_offset(p4d_k, addr);
 	if (!pud_present(*pud_k)) {
 		no_context(regs, addr);
@@ -150,13 +148,11 @@ static inline void vmalloc_fault(struct pt_regs *regs, int code, unsigned long a
 	 * Since the vmalloc area is global, it is unnecessary
 	 * to copy individual PTEs
 	 */
-	pmd = pmd_offset(pud, addr);
 	pmd_k = pmd_offset(pud_k, addr);
 	if (!pmd_present(*pmd_k)) {
 		no_context(regs, addr);
 		return;
 	}
-	set_pmd(pmd, *pmd_k);
 
 	/*
 	 * Make sure the actual PTE exists as well to
-- 
2.7.4