diff options
| author |   Xiao Guangrong <xiaoguangrong@linux.vnet.ibm.com> | 2012-10-16 20:10:59 +0800 |
|---|---|---|
| committer |   Marcelo Tosatti <mtosatti@redhat.com> | 2012-10-29 20:31:04 -0200 |
| commit | 81c52c56e2b43589091ee29038bcf793d3f184ab (patch) | |
| tree | 763236629eb034e519db87d8568946883499a155 /arch/powerpc/kvm/e500_tlb.c | |
| parent | Merge remote-tracking branch 'master' into queue (diff) | |
| download | linux-dev-81c52c56e2b43589091ee29038bcf793d3f184ab.tar.xz linux-dev-81c52c56e2b43589091ee29038bcf793d3f184ab.zip | |
KVM: do not treat noslot pfn as a error pfn
This patch filters noslot pfn out from error pfns based on Marcelo comment:
noslot pfn is not a error pfn
After this patch,
- is_noslot_pfn indicates that the gfn is not in slot
- is_error_pfn indicates that the gfn is in slot but the error is occurred
when translate the gfn to pfn
- is_error_noslot_pfn indicates that the pfn either it is error pfns or it
is noslot pfn
And is_invalid_pfn can be removed, it makes the code more clean
Signed-off-by: Xiao Guangrong <xiaoguangrong@linux.vnet.ibm.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Diffstat (limited to 'arch/powerpc/kvm/e500_tlb.c')
| -rw-r--r-- | arch/powerpc/kvm/e500_tlb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/kvm/e500_tlb.c b/arch/powerpc/kvm/e500_tlb.c index c73389477d17..6305ee692ef7 100644 --- a/arch/powerpc/kvm/e500_tlb.c +++ b/arch/powerpc/kvm/e500_tlb.c @@ -524,7 +524,7 @@ static inline void kvmppc_e500_shadow_map(struct kvmppc_vcpu_e500 *vcpu_e500, if (likely(!pfnmap)) { unsigned long tsize_pages = 1 << (tsize + 10 - PAGE_SHIFT); pfn = gfn_to_pfn_memslot(slot, gfn); - if (is_error_pfn(pfn)) { + if (is_error_noslot_pfn(pfn)) { printk(KERN_ERR "Couldn't get real page for gfn %lx!\n", (long)gfn); return; |