Re: [PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Alexander Graf
On 18.01.2013, at 01:47, Scott Wood wrote: > On 01/17/2013 06:20:03 PM, Alexander Graf wrote: >> On 18.01.2013, at 01:11, Scott Wood wrote: >> > On 01/17/2013 04:50:39 PM, Alexander Graf wrote: >> >> @@ -1024,9 +1001,11 @@ void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 >> >> eaddr, gpa_t gpaddr,

Re: [PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Alexander Graf
On 18.01.2013, at 01:47, Scott Wood wrote: > On 01/17/2013 06:20:03 PM, Alexander Graf wrote: >> On 18.01.2013, at 01:11, Scott Wood wrote: >> > On 01/17/2013 04:50:39 PM, Alexander Graf wrote: >> >> @@ -1024,9 +1001,11 @@ void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 >> >> eaddr, gpa_t gpaddr,

Re: [PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Scott Wood
On 01/17/2013 06:20:03 PM, Alexander Graf wrote: On 18.01.2013, at 01:11, Scott Wood wrote: > On 01/17/2013 04:50:39 PM, Alexander Graf wrote: >> @@ -1024,9 +1001,11 @@ void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 eaddr, gpa_t gpaddr, >> { >>struct kvmppc_vcpu_e500 *vcpu_e500 = to_

Re: [PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Scott Wood
On 01/17/2013 06:29:56 PM, Alexander Graf wrote: On 18.01.2013, at 01:20, Alexander Graf wrote: > > On 18.01.2013, at 01:11, Scott Wood wrote: > >> It also seems like it would be cleaner to just invalidate the old entry >> in tlbwe, and then this function doesn't need to change at all. I a

Re: [PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Alexander Graf
On 18.01.2013, at 01:20, Alexander Graf wrote: > > On 18.01.2013, at 01:11, Scott Wood wrote: > >> On 01/17/2013 04:50:39 PM, Alexander Graf wrote: >>> @@ -1024,9 +1001,11 @@ void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 >>> eaddr, gpa_t gpaddr, >>> { >>> struct kvmppc_vcpu_e500 *vcpu_e50

Re: [PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Alexander Graf
On 18.01.2013, at 01:11, Scott Wood wrote: > On 01/17/2013 04:50:39 PM, Alexander Graf wrote: >> @@ -1024,9 +1001,11 @@ void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 eaddr, >> gpa_t gpaddr, >> { >> struct kvmppc_vcpu_e500 *vcpu_e500 = to_e500(vcpu); >> struct tlbe_priv *priv; >> -

Re: [PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Scott Wood
On 01/17/2013 04:50:39 PM, Alexander Graf wrote: @@ -1024,9 +1001,11 @@ void kvmppc_mmu_map(struct kvm_vcpu *vcpu, u64 eaddr, gpa_t gpaddr, { struct kvmppc_vcpu_e500 *vcpu_e500 = to_e500(vcpu); struct tlbe_priv *priv; - struct kvm_book3e_206_tlb_entry *gtlbe, stlbe; +

[PATCH 1/3] KVM: PPC: e500: Call kvmppc_mmu_map for initial mapping

2013-01-17 Thread Alexander Graf
When emulating tlbwe, we want to automatically map the entry that just got written in our shadow TLB map, because chances are quite high that it's going to be used very soon. Today this happens explicitly, duplicating all the logic that is in kvmppc_mmu_map() already. Just call that one instead.