[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250427100442.958352-4-ye.liu@linux.dev>
Date: Sun, 27 Apr 2025 18:04:42 +0800
From: Ye Liu <ye.liu@...ux.dev>
To: akpm@...ux-foundation.org
Cc: linux-kernel@...r.kernel.org,
linux-mm@...ck.org,
rppt@...nel.org,
lorenzo.stoakes@...cle.com,
Liam.Howlett@...cle.com,
david@...hat.com,
harry.yoo@...cle.com,
riel@...riel.com,
vbabka@...e.cz,
liuye@...inos.cn,
ye.liu@...ux.dev
Subject: [PATCH 3/3] mm/numa: remove unnecessary local variable in alloc_node_data()
From: Ye Liu <liuye@...inos.cn>
The temporary local variable 'nd' is redundant. Directly assign the
virtual address to node_data[nid] to simplify the code.
No functional change.
Signed-off-by: Ye Liu <liuye@...inos.cn>
---
mm/numa.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/mm/numa.c b/mm/numa.c
index f1787d7713a6..7d5e06fe5bd4 100644
--- a/mm/numa.c
+++ b/mm/numa.c
@@ -13,7 +13,6 @@ void __init alloc_node_data(int nid)
{
const size_t nd_size = roundup(sizeof(pg_data_t), SMP_CACHE_BYTES);
u64 nd_pa;
- void *nd;
int tnid;
/* Allocate node data. Try node-local memory and then any node. */
@@ -21,7 +20,6 @@ void __init alloc_node_data(int nid)
if (!nd_pa)
panic("Cannot allocate %zu bytes for node %d data\n",
nd_size, nid);
- nd = __va(nd_pa);
/* report and initialize */
pr_info("NODE_DATA(%d) allocated [mem %#010Lx-%#010Lx]\n", nid,
@@ -30,7 +28,7 @@ void __init alloc_node_data(int nid)
if (tnid != nid)
pr_info(" NODE_DATA(%d) on node %d\n", nid, tnid);
- node_data[nid] = nd;
+ node_data[nid] = __va(nd_pa);
memset(NODE_DATA(nid), 0, sizeof(pg_data_t));
}
--
2.25.1
Powered by blists - more mailing lists