KVM: PPC: e500: Stop keeping shadow TLB
authorLiu Yu <yu.liu@freescale.com>
Tue, 14 Jun 2011 23:34:59 +0000 (18:34 -0500)
committerAvi Kivity <avi@redhat.com>
Tue, 12 Jul 2011 10:16:38 +0000 (13:16 +0300)
Instead of a fully separate set of TLB entries, keep just the
pfn and dirty status.

Signed-off-by: Liu Yu <yu.liu@freescale.com>
Signed-off-by: Scott Wood <scottwood@freescale.com>
Signed-off-by: Alexander Graf <agraf@suse.de>
arch/powerpc/include/asm/kvm_e500.h
arch/powerpc/kvm/e500_tlb.c

index f181ad1d604510f8a4ee659710829808da72ea25..4a6d77a8b8a06c9d2f65e61ae34ff0ffe2ab1551 100644 (file)
@@ -29,15 +29,23 @@ struct tlbe{
        u32 mas7;
 };
 
+#define E500_TLB_VALID 1
+#define E500_TLB_DIRTY 2
+
+struct tlbe_priv {
+       pfn_t pfn;
+       unsigned int flags; /* E500_TLB_* */
+};
+
 struct kvmppc_vcpu_e500 {
        /* Unmodified copy of the guest's TLB. */
-       struct tlbe *guest_tlb[E500_TLB_NUM];
-       /* TLB that's actually used when the guest is running. */
-       struct tlbe *shadow_tlb[E500_TLB_NUM];
+       struct tlbe *gtlb_arch[E500_TLB_NUM];
+
+       /* KVM internal information associated with each guest TLB entry */
+       struct tlbe_priv *gtlb_priv[E500_TLB_NUM];
 
-       unsigned int guest_tlb_size[E500_TLB_NUM];
-       unsigned int shadow_tlb_size[E500_TLB_NUM];
-       unsigned int guest_tlb_nv[E500_TLB_NUM];
+       unsigned int gtlb_size[E500_TLB_NUM];
+       unsigned int gtlb_nv[E500_TLB_NUM];
 
        u32 host_pid[E500_PID_NUM];
        u32 pid[E500_PID_NUM];
index c09e642ee5373b95467db10942c8878f29d9177e..9d1e28d443c4040dcd8bc76347b3e2095c5f884f 100644 (file)
@@ -41,25 +41,14 @@ void kvmppc_dump_tlbs(struct kvm_vcpu *vcpu)
 
        for (tlbsel = 0; tlbsel < 2; tlbsel++) {
                printk("Guest TLB%d:\n", tlbsel);
-               for (i = 0; i < vcpu_e500->guest_tlb_size[tlbsel]; i++) {
-                       tlbe = &vcpu_e500->guest_tlb[tlbsel][i];
+               for (i = 0; i < vcpu_e500->gtlb_size[tlbsel]; i++) {
+                       tlbe = &vcpu_e500->gtlb_arch[tlbsel][i];
                        if (tlbe->mas1 & MAS1_VALID)
                                printk(" G[%d][%3d] |  %08X | %08X | %08X | %08X |\n",
                                        tlbsel, i, tlbe->mas1, tlbe->mas2,
                                        tlbe->mas3, tlbe->mas7);
                }
        }
-
-       for (tlbsel = 0; tlbsel < 2; tlbsel++) {
-               printk("Shadow TLB%d:\n", tlbsel);
-               for (i = 0; i < vcpu_e500->shadow_tlb_size[tlbsel]; i++) {
-                       tlbe = &vcpu_e500->shadow_tlb[tlbsel][i];
-                       if (tlbe->mas1 & MAS1_VALID)
-                               printk(" S[%d][%3d] |  %08X | %08X | %08X | %08X |\n",
-                                       tlbsel, i, tlbe->mas1, tlbe->mas2,
-                                       tlbe->mas3, tlbe->mas7);
-               }
-       }
 }
 
 static inline unsigned int tlb0_get_next_victim(
@@ -67,9 +56,9 @@ static inline unsigned int tlb0_get_next_victim(
 {
        unsigned int victim;
 
-       victim = vcpu_e500->guest_tlb_nv[0]++;
-       if (unlikely(vcpu_e500->guest_tlb_nv[0] >= KVM_E500_TLB0_WAY_NUM))
-               vcpu_e500->guest_tlb_nv[0] = 0;
+       victim = vcpu_e500->gtlb_nv[0]++;
+       if (unlikely(vcpu_e500->gtlb_nv[0] >= KVM_E500_TLB0_WAY_NUM))
+               vcpu_e500->gtlb_nv[0] = 0;
 
        return victim;
 }
@@ -128,10 +117,8 @@ static inline void __write_host_tlbe(struct tlbe *stlbe, uint32_t mas0)
 }
 
 static inline void write_host_tlbe(struct kvmppc_vcpu_e500 *vcpu_e500,
-               int tlbsel, int esel)
+               int tlbsel, int esel, struct tlbe *stlbe)
 {
-       struct tlbe *stlbe = &vcpu_e500->shadow_tlb[tlbsel][esel];
-
        if (tlbsel == 0) {
                __write_host_tlbe(stlbe,
                                  MAS0_TLBSEL(0) |
@@ -141,6 +128,8 @@ static inline void write_host_tlbe(struct kvmppc_vcpu_e500 *vcpu_e500,
                                  MAS0_TLBSEL(1) |
                                  MAS0_ESEL(to_htlb1_esel(esel)));
        }
+       trace_kvm_stlb_write(index_of(tlbsel, esel), stlbe->mas1, stlbe->mas2,
+                            stlbe->mas3, stlbe->mas7);
 }
 
 void kvmppc_map_magic(struct kvm_vcpu *vcpu)
@@ -178,8 +167,8 @@ static int kvmppc_e500_tlb_index(struct kvmppc_vcpu_e500 *vcpu_e500,
        int i;
 
        /* XXX Replace loop with fancy data structures. */
-       for (i = 0; i < vcpu_e500->guest_tlb_size[tlbsel]; i++) {
-               struct tlbe *tlbe = &vcpu_e500->guest_tlb[tlbsel][i];
+       for (i = 0; i < vcpu_e500->gtlb_size[tlbsel]; i++) {
+               struct tlbe *tlbe = &vcpu_e500->gtlb_arch[tlbsel][i];
                unsigned int tid;
 
                if (eaddr < get_tlb_eaddr(tlbe))
@@ -204,60 +193,33 @@ static int kvmppc_e500_tlb_index(struct kvmppc_vcpu_e500 *vcpu_e500,
        return -1;
 }
 
-static void kvmppc_e500_shadow_release(struct kvmppc_vcpu_e500 *vcpu_e500,
-               int tlbsel, int esel)
+static inline void kvmppc_e500_priv_setup(struct tlbe_priv *priv,
+                                         struct tlbe *gtlbe,
+                                         pfn_t pfn)
 {
-       struct tlbe *stlbe = &vcpu_e500->shadow_tlb[tlbsel][esel];
-       unsigned long pfn;
-
-       pfn = stlbe->mas3 >> PAGE_SHIFT;
-       pfn |= stlbe->mas7 << (32 - PAGE_SHIFT);
+       priv->pfn = pfn;
+       priv->flags = E500_TLB_VALID;
 
-       if (get_tlb_v(stlbe)) {
-               if (tlbe_is_writable(stlbe))
-                       kvm_release_pfn_dirty(pfn);
-               else
-                       kvm_release_pfn_clean(pfn);
-       }
+       if (tlbe_is_writable(gtlbe))
+               priv->flags |= E500_TLB_DIRTY;
 }
 
-static void kvmppc_e500_stlbe_invalidate(struct kvmppc_vcpu_e500 *vcpu_e500,
-               int tlbsel, int esel)
+static inline void kvmppc_e500_priv_release(struct tlbe_priv *priv)
 {
-       struct tlbe *stlbe = &vcpu_e500->shadow_tlb[tlbsel][esel];
+       if (priv->flags & E500_TLB_VALID) {
+               if (priv->flags & E500_TLB_DIRTY)
+                       kvm_release_pfn_dirty(priv->pfn);
+               else
+                       kvm_release_pfn_clean(priv->pfn);
 
-       kvmppc_e500_shadow_release(vcpu_e500, tlbsel, esel);
-       stlbe->mas1 = 0;
-       trace_kvm_stlb_inval(index_of(tlbsel, esel));
+               priv->flags = 0;
+       }
 }
 
 static void kvmppc_e500_tlb1_invalidate(struct kvmppc_vcpu_e500 *vcpu_e500,
-               gva_t eaddr, gva_t eend, u32 tid)
+                                       int esel)
 {
-       unsigned int pid = tid & 0xff;
-       unsigned int i;
-
-       /* XXX Replace loop with fancy data structures. */
-       for (i = 0; i < vcpu_e500->guest_tlb_size[1]; i++) {
-               struct tlbe *stlbe = &vcpu_e500->shadow_tlb[1][i];
-               unsigned int tid;
-
-               if (!get_tlb_v(stlbe))
-                       continue;
-
-               if (eend < get_tlb_eaddr(stlbe))
-                       continue;
-
-               if (eaddr > get_tlb_end(stlbe))
-                       continue;
-
-               tid = get_tlb_tid(stlbe);
-               if (tid && (tid != pid))
-                       continue;
-
-               kvmppc_e500_stlbe_invalidate(vcpu_e500, 1, i);
-               write_host_tlbe(vcpu_e500, 1, i);
-       }
+       mtspr(SPRN_MMUCSR0, MMUCSR0_TLB1FI);
 }
 
 static inline void kvmppc_e500_deliver_tlb_miss(struct kvm_vcpu *vcpu,
@@ -274,7 +236,7 @@ static inline void kvmppc_e500_deliver_tlb_miss(struct kvm_vcpu *vcpu,
        tsized = (vcpu_e500->mas4 >> 7) & 0x1f;
 
        vcpu_e500->mas0 = MAS0_TLBSEL(tlbsel) | MAS0_ESEL(victim)
-               | MAS0_NV(vcpu_e500->guest_tlb_nv[tlbsel]);
+               | MAS0_NV(vcpu_e500->gtlb_nv[tlbsel]);
        vcpu_e500->mas1 = MAS1_VALID | (as ? MAS1_TS : 0)
                | MAS1_TID(vcpu_e500->pid[pidsel])
                | MAS1_TSIZE(tsized);
@@ -287,16 +249,35 @@ static inline void kvmppc_e500_deliver_tlb_miss(struct kvm_vcpu *vcpu,
        vcpu_e500->mas7 = 0;
 }
 
+static inline void kvmppc_e500_setup_stlbe(struct kvmppc_vcpu_e500 *vcpu_e500,
+                                          struct tlbe *gtlbe, int tsize,
+                                          struct tlbe_priv *priv,
+                                          u64 gvaddr, struct tlbe *stlbe)
+{
+       pfn_t pfn = priv->pfn;
+
+       /* Force TS=1 IPROT=0 for all guest mappings. */
+       stlbe->mas1 = MAS1_TSIZE(tsize)
+               | MAS1_TID(get_tlb_tid(gtlbe)) | MAS1_TS | MAS1_VALID;
+       stlbe->mas2 = (gvaddr & MAS2_EPN)
+               | e500_shadow_mas2_attrib(gtlbe->mas2,
+                               vcpu_e500->vcpu.arch.shared->msr & MSR_PR);
+       stlbe->mas3 = ((pfn << PAGE_SHIFT) & MAS3_RPN)
+               | e500_shadow_mas3_attrib(gtlbe->mas3,
+                               vcpu_e500->vcpu.arch.shared->msr & MSR_PR);
+       stlbe->mas7 = (pfn >> (32 - PAGE_SHIFT)) & MAS7_RPN;
+}
+
+
 static inline void kvmppc_e500_shadow_map(struct kvmppc_vcpu_e500 *vcpu_e500,
-       u64 gvaddr, gfn_t gfn, struct tlbe *gtlbe, int tlbsel, int esel)
+       u64 gvaddr, gfn_t gfn, struct tlbe *gtlbe, int tlbsel, int esel,
+       struct tlbe *stlbe)
 {
        struct kvm_memory_slot *slot;
-       struct tlbe *stlbe;
        unsigned long pfn, hva;
        int pfnmap = 0;
        int tsize = BOOK3E_PAGESZ_4K;
-
-       stlbe = &vcpu_e500->shadow_tlb[tlbsel][esel];
+       struct tlbe_priv *priv;
 
        /*
         * Translate guest physical to true physical, acquiring
@@ -392,35 +373,25 @@ static inline void kvmppc_e500_shadow_map(struct kvmppc_vcpu_e500 *vcpu_e500,
                }
        }
 
-       /* Drop reference to old page. */
-       kvmppc_e500_shadow_release(vcpu_e500, tlbsel, esel);
-
-       /* Force TS=1 IPROT=0 for all guest mappings. */
-       stlbe->mas1 = MAS1_TSIZE(tsize)
-               | MAS1_TID(get_tlb_tid(gtlbe)) | MAS1_TS | MAS1_VALID;
-       stlbe->mas2 = (gvaddr & MAS2_EPN)
-               | e500_shadow_mas2_attrib(gtlbe->mas2,
-                               vcpu_e500->vcpu.arch.shared->msr & MSR_PR);
-       stlbe->mas3 = ((pfn << PAGE_SHIFT) & MAS3_RPN)
-               | e500_shadow_mas3_attrib(gtlbe->mas3,
-                               vcpu_e500->vcpu.arch.shared->msr & MSR_PR);
-       stlbe->mas7 = (pfn >> (32 - PAGE_SHIFT)) & MAS7_RPN;
+       /* Drop old priv and setup new one. */
+       priv = &vcpu_e500->gtlb_priv[tlbsel][esel];
+       kvmppc_e500_priv_release(priv);
+       kvmppc_e500_priv_setup(priv, gtlbe, pfn);
 
-       trace_kvm_stlb_write(index_of(tlbsel, esel), stlbe->mas1, stlbe->mas2,
-                            stlbe->mas3, stlbe->mas7);
+       kvmppc_e500_setup_stlbe(vcpu_e500, gtlbe, tsize, priv, gvaddr, stlbe);
 }
 
 /* XXX only map the one-one case, for now use TLB0 */
-static int kvmppc_e500_stlbe_map(struct kvmppc_vcpu_e500 *vcpu_e500,
-               int tlbsel, int esel)
+static int kvmppc_e500_tlb0_map(struct kvmppc_vcpu_e500 *vcpu_e500,
+                               int esel, struct tlbe *stlbe)
 {
        struct tlbe *gtlbe;
 
-       gtlbe = &vcpu_e500->guest_tlb[tlbsel][esel];
+       gtlbe = &vcpu_e500->gtlb_arch[0][esel];
 
        kvmppc_e500_shadow_map(vcpu_e500, get_tlb_eaddr(gtlbe),
                        get_tlb_raddr(gtlbe) >> PAGE_SHIFT,
-                       gtlbe, tlbsel, esel);
+                       gtlbe, 0, esel, stlbe);
 
        return esel;
 }
@@ -429,16 +400,16 @@ static int kvmppc_e500_stlbe_map(struct kvmppc_vcpu_e500 *vcpu_e500,
  * the shadow TLB. */
 /* XXX for both one-one and one-to-many , for now use TLB1 */
 static int kvmppc_e500_tlb1_map(struct kvmppc_vcpu_e500 *vcpu_e500,
-               u64 gvaddr, gfn_t gfn, struct tlbe *gtlbe)
+               u64 gvaddr, gfn_t gfn, struct tlbe *gtlbe, struct tlbe *stlbe)
 {
        unsigned int victim;
 
-       victim = vcpu_e500->guest_tlb_nv[1]++;
+       victim = vcpu_e500->gtlb_nv[1]++;
 
-       if (unlikely(vcpu_e500->guest_tlb_nv[1] >= tlb1_max_shadow_size()))
-               vcpu_e500->guest_tlb_nv[1] = 0;
+       if (unlikely(vcpu_e500->gtlb_nv[1] >= tlb1_max_shadow_size()))
+               vcpu_e500->gtlb_nv[1] = 0;
 
-       kvmppc_e500_shadow_map(vcpu_e500, gvaddr, gfn, gtlbe, 1, victim);
+       kvmppc_e500_shadow_map(vcpu_e500, gvaddr, gfn, gtlbe, 1, victim, stlbe);
 
        return victim;
 }
@@ -449,33 +420,19 @@ static int kvmppc_e500_tlb1_map(struct kvmppc_vcpu_e500 *vcpu_e500,
 void kvmppc_mmu_priv_switch(struct kvm_vcpu *vcpu, int usermode)
 {
        if (usermode) {
-               struct kvmppc_vcpu_e500 *vcpu_e500 = to_e500(vcpu);
-               int i;
-
-               /* XXX Replace loop with fancy data structures. */
-               for (i = 0; i < tlb1_max_shadow_size(); i++)
-                       kvmppc_e500_stlbe_invalidate(vcpu_e500, 1, i);
-
                _tlbil_all();
        }
 }
 
-static int kvmppc_e500_gtlbe_invalidate(struct kvmppc_vcpu_e500 *vcpu_e500,
-               int tlbsel, int esel)
+static inline int kvmppc_e500_gtlbe_invalidate(
+                               struct kvmppc_vcpu_e500 *vcpu_e500,
+                               int tlbsel, int esel)
 {
-       struct tlbe *gtlbe = &vcpu_e500->guest_tlb[tlbsel][esel];
+       struct tlbe *gtlbe = &vcpu_e500->gtlb_arch[tlbsel][esel];
 
        if (unlikely(get_tlb_iprot(gtlbe)))
                return -1;
 
-       if (tlbsel == 1) {
-               kvmppc_e500_tlb1_invalidate(vcpu_e500, get_tlb_eaddr(gtlbe),
-                               get_tlb_end(gtlbe),
-                               get_tlb_tid(gtlbe));
-       } else {
-               kvmppc_e500_stlbe_invalidate(vcpu_e500, tlbsel, esel);
-       }
-
        gtlbe->mas1 = 0;
 
        return 0;
@@ -486,10 +443,10 @@ int kvmppc_e500_emul_mt_mmucsr0(struct kvmppc_vcpu_e500 *vcpu_e500, ulong value)
        int esel;
 
        if (value & MMUCSR0_TLB0FI)
-               for (esel = 0; esel < vcpu_e500->guest_tlb_size[0]; esel++)
+               for (esel = 0; esel < vcpu_e500->gtlb_size[0]; esel++)
                        kvmppc_e500_gtlbe_invalidate(vcpu_e500, 0, esel);
        if (value & MMUCSR0_TLB1FI)
-               for (esel = 0; esel < vcpu_e500->guest_tlb_size[1]; esel++)
+               for (esel = 0; esel < vcpu_e500->gtlb_size[1]; esel++)
                        kvmppc_e500_gtlbe_invalidate(vcpu_e500, 1, esel);
 
        _tlbil_all();
@@ -513,7 +470,7 @@ int kvmppc_e500_emul_tlbivax(struct kvm_vcpu *vcpu, int ra, int rb)
 
        if (ia) {
                /* invalidate all entries */
-               for (esel = 0; esel < vcpu_e500->guest_tlb_size[tlbsel]; esel++)
+               for (esel = 0; esel < vcpu_e500->gtlb_size[tlbsel]; esel++)
                        kvmppc_e500_gtlbe_invalidate(vcpu_e500, tlbsel, esel);
        } else {
                ea &= 0xfffff000;
@@ -537,9 +494,9 @@ int kvmppc_e500_emul_tlbre(struct kvm_vcpu *vcpu)
        tlbsel = get_tlb_tlbsel(vcpu_e500);
        esel = get_tlb_esel(vcpu_e500, tlbsel);
 
-       gtlbe = &vcpu_e500->guest_tlb[tlbsel][esel];
+       gtlbe = &vcpu_e500->gtlb_arch[tlbsel][esel];
        vcpu_e500->mas0 &= ~MAS0_NV(~0);
-       vcpu_e500->mas0 |= MAS0_NV(vcpu_e500->guest_tlb_nv[tlbsel]);
+       vcpu_e500->mas0 |= MAS0_NV(vcpu_e500->gtlb_nv[tlbsel]);
        vcpu_e500->mas1 = gtlbe->mas1;
        vcpu_e500->mas2 = gtlbe->mas2;
        vcpu_e500->mas3 = gtlbe->mas3;
@@ -562,14 +519,14 @@ int kvmppc_e500_emul_tlbsx(struct kvm_vcpu *vcpu, int rb)
        for (tlbsel = 0; tlbsel < 2; tlbsel++) {
                esel = kvmppc_e500_tlb_index(vcpu_e500, ea, tlbsel, pid, as);
                if (esel >= 0) {
-                       gtlbe = &vcpu_e500->guest_tlb[tlbsel][esel];
+                       gtlbe = &vcpu_e500->gtlb_arch[tlbsel][esel];
                        break;
                }
        }
 
        if (gtlbe) {
                vcpu_e500->mas0 = MAS0_TLBSEL(tlbsel) | MAS0_ESEL(esel)
-                       | MAS0_NV(vcpu_e500->guest_tlb_nv[tlbsel]);
+                       | MAS0_NV(vcpu_e500->gtlb_nv[tlbsel]);
                vcpu_e500->mas1 = gtlbe->mas1;
                vcpu_e500->mas2 = gtlbe->mas2;
                vcpu_e500->mas3 = gtlbe->mas3;
@@ -582,7 +539,7 @@ int kvmppc_e500_emul_tlbsx(struct kvm_vcpu *vcpu, int rb)
                victim = (tlbsel == 0) ? tlb0_get_next_victim(vcpu_e500) : 0;
 
                vcpu_e500->mas0 = MAS0_TLBSEL(tlbsel) | MAS0_ESEL(victim)
-                       | MAS0_NV(vcpu_e500->guest_tlb_nv[tlbsel]);
+                       | MAS0_NV(vcpu_e500->gtlb_nv[tlbsel]);
                vcpu_e500->mas1 = (vcpu_e500->mas6 & MAS6_SPID0)
                        | (vcpu_e500->mas6 & (MAS6_SAS ? MAS1_TS : 0))
                        | (vcpu_e500->mas4 & MAS4_TSIZED(~0));
@@ -599,23 +556,16 @@ int kvmppc_e500_emul_tlbsx(struct kvm_vcpu *vcpu, int rb)
 int kvmppc_e500_emul_tlbwe(struct kvm_vcpu *vcpu)
 {
        struct kvmppc_vcpu_e500 *vcpu_e500 = to_e500(vcpu);
-       u64 eaddr;
-       u64 raddr;
-       u32 tid;
        struct tlbe *gtlbe;
-       int tlbsel, esel, stlbsel, sesel;
+       int tlbsel, esel;
 
        tlbsel = get_tlb_tlbsel(vcpu_e500);
        esel = get_tlb_esel(vcpu_e500, tlbsel);
 
-       gtlbe = &vcpu_e500->guest_tlb[tlbsel][esel];
+       gtlbe = &vcpu_e500->gtlb_arch[tlbsel][esel];
 
-       if (get_tlb_v(gtlbe) && tlbsel == 1) {
-               eaddr = get_tlb_eaddr(gtlbe);
-               tid = get_tlb_tid(gtlbe);
-               kvmppc_e500_tlb1_invalidate(vcpu_e500, eaddr,
-                               get_tlb_end(gtlbe), tid);
-       }
+       if (get_tlb_v(gtlbe) && tlbsel == 1)
+               kvmppc_e500_tlb1_invalidate(vcpu_e500, esel);
 
        gtlbe->mas1 = vcpu_e500->mas1;
        gtlbe->mas2 = vcpu_e500->mas2;
@@ -627,6 +577,11 @@ int kvmppc_e500_emul_tlbwe(struct kvm_vcpu *vcpu)
 
        /* Invalidate shadow mappings for the about-to-be-clobbered TLBE. */
        if (tlbe_is_host_safe(vcpu, gtlbe)) {
+               struct tlbe stlbe;
+               int stlbsel, sesel;
+               u64 eaddr;
+               u64 raddr;
+
                switch (tlbsel) {
                case 0:
                        /* TLB0 */
@@ -634,7 +589,7 @@ int kvmppc_e500_emul_tlbwe(struct kvm_vcpu *vcpu)
                        gtlbe->mas1 |= MAS1_TSIZE(BOOK3E_PAGESZ_4K);
 
                        stlbsel = 0;
-                       sesel = kvmppc_e500_stlbe_map(vcpu_e500, 0, esel);
+                       sesel = kvmppc_e500_tlb0_map(vcpu_e500, esel, &stlbe);
 
                        break;
 
@@ -649,13 +604,13 @@ int kvmppc_e500_emul_tlbwe(struct kvm_vcpu *vcpu)
                         * are mapped on the fly. */
                        stlbsel = 1;
                        sesel = kvmppc_e500_tlb1_map(vcpu_e500, eaddr,
-                                       raddr >> PAGE_SHIFT, gtlbe);
+                                       raddr >> PAGE_SHIFT, gtlbe, &stlbe);
                        break;
 
                default:
                        BUG();
                }
-               write_host_tlbe(vcpu_e500, stlbsel, sesel);
+               write_host_tlbe(vcpu_e500, stlbsel, sesel, &stlbe);
        }
 
        kvmppc_set_exit_type(vcpu, EMULATED_TLBWE_EXITS);
@@ -695,7 +650,7 @@ gpa_t kvmppc_mmu_xlate(struct kvm_vcpu *vcpu, unsigned int index,
 {
        struct kvmppc_vcpu_e500 *vcpu_e500 = to_e500(vcpu);
        struct tlbe *gtlbe =
-               &vcpu_e500->guest_tlb[tlbsel_of(index)][esel_of(index)];
+               &vcpu_e500->gtlb_arch[tlbsel_of(index)][esel_of(index)];
        u64 pgmask = get_tlb_bytes(gtlbe) - 1;
 
        return get_tlb_raddr(gtlbe) | (eaddr & pgmask);
@@ -703,38 +658,36 @@ gpa_t kvmppc_mmu_xlate(struct kvm_vcpu *vcpu, unsigned int index,
 
 void kvmppc_mmu_destroy(struct kvm_vcpu *vcpu)
 {
-       struct kvmppc_vcpu_e500 *vcpu_e500 = to_e500(vcpu);
-       int tlbsel, i;
-
-       for (tlbsel = 0; tlbsel < 2; tlbsel++)
-               for (i = 0; i < vcpu_e500->guest_tlb_size[tlbsel]; i++)
-                       kvmppc_e500_shadow_release(vcpu_e500, tlbsel, i);
-
-       /* discard all guest mapping */
-       _tlbil_all();
 }
 
 void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 eaddr, gpa_t gpaddr,
                        unsigned int index)
 {
        struct kvmppc_vcpu_e500 *vcpu_e500 = to_e500(vcpu);
+       struct tlbe_priv *priv;
+       struct tlbe *gtlbe, stlbe;
        int tlbsel = tlbsel_of(index);
        int esel = esel_of(index);
        int stlbsel, sesel;
 
+       gtlbe = &vcpu_e500->gtlb_arch[tlbsel][esel];
+
        switch (tlbsel) {
        case 0:
                stlbsel = 0;
                sesel = esel;
+               priv = &vcpu_e500->gtlb_priv[stlbsel][sesel];
+
+               kvmppc_e500_setup_stlbe(vcpu_e500, gtlbe, BOOK3E_PAGESZ_4K,
+                                       priv, eaddr, &stlbe);
                break;
 
        case 1: {
                gfn_t gfn = gpaddr >> PAGE_SHIFT;
-               struct tlbe *gtlbe
-                       = &vcpu_e500->guest_tlb[tlbsel][esel];
 
                stlbsel = 1;
-               sesel = kvmppc_e500_tlb1_map(vcpu_e500, eaddr, gfn, gtlbe);
+               sesel = kvmppc_e500_tlb1_map(vcpu_e500, eaddr, gfn,
+                                            gtlbe, &stlbe);
                break;
        }
 
@@ -742,7 +695,8 @@ void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 eaddr, gpa_t gpaddr,
                BUG();
                break;
        }
-       write_host_tlbe(vcpu_e500, stlbsel, sesel);
+
+       write_host_tlbe(vcpu_e500, stlbsel, sesel, &stlbe);
 }
 
 int kvmppc_e500_tlb_search(struct kvm_vcpu *vcpu,
@@ -773,14 +727,14 @@ void kvmppc_e500_tlb_setup(struct kvmppc_vcpu_e500 *vcpu_e500)
        struct tlbe *tlbe;
 
        /* Insert large initial mapping for guest. */
-       tlbe = &vcpu_e500->guest_tlb[1][0];
+       tlbe = &vcpu_e500->gtlb_arch[1][0];
        tlbe->mas1 = MAS1_VALID | MAS1_TSIZE(BOOK3E_PAGESZ_256M);
        tlbe->mas2 = 0;
        tlbe->mas3 = E500_TLB_SUPER_PERM_MASK;
        tlbe->mas7 = 0;
 
        /* 4K map for serial output. Used by kernel wrapper. */
-       tlbe = &vcpu_e500->guest_tlb[1][1];
+       tlbe = &vcpu_e500->gtlb_arch[1][1];
        tlbe->mas1 = MAS1_VALID | MAS1_TSIZE(BOOK3E_PAGESZ_4K);
        tlbe->mas2 = (0xe0004500 & 0xFFFFF000) | MAS2_I | MAS2_G;
        tlbe->mas3 = (0xe0004500 & 0xFFFFF000) | E500_TLB_SUPER_PERM_MASK;
@@ -791,52 +745,61 @@ int kvmppc_e500_tlb_init(struct kvmppc_vcpu_e500 *vcpu_e500)
 {
        tlb1_entry_num = mfspr(SPRN_TLB1CFG) & 0xFFF;
 
-       vcpu_e500->guest_tlb_size[0] = KVM_E500_TLB0_SIZE;
-       vcpu_e500->guest_tlb[0] =
+       vcpu_e500->gtlb_size[0] = KVM_E500_TLB0_SIZE;
+       vcpu_e500->gtlb_arch[0] =
                kzalloc(sizeof(struct tlbe) * KVM_E500_TLB0_SIZE, GFP_KERNEL);
-       if (vcpu_e500->guest_tlb[0] == NULL)
+       if (vcpu_e500->gtlb_arch[0] == NULL)
                goto err_out;
 
-       vcpu_e500->shadow_tlb_size[0] = KVM_E500_TLB0_SIZE;
-       vcpu_e500->shadow_tlb[0] =
-               kzalloc(sizeof(struct tlbe) * KVM_E500_TLB0_SIZE, GFP_KERNEL);
-       if (vcpu_e500->shadow_tlb[0] == NULL)
-               goto err_out_guest0;
-
-       vcpu_e500->guest_tlb_size[1] = KVM_E500_TLB1_SIZE;
-       vcpu_e500->guest_tlb[1] =
+       vcpu_e500->gtlb_size[1] = KVM_E500_TLB1_SIZE;
+       vcpu_e500->gtlb_arch[1] =
                kzalloc(sizeof(struct tlbe) * KVM_E500_TLB1_SIZE, GFP_KERNEL);
-       if (vcpu_e500->guest_tlb[1] == NULL)
-               goto err_out_shadow0;
+       if (vcpu_e500->gtlb_arch[1] == NULL)
+               goto err_out_guest0;
 
-       vcpu_e500->shadow_tlb_size[1] = tlb1_entry_num;
-       vcpu_e500->shadow_tlb[1] =
-               kzalloc(sizeof(struct tlbe) * tlb1_entry_num, GFP_KERNEL);
-       if (vcpu_e500->shadow_tlb[1] == NULL)
+       vcpu_e500->gtlb_priv[0] = (struct tlbe_priv *)
+               kzalloc(sizeof(struct tlbe_priv) * KVM_E500_TLB0_SIZE, GFP_KERNEL);
+       if (vcpu_e500->gtlb_priv[0] == NULL)
                goto err_out_guest1;
+       vcpu_e500->gtlb_priv[1] = (struct tlbe_priv *)
+               kzalloc(sizeof(struct tlbe_priv) * KVM_E500_TLB1_SIZE, GFP_KERNEL);
+
+       if (vcpu_e500->gtlb_priv[1] == NULL)
+               goto err_out_priv0;
 
        /* Init TLB configuration register */
        vcpu_e500->tlb0cfg = mfspr(SPRN_TLB0CFG) & ~0xfffUL;
-       vcpu_e500->tlb0cfg |= vcpu_e500->guest_tlb_size[0];
+       vcpu_e500->tlb0cfg |= vcpu_e500->gtlb_size[0];
        vcpu_e500->tlb1cfg = mfspr(SPRN_TLB1CFG) & ~0xfffUL;
-       vcpu_e500->tlb1cfg |= vcpu_e500->guest_tlb_size[1];
+       vcpu_e500->tlb1cfg |= vcpu_e500->gtlb_size[1];
 
        return 0;
 
+err_out_priv0:
+       kfree(vcpu_e500->gtlb_priv[0]);
 err_out_guest1:
-       kfree(vcpu_e500->guest_tlb[1]);
-err_out_shadow0:
-       kfree(vcpu_e500->shadow_tlb[0]);
+       kfree(vcpu_e500->gtlb_arch[1]);
 err_out_guest0:
-       kfree(vcpu_e500->guest_tlb[0]);
+       kfree(vcpu_e500->gtlb_arch[0]);
 err_out:
        return -1;
 }
 
 void kvmppc_e500_tlb_uninit(struct kvmppc_vcpu_e500 *vcpu_e500)
 {
-       kfree(vcpu_e500->shadow_tlb[1]);
-       kfree(vcpu_e500->guest_tlb[1]);
-       kfree(vcpu_e500->shadow_tlb[0]);
-       kfree(vcpu_e500->guest_tlb[0]);
+       int stlbsel, i;
+
+       /* release all privs */
+       for (stlbsel = 0; stlbsel < 2; stlbsel++)
+               for (i = 0; i < vcpu_e500->gtlb_size[stlbsel]; i++) {
+                       struct tlbe_priv *priv =
+                               &vcpu_e500->gtlb_priv[stlbsel][i];
+                       kvmppc_e500_priv_release(priv);
+               }
+
+       /* discard all guest mapping */
+       _tlbil_all();
+
+       kfree(vcpu_e500->gtlb_arch[1]);
+       kfree(vcpu_e500->gtlb_arch[0]);
 }