powerpc/tau: Remove duplicated set_thresholds() call
authorFinn Thain <fthain@telegraphics.com.au>
Fri, 4 Sep 2020 23:02:20 +0000 (09:02 +1000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 29 Oct 2020 08:07:09 +0000 (09:07 +0100)
[ Upstream commit 420ab2bc7544d978a5d0762ee736412fe9c796ab ]

The commentary at the call site seems to disagree with the code. The
conditional prevents calling set_thresholds() via the exception handler,
which appears to crash. Perhaps that's because it immediately triggers
another TAU exception. Anyway, calling set_thresholds() from TAUupdate()
is redundant because tau_timeout() does so.

Fixes: 1da177e4c3f41 ("Linux-2.6.12-rc2")
Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Tested-by: Stan Johnson <userm57@yahoo.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/d7c7ee33232cf72a6a6bbb6ef05838b2e2b113c0.1599260540.git.fthain@telegraphics.com.au
Signed-off-by: Sasha Levin <sashal@kernel.org>
arch/powerpc/kernel/tau_6xx.c

index 64a27b20cf55ef71e6222606dd5e2a990ef21bab..9e8b709a2aae4e44aaa72c685e917b43b633d3cf 100644 (file)
@@ -108,11 +108,6 @@ void TAUupdate(int cpu)
 #ifdef DEBUG
        printk("grew = %d\n", tau[cpu].grew);
 #endif
-
-#ifndef CONFIG_TAU_INT /* tau_timeout will do this if not using interrupts */
-       set_thresholds(cpu);
-#endif
-
 }
 
 #ifdef CONFIG_TAU_INT