[PATCH linux-next] x86:mmu: fix boolreturn.cocci warnings

From: CGEL
Date: Tue Aug 24 2021 - 03:11:05 EST


From: Jing Yangyang <jing.yangyang@xxxxxxxxxx>

./arch/x86/kvm/mmu/mmu.c:1501: 9-10:WARNING:return of 0/1 in
function 'kvm_set_pte_rmapp' with return type bool
./arch/x86/kvm/mmu/mmu.c:1642:10-11:WARNING:return of 0/1 in
function 'kvm_test_age_rmapp' with return type bool

Return statements in functions returning bool should use true/false
instead of 1/0.

Generated by: scripts/coccinelle/misc/boolreturn.cocci

Reported-by: Zeal Robot <zealci@xxxxxxxxxx>
Signed-off-by: Jing Yangyang <jing.yangyang@xxxxxxxxxx>
---
arch/x86/kvm/mmu/mmu.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/x86/kvm/mmu/mmu.c b/arch/x86/kvm/mmu/mmu.c
index c5232a0..78025e0 100644
--- a/arch/x86/kvm/mmu/mmu.c
+++ b/arch/x86/kvm/mmu/mmu.c
@@ -1498,7 +1498,7 @@ static bool kvm_set_pte_rmapp(struct kvm *kvm, struct kvm_rmap_head *rmap_head,

if (need_flush && kvm_available_flush_tlb_with_range()) {
kvm_flush_remote_tlbs_with_address(kvm, gfn, 1);
- return 0;
+ return false;
}

return need_flush;
@@ -1639,8 +1639,8 @@ static bool kvm_test_age_rmapp(struct kvm *kvm, struct kvm_rmap_head *rmap_head,

for_each_rmap_spte(rmap_head, &iter, sptep)
if (is_accessed_spte(*sptep))
- return 1;
- return 0;
+ return true;
+ return false;
}

#define RMAP_RECYCLE_THRESHOLD 1000
--
1.8.3.1