kvm/ppc/e500: h2g_tlb1_rmap: esel 0 is valid
authorScott Wood <scottwood@freescale.com>
Wed, 13 Feb 2013 19:37:48 +0000 (19:37 +0000)
committerAlexander Graf <agraf@suse.de>
Fri, 22 Mar 2013 00:21:11 +0000 (01:21 +0100)
Add one to esel values in h2g_tlb1_rmap, so that "no mapping" can be
distinguished from "esel 0".  Note that we're not saved by the fact
that host esel 0 is reserved for non-KVM use, because KVM host esel
numbering is not the raw host numbering (see to_htlb1_esel).

Signed-off-by: Scott Wood <scottwood@freescale.com>
Signed-off-by: Alexander Graf <agraf@suse.de>
arch/powerpc/kvm/e500_mmu_host.c

index a222edfb9a9bce72e7d642b05853398b493bc679..35fb80ec1f571bb7692acc12c43faea98006022f 100644 (file)
@@ -511,10 +511,10 @@ static int kvmppc_e500_tlb1_map_tlb1(struct kvmppc_vcpu_e500 *vcpu_e500,
        vcpu_e500->g2h_tlb1_map[esel] |= (u64)1 << sesel;
        vcpu_e500->gtlb_priv[1][esel].ref.flags |= E500_TLB_BITMAP;
        if (vcpu_e500->h2g_tlb1_rmap[sesel]) {
-               unsigned int idx = vcpu_e500->h2g_tlb1_rmap[sesel];
+               unsigned int idx = vcpu_e500->h2g_tlb1_rmap[sesel] - 1;
                vcpu_e500->g2h_tlb1_map[idx] &= ~(1ULL << sesel);
        }
-       vcpu_e500->h2g_tlb1_rmap[sesel] = esel;
+       vcpu_e500->h2g_tlb1_rmap[sesel] = esel + 1;
 
        return sesel;
 }