kvm: fix kvm_is_mmio_pfn() and rename to kvm_is_reserved_pfn()
authorArd Biesheuvel <ard.biesheuvel@linaro.org>
Mon, 10 Nov 2014 08:33:56 +0000 (08:33 +0000)
committerMarc Zyngier <marc.zyngier@arm.com>
Tue, 25 Nov 2014 13:57:26 +0000 (13:57 +0000)
commitbf4bea8e9a9058319a19f8c2710a6f0ef2459983
tree192e21f7d1509b7ea3303ac64bc21f9123bffcd4
parent07a9748c78cfc39b54f06125a216b67b9c8f09ed
kvm: fix kvm_is_mmio_pfn() and rename to kvm_is_reserved_pfn()

This reverts commit 85c8555ff0 ("KVM: check for !is_zero_pfn() in
kvm_is_mmio_pfn()") and renames the function to kvm_is_reserved_pfn.

The problem being addressed by the patch above was that some ARM code
based the memory mapping attributes of a pfn on the return value of
kvm_is_mmio_pfn(), whose name indeed suggests that such pfns should
be mapped as device memory.

However, kvm_is_mmio_pfn() doesn't do quite what it says on the tin,
and the existing non-ARM users were already using it in a way which
suggests that its name should probably have been 'kvm_is_reserved_pfn'
from the beginning, e.g., whether or not to call get_page/put_page on
it etc. This means that returning false for the zero page is a mistake
and the patch above should be reverted.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
arch/ia64/kvm/kvm-ia64.c
arch/x86/kvm/mmu.c
include/linux/kvm_host.h
virt/kvm/kvm_main.c