Commit 1e301feb authored by Joerg Roedel's avatar Joerg Roedel Committed by Avi Kivity

KVM: MMU: Introduce generic walk_addr function

This is the first patch in the series towards a generic
walk_addr implementation which could walk two-dimensional
page tables in the end. In this first step the walk_addr
function is renamed into walk_addr_generic which takes a
mmu context as an additional parameter.
Signed-off-by: default avatarJoerg Roedel <joerg.roedel@amd.com>
Signed-off-by: default avatarAvi Kivity <avi@redhat.com>
parent 8df25a32
...@@ -114,9 +114,10 @@ static unsigned FNAME(gpte_access)(struct kvm_vcpu *vcpu, pt_element_t gpte) ...@@ -114,9 +114,10 @@ static unsigned FNAME(gpte_access)(struct kvm_vcpu *vcpu, pt_element_t gpte)
/* /*
* Fetch a guest pte for a guest virtual address * Fetch a guest pte for a guest virtual address
*/ */
static int FNAME(walk_addr)(struct guest_walker *walker, static int FNAME(walk_addr_generic)(struct guest_walker *walker,
struct kvm_vcpu *vcpu, gva_t addr, struct kvm_vcpu *vcpu, struct kvm_mmu *mmu,
int write_fault, int user_fault, int fetch_fault) gva_t addr, int write_fault,
int user_fault, int fetch_fault)
{ {
pt_element_t pte; pt_element_t pte;
gfn_t table_gfn; gfn_t table_gfn;
...@@ -129,10 +130,11 @@ static int FNAME(walk_addr)(struct guest_walker *walker, ...@@ -129,10 +130,11 @@ static int FNAME(walk_addr)(struct guest_walker *walker,
walk: walk:
present = true; present = true;
eperm = rsvd_fault = false; eperm = rsvd_fault = false;
walker->level = vcpu->arch.mmu.root_level; walker->level = mmu->root_level;
pte = vcpu->arch.mmu.get_cr3(vcpu); pte = mmu->get_cr3(vcpu);
#if PTTYPE == 64 #if PTTYPE == 64
if (vcpu->arch.mmu.root_level == PT32E_ROOT_LEVEL) { if (walker->level == PT32E_ROOT_LEVEL) {
pte = kvm_pdptr_read(vcpu, (addr >> 30) & 3); pte = kvm_pdptr_read(vcpu, (addr >> 30) & 3);
trace_kvm_mmu_paging_element(pte, walker->level); trace_kvm_mmu_paging_element(pte, walker->level);
if (!is_present_gpte(pte)) { if (!is_present_gpte(pte)) {
...@@ -143,7 +145,7 @@ static int FNAME(walk_addr)(struct guest_walker *walker, ...@@ -143,7 +145,7 @@ static int FNAME(walk_addr)(struct guest_walker *walker,
} }
#endif #endif
ASSERT((!is_long_mode(vcpu) && is_pae(vcpu)) || ASSERT((!is_long_mode(vcpu) && is_pae(vcpu)) ||
(vcpu->arch.mmu.get_cr3(vcpu) & CR3_NONPAE_RESERVED_BITS) == 0); (mmu->get_cr3(vcpu) & CR3_NONPAE_RESERVED_BITS) == 0);
pt_access = ACC_ALL; pt_access = ACC_ALL;
...@@ -205,7 +207,7 @@ static int FNAME(walk_addr)(struct guest_walker *walker, ...@@ -205,7 +207,7 @@ static int FNAME(walk_addr)(struct guest_walker *walker,
(PTTYPE == 64 || is_pse(vcpu))) || (PTTYPE == 64 || is_pse(vcpu))) ||
((walker->level == PT_PDPE_LEVEL) && ((walker->level == PT_PDPE_LEVEL) &&
is_large_pte(pte) && is_large_pte(pte) &&
vcpu->arch.mmu.root_level == PT64_ROOT_LEVEL)) { mmu->root_level == PT64_ROOT_LEVEL)) {
int lvl = walker->level; int lvl = walker->level;
walker->gfn = gpte_to_gfn_lvl(pte, lvl); walker->gfn = gpte_to_gfn_lvl(pte, lvl);
...@@ -266,6 +268,14 @@ static int FNAME(walk_addr)(struct guest_walker *walker, ...@@ -266,6 +268,14 @@ static int FNAME(walk_addr)(struct guest_walker *walker,
return 0; return 0;
} }
static int FNAME(walk_addr)(struct guest_walker *walker,
struct kvm_vcpu *vcpu, gva_t addr,
int write_fault, int user_fault, int fetch_fault)
{
return FNAME(walk_addr_generic)(walker, vcpu, &vcpu->arch.mmu, addr,
write_fault, user_fault, fetch_fault);
}
static void FNAME(update_pte)(struct kvm_vcpu *vcpu, struct kvm_mmu_page *sp, static void FNAME(update_pte)(struct kvm_vcpu *vcpu, struct kvm_mmu_page *sp,
u64 *spte, const void *pte) u64 *spte, const void *pte)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment