Re: [PATCH v2 1/2] KVM: PPC: Book3S HV: Drop prepare_done from struct kvm_resize_hpt and cleanups

[Joe Perches]

On Mon, 2017-12-04 at 09:36 -0500, Serhii Popovych wrote:
> Change comparison against zero to make checkpatch.pl happy.

Huh?  Either I'm confused or you are incorrect here.

checkpatch should or does not warn against comparisons
to zero.  There is a test for comparison to NULL.

> diff --git a/arch/powerpc/kvm/book3s_64_mmu_hv.c 
> b/arch/powerpc/kvm/book3s_64_mmu_hv.c
[]
> @@ -1547,16 +1561,12 @@ long kvm_vm_ioctl_resize_hpt_commit(struct kvm *kvm,
>       if (!resize || (resize->order != shift))
>               goto out;
>  
> -     ret = -EBUSY;
> -     if (!resize->prepare_done)
> -             goto out;
> -
>       ret = resize->error;
> -     if (ret != 0)
> +     if (ret)
>               goto out;
>  
>       ret = resize_hpt_rehash(resize);
> -     if (ret != 0)
> +     if (ret)
>               goto out;
>  
>       resize_hpt_pivot(resize);