summary refs log tree commit diff
path: root/arch/x86
diff options
context:
space:
mode:
authorKim Phillips <kim.phillips@amd.com>2021-08-17 17:10:44 -0500
committerIngo Molnar <mingo@kernel.org>2021-08-26 09:14:36 +0200
commit6cf295b21608f9253037335f47cd0dfcce812d81 (patch)
tree3049a9519814750d6d11d93adbfdc29990f3953d /arch/x86
parentffec09f9c7d7b21b0aff29dd5c3972f4631c0b6b (diff)
downloadlinux-6cf295b21608f9253037335f47cd0dfcce812d81.tar.gz
perf/amd/uncore: Simplify code, use free_percpu()'s built-in check for NULL
free_percpu() has its own check for NULL, no need to open-code it.

Signed-off-by: Kim Phillips <kim.phillips@amd.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/20210817221048.88063-5-kim.phillips@amd.com
Diffstat (limited to 'arch/x86')
-rw-r--r--arch/x86/events/amd/uncore.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/arch/x86/events/amd/uncore.c b/arch/x86/events/amd/uncore.c
index 582c0ffb5e98..05bdb4cba300 100644
--- a/arch/x86/events/amd/uncore.c
+++ b/arch/x86/events/amd/uncore.c
@@ -659,11 +659,9 @@ fail_prep:
 fail_llc:
 	if (boot_cpu_has(X86_FEATURE_PERFCTR_NB))
 		perf_pmu_unregister(&amd_nb_pmu);
-	if (amd_uncore_llc)
-		free_percpu(amd_uncore_llc);
+	free_percpu(amd_uncore_llc);
 fail_nb:
-	if (amd_uncore_nb)
-		free_percpu(amd_uncore_nb);
+	free_percpu(amd_uncore_nb);
 
 	return ret;
 }