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

From: Thomas Bogendoerfer
Date: Sun Aug 29 2021 - 07:36:59 EST


On Mon, Aug 23, 2021 at 11:30:04PM -0700, CGEL wrote:
> From: Jing Yangyang <jing.yangyang@xxxxxxxxxx>
>
> ./arch/mips/kvm/mmu.c:489:9-10:WARNING: return of 0/1 in function
> 'kvm_test_age_gfn' with return type bool
> ./arch/mips/kvm/mmu.c:445:8-9 WARNING: return of 0/1 in function
> 'kvm_unmap_gfn_range' 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/mips/kvm/mmu.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/mips/kvm/mmu.c b/arch/mips/kvm/mmu.c
> index 6d1f68c..1bfd1b5 100644
> --- a/arch/mips/kvm/mmu.c
> +++ b/arch/mips/kvm/mmu.c
> @@ -442,7 +442,7 @@ static int kvm_mips_mkold_gpa_pt(struct kvm *kvm, gfn_t start_gfn,
> bool kvm_unmap_gfn_range(struct kvm *kvm, struct kvm_gfn_range *range)
> {
> kvm_mips_flush_gpa_pt(kvm, range->start, range->end);
> - return 1;
> + return true;
> }
>
> bool kvm_set_spte_gfn(struct kvm *kvm, struct kvm_gfn_range *range)
> @@ -486,7 +486,7 @@ bool kvm_test_age_gfn(struct kvm *kvm, struct kvm_gfn_range *range)
> pte_t *gpa_pte = kvm_mips_pte_for_gpa(kvm, NULL, gpa);
>
> if (!gpa_pte)
> - return 0;
> + return false;
> return pte_young(*gpa_pte);

this is already "fixed" in mips-next by

126b39368604 MIPS: Return true/false (not 1/0) from bool functions

Thomas.

--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]