From: Heiko Carstens <hca@linux.ibm.com>
Date: Mon, 13 Feb 2023 11:35:17 +0000 (+0100)
Subject: s390/processor: let cpu helper functions return boolean values
X-Git-Tag: v6.3-rc1~171^2~2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b977f03ec44aef7f6728bfe1a48f3ee5b0776001;p=thirdparty%2Fkernel%2Flinux.git

s390/processor: let cpu helper functions return boolean values

Let cpu helper functions return boolean values. This also allows to
make the code a bit simpler by getting rid of the "!!" construct.

Reviewed-by: Sven Schnelle <svens@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
---

diff --git a/arch/s390/include/asm/processor.h b/arch/s390/include/asm/processor.h
index 53172b27a3eb4..9f89d3821d166 100644
--- a/arch/s390/include/asm/processor.h
+++ b/arch/s390/include/asm/processor.h
@@ -54,19 +54,20 @@ static __always_inline void clear_cpu_flag(int flag)
 	S390_lowcore.cpu_flags &= ~(1UL << flag);
 }
 
-static __always_inline int test_cpu_flag(int flag)
+static __always_inline bool test_cpu_flag(int flag)
 {
-	return !!(S390_lowcore.cpu_flags & (1UL << flag));
+	return S390_lowcore.cpu_flags & (1UL << flag);
 }
 
 /*
  * Test CIF flag of another CPU. The caller needs to ensure that
  * CPU hotplug can not happen, e.g. by disabling preemption.
  */
-static __always_inline int test_cpu_flag_of(int flag, int cpu)
+static __always_inline bool test_cpu_flag_of(int flag, int cpu)
 {
 	struct lowcore *lc = lowcore_ptr[cpu];
-	return !!(lc->cpu_flags & (1UL << flag));
+
+	return lc->cpu_flags & (1UL << flag);
 }
 
 #define arch_needs_cpu() test_cpu_flag(CIF_NOHZ_DELAY)