diff options
author | Avi Kivity <avi@redhat.com> | 2010-11-29 09:12:30 -0500 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2011-01-12 04:30:02 -0500 |
commit | 6389ee946303cb4313dba0a49865e495a53351ff (patch) | |
tree | 478c8f5f4d9f75d77808d648adbc8948050ffee7 /arch/x86/kvm/mmu.c | |
parent | 8c28d031657adaf238634ac406834a839d7fa43d (diff) |
KVM: Pull extra page fault information into struct x86_exception
Currently page fault cr2 and nesting infomation are carried outside
the fault data structure. Instead they are placed in the vcpu struct,
which results in confusion as global variables are manipulated instead
of passing parameters.
Fix this issue by adding address and nested fields to struct x86_exception,
so this struct can carry all information associated with a fault.
Signed-off-by: Avi Kivity <avi@redhat.com>
Tested-by: Joerg Roedel <joerg.roedel@amd.com>
Tested-by: Gleb Natapov <gleb@redhat.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Diffstat (limited to 'arch/x86/kvm/mmu.c')
-rw-r--r-- | arch/x86/kvm/mmu.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c index 9ce041469a8e..d35950087e3d 100644 --- a/arch/x86/kvm/mmu.c +++ b/arch/x86/kvm/mmu.c | |||
@@ -2736,9 +2736,10 @@ static unsigned long get_cr3(struct kvm_vcpu *vcpu) | |||
2736 | return vcpu->arch.cr3; | 2736 | return vcpu->arch.cr3; |
2737 | } | 2737 | } |
2738 | 2738 | ||
2739 | static void inject_page_fault(struct kvm_vcpu *vcpu) | 2739 | static void inject_page_fault(struct kvm_vcpu *vcpu, |
2740 | struct x86_exception *fault) | ||
2740 | { | 2741 | { |
2741 | vcpu->arch.mmu.inject_page_fault(vcpu); | 2742 | vcpu->arch.mmu.inject_page_fault(vcpu, fault); |
2742 | } | 2743 | } |
2743 | 2744 | ||
2744 | static void paging_free(struct kvm_vcpu *vcpu) | 2745 | static void paging_free(struct kvm_vcpu *vcpu) |