KVM: arm/arm64: Change misleading use of is_error_pfn
authorChristoffer Dall <christoffer.dall@linaro.org>
Wed, 17 Aug 2016 08:46:10 +0000 (10:46 +0200)
committerChristoffer Dall <christoffer.dall@linaro.org>
Wed, 17 Aug 2016 09:38:03 +0000 (11:38 +0200)
commit9ac715954682b23d293d910ad2697554171035e7
treed0d273f5f6a82458ac18addcf29cd57ea7ceef45
parent286054a7a8674b256da16564d96772b88896eb35
KVM: arm/arm64: Change misleading use of is_error_pfn

When converting a gfn to a pfn, we call gfn_to_pfn_prot, which returns
various kinds of error values.  It turns out that is_error_pfn() only
returns true when the gfn was found in a memory slot and could somehow
not be used, but it does not return true if the gfn does not belong to
any memory slot.

Change use to is_error_noslot_pfn() which covers both cases.

Note: Since we already check for kvm_is_error_hva(hva) explicitly in the
caller of this function while holding the kvm->srcu lock protecting the
memory slots, this should never be a problem, but nevertheless this
change is warranted as it shows the intention of the code.

Reported-by: James Hogan <james.hogan@imgtec.com>
Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
arch/arm/kvm/mmu.c