From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Date: Wed, 18 Nov 2015 11:05:15 +0000 Subject: Re: [PATCH] KVM: powerpc: kvmppc_visible_gpa can be boolean Message-Id: <564C5B6B.8070305@redhat.com> List-Id: References: <1447643424-4058-1-git-send-email-baiyaowei@cmss.chinamobile.com> In-Reply-To: <1447643424-4058-1-git-send-email-baiyaowei@cmss.chinamobile.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Yaowei Bai , agraf@suse.sz, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, gleb@kernel.org Cc: kvm-ppc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, kvm@vger.kernel.org, linux-kernel@vger.kernel.org On 16/11/2015 04:10, Yaowei Bai wrote: > In another patch kvm_is_visible_gfn is maken return bool due to this > function only returns zero or one as its return value, let's also make > kvmppc_visible_gpa return bool to keep consistent. > > No functional change. > > Signed-off-by: Yaowei Bai > --- > arch/powerpc/kvm/book3s_pr.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/kvm/book3s_pr.c b/arch/powerpc/kvm/book3s_pr.c > index 64891b0..70fb08d 100644 > --- a/arch/powerpc/kvm/book3s_pr.c > +++ b/arch/powerpc/kvm/book3s_pr.c > @@ -512,7 +512,7 @@ static void kvmppc_patch_dcbz(struct kvm_vcpu *vcpu, struct kvmppc_pte *pte) > put_page(hpage); > } > > -static int kvmppc_visible_gpa(struct kvm_vcpu *vcpu, gpa_t gpa) > +static bool kvmppc_visible_gpa(struct kvm_vcpu *vcpu, gpa_t gpa) > { > ulong mp_pa = vcpu->arch.magic_page_pa; > > @@ -521,7 +521,7 @@ static int kvmppc_visible_gpa(struct kvm_vcpu *vcpu, gpa_t gpa) > > gpa &= ~0xFFFULL; > if (unlikely(mp_pa) && unlikely((mp_pa & KVM_PAM) = (gpa & KVM_PAM))) { > - return 1; > + return true; > } > > return kvm_is_visible_gfn(vcpu->kvm, gpa >> PAGE_SHIFT); > Applied all three patches to kvm/queue, thanks. Paolo