Michal Suchanek 607722
From 50e0cf3742a01e72f4ea4a8fe9221b152e22871b Mon Sep 17 00:00:00 2001
Michal Suchanek 607722
From: Nathan Lynch <nathanl@linux.ibm.com>
Michal Suchanek 607722
Date: Fri, 12 Jun 2020 00:12:26 -0500
Michal Suchanek 607722
Subject: [PATCH] powerpc/numa: remove unreachable topology timer code
Michal Suchanek 607722
Michal Suchanek 607722
References: bsc#1181674 ltc#189159
Michal Suchanek 607722
Patch-mainline: v5.9-rc1
Michal Suchanek 607722
Git-commit: 50e0cf3742a01e72f4ea4a8fe9221b152e22871b
Michal Suchanek 607722
Michal Suchanek 607722
Since vphn_enabled is always 0, we can stub out
Michal Suchanek 607722
timed_topology_update() and remove the code which becomes unreachable.
Michal Suchanek 607722
Michal Suchanek 607722
Signed-off-by: Nathan Lynch <nathanl@linux.ibm.com>
Michal Suchanek 607722
Reviewed-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Michal Suchanek 607722
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Michal Suchanek 607722
Link: https://lore.kernel.org/r/20200612051238.1007764-7-nathanl@linux.ibm.com
Michal Suchanek 607722
Acked-by: Michal Suchanek <msuchanek@suse.de>
Michal Suchanek 607722
---
Michal Suchanek 607722
 arch/powerpc/mm/numa.c | 21 ---------------------
Michal Suchanek 607722
 1 file changed, 21 deletions(-)
Michal Suchanek 607722
Michal Suchanek 607722
diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c
Michal Suchanek 607722
--- a/arch/powerpc/mm/numa.c
Michal Suchanek 607722
+++ b/arch/powerpc/mm/numa.c
Michal Suchanek 607722
@@ -1129,13 +1129,9 @@ struct topology_update_data {
Michal Suchanek 607722
 	int new_nid;
Michal Suchanek 607722
 };
Michal Suchanek 607722
 
Michal Suchanek 607722
-#define TOPOLOGY_DEF_TIMER_SECS	60
Michal Suchanek 607722
-
Michal Suchanek 607722
 static cpumask_t cpu_associativity_changes_mask;
Michal Suchanek 607722
 static const int vphn_enabled;
Michal Suchanek 607722
 static const int prrn_enabled;
Michal Suchanek 607722
-static void reset_topology_timer(void);
Michal Suchanek 607722
-static int topology_timer_secs = 1;
Michal Suchanek 607722
 static int topology_inited;
Michal Suchanek 607722
 
Michal Suchanek 607722
 static long hcall_vphn(unsigned long cpu, u64 flags, __be32 *associativity)
Michal Suchanek 607722
@@ -1143,15 +1139,6 @@ static int topology_inited;
Michal Suchanek 607722
  */
Michal Suchanek 607722
 int timed_topology_update(int nsecs)
Michal Suchanek 607722
 {
Michal Suchanek 607722
-	if (vphn_enabled) {
Michal Suchanek 607722
-		if (nsecs > 0)
Michal Suchanek 607722
-			topology_timer_secs = nsecs;
Michal Suchanek 607722
-		else
Michal Suchanek 607722
-			topology_timer_secs = TOPOLOGY_DEF_TIMER_SECS;
Michal Suchanek 607722
-
Michal Suchanek 607722
-		reset_topology_timer();
Michal Suchanek 607722
-	}
Michal Suchanek 607722
-
Michal Suchanek 607722
 	return 0;
Michal Suchanek 607722
 }
Michal Suchanek 607722
 
Michal Suchanek 607722
@@ -1438,16 +1425,6 @@ static void topology_schedule_update(void)
Michal Suchanek 607722
 	schedule_work(&topology_work);
Michal Suchanek 607722
 }
Michal Suchanek 607722
 
Michal Suchanek 607722
-static struct timer_list topology_timer =
Michal Suchanek 607722
-	TIMER_INITIALIZER(topology_timer_fn, 0, 0);
Michal Suchanek 607722
-
Michal Suchanek 607722
-static void reset_topology_timer(void)
Michal Suchanek 607722
-{
Michal Suchanek 607722
-	topology_timer.data = 0;
Michal Suchanek 607722
-	topology_timer.expires = jiffies + topology_timer_secs * HZ;
Michal Suchanek 607722
-	mod_timer(&topology_timer, topology_timer.expires);
Michal Suchanek 607722
-}
Michal Suchanek 607722
-
Michal Suchanek 607722
 /*
Michal Suchanek 607722
  * Start polling for associativity changes.
Michal Suchanek 607722
  */
Michal Suchanek 607722
-- 
Michal Suchanek 607722
2.26.2
Michal Suchanek 607722