diff options
author | Igor Stoppa <igor.stoppa@gmail.com> | 2018-09-07 11:35:26 -0400 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2019-01-14 19:38:05 -0500 |
commit | 63da6caeb84cfad3d1e5774b7049dd1d2c9dec62 (patch) | |
tree | 6a129acf94fc9d19cc0d76b329205718fd58d204 | |
parent | 9bd10b649826774bb5e1e7fb67544e6500e3f005 (diff) |
powerpc: remove unnecessary unlikely()
WARN_ON() already contains an unlikely(), so it's not necessary to
wrap it into another.
Signed-off-by: Igor Stoppa <igor.stoppa@huawei.com>
Cc: Arseny Solokha <asolokha@kb.kras.ru>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
-rw-r--r-- | arch/powerpc/mm/tlb_nohash.c | 2 | ||||
-rw-r--r-- | arch/powerpc/sysdev/xive/common.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/arch/powerpc/mm/tlb_nohash.c b/arch/powerpc/mm/tlb_nohash.c index ae5d568e267f..ac23dc1c6535 100644 --- a/arch/powerpc/mm/tlb_nohash.c +++ b/arch/powerpc/mm/tlb_nohash.c | |||
@@ -302,7 +302,7 @@ void __flush_tlb_page(struct mm_struct *mm, unsigned long vmaddr, | |||
302 | * This function as well as __local_flush_tlb_page() must only be called | 302 | * This function as well as __local_flush_tlb_page() must only be called |
303 | * for user contexts. | 303 | * for user contexts. |
304 | */ | 304 | */ |
305 | if (unlikely(WARN_ON(!mm))) | 305 | if (WARN_ON(!mm)) |
306 | return; | 306 | return; |
307 | 307 | ||
308 | preempt_disable(); | 308 | preempt_disable(); |
diff --git a/arch/powerpc/sysdev/xive/common.c b/arch/powerpc/sysdev/xive/common.c index 94a69a62f5db..70a8f9e31a2d 100644 --- a/arch/powerpc/sysdev/xive/common.c +++ b/arch/powerpc/sysdev/xive/common.c | |||
@@ -442,7 +442,7 @@ static void xive_dec_target_count(int cpu) | |||
442 | struct xive_cpu *xc = per_cpu(xive_cpu, cpu); | 442 | struct xive_cpu *xc = per_cpu(xive_cpu, cpu); |
443 | struct xive_q *q = &xc->queue[xive_irq_priority]; | 443 | struct xive_q *q = &xc->queue[xive_irq_priority]; |
444 | 444 | ||
445 | if (unlikely(WARN_ON(cpu < 0 || !xc))) { | 445 | if (WARN_ON(cpu < 0 || !xc)) { |
446 | pr_err("%s: cpu=%d xc=%p\n", __func__, cpu, xc); | 446 | pr_err("%s: cpu=%d xc=%p\n", __func__, cpu, xc); |
447 | return; | 447 | return; |
448 | } | 448 | } |