s390/processor: let cpu helper functions return boolean values
authorHeiko Carstens <hca@linux.ibm.com>
Mon, 13 Feb 2023 11:35:17 +0000 (12:35 +0100)
committerHeiko Carstens <hca@linux.ibm.com>
Wed, 15 Feb 2023 10:07:01 +0000 (11:07 +0100)
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>
arch/s390/include/asm/processor.h

index 53172b2..9f89d38 100644 (file)
@@ -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)