include: mman: use bool instead of int for the return value of arch_validate_prot
authorChen Gang <chengang@emindsoft.com.cn>
Tue, 2 Aug 2016 21:03:42 +0000 (14:03 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 2 Aug 2016 23:35:02 +0000 (19:35 -0400)
For pure bool function's return value, bool is a little better more or
less than int.

Link: http://lkml.kernel.org/r/1469331815-2026-1-git-send-email-chengang@emindsoft.com.cn
Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
Acked-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/powerpc/include/asm/mman.h
include/linux/mman.h

index 2563c435a4b166593e71812086baa791a01de43b..fc420cedecae59c6ee90579b67bf817bb2123491 100644 (file)
@@ -31,13 +31,13 @@ static inline pgprot_t arch_vm_get_page_prot(unsigned long vm_flags)
 }
 #define arch_vm_get_page_prot(vm_flags) arch_vm_get_page_prot(vm_flags)
 
-static inline int arch_validate_prot(unsigned long prot)
+static inline bool arch_validate_prot(unsigned long prot)
 {
        if (prot & ~(PROT_READ | PROT_WRITE | PROT_EXEC | PROT_SEM | PROT_SAO))
-               return 0;
+               return false;
        if ((prot & PROT_SAO) && !cpu_has_feature(CPU_FTR_SAO))
-               return 0;
-       return 1;
+               return false;
+       return true;
 }
 #define arch_validate_prot(prot) arch_validate_prot(prot)
 
index 33e17f6a327a2cb9972d1e31134ac354be27f045..634c4c51fe3adaee4b65d9a977e954f42ecf6131 100644 (file)
@@ -49,7 +49,7 @@ static inline void vm_unacct_memory(long pages)
  *
  * Returns true if the prot flags are valid
  */
-static inline int arch_validate_prot(unsigned long prot)
+static inline bool arch_validate_prot(unsigned long prot)
 {
        return (prot & ~(PROT_READ | PROT_WRITE | PROT_EXEC | PROT_SEM)) == 0;
 }