diff options
author | Avi Kivity <avi@redhat.com> | 2010-07-29 08:11:50 -0400 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2010-10-24 04:50:21 -0400 |
commit | 9aabc88fc8687ba3a520e2ec459821d05f72474e (patch) | |
tree | 6e57c011a783af6a8a0e2ca92e3f55eb4d2701ae /arch/x86/kvm/x86.c | |
parent | ab85b12b1a7fd125588f9447653a71ec8e1b5024 (diff) |
KVM: x86 emulator: store x86_emulate_ops in emulation context
It doesn't ever change, so we don't need to pass it around everywhere.
Signed-off-by: Avi Kivity <avi@redhat.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Diffstat (limited to 'arch/x86/kvm/x86.c')
-rw-r--r-- | arch/x86/kvm/x86.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 3a09c625d526..33deb75f16ee 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c | |||
@@ -3998,7 +3998,7 @@ int emulate_instruction(struct kvm_vcpu *vcpu, | |||
3998 | vcpu->arch.emulate_ctxt.interruptibility = 0; | 3998 | vcpu->arch.emulate_ctxt.interruptibility = 0; |
3999 | vcpu->arch.emulate_ctxt.exception = -1; | 3999 | vcpu->arch.emulate_ctxt.exception = -1; |
4000 | 4000 | ||
4001 | r = x86_decode_insn(&vcpu->arch.emulate_ctxt, &emulate_ops); | 4001 | r = x86_decode_insn(&vcpu->arch.emulate_ctxt); |
4002 | trace_kvm_emulate_insn_start(vcpu); | 4002 | trace_kvm_emulate_insn_start(vcpu); |
4003 | 4003 | ||
4004 | /* Only allow emulation of specific instructions on #UD | 4004 | /* Only allow emulation of specific instructions on #UD |
@@ -4048,7 +4048,7 @@ int emulate_instruction(struct kvm_vcpu *vcpu, | |||
4048 | memcpy(c->regs, vcpu->arch.regs, sizeof c->regs); | 4048 | memcpy(c->regs, vcpu->arch.regs, sizeof c->regs); |
4049 | 4049 | ||
4050 | restart: | 4050 | restart: |
4051 | r = x86_emulate_insn(&vcpu->arch.emulate_ctxt, &emulate_ops); | 4051 | r = x86_emulate_insn(&vcpu->arch.emulate_ctxt); |
4052 | 4052 | ||
4053 | if (r) { /* emulation failed */ | 4053 | if (r) { /* emulation failed */ |
4054 | if (reexecute_instruction(vcpu, cr2)) | 4054 | if (reexecute_instruction(vcpu, cr2)) |
@@ -5067,7 +5067,7 @@ int kvm_task_switch(struct kvm_vcpu *vcpu, u16 tss_selector, int reason, | |||
5067 | memset(c, 0, sizeof(struct decode_cache)); | 5067 | memset(c, 0, sizeof(struct decode_cache)); |
5068 | memcpy(c->regs, vcpu->arch.regs, sizeof c->regs); | 5068 | memcpy(c->regs, vcpu->arch.regs, sizeof c->regs); |
5069 | 5069 | ||
5070 | ret = emulator_task_switch(&vcpu->arch.emulate_ctxt, &emulate_ops, | 5070 | ret = emulator_task_switch(&vcpu->arch.emulate_ctxt, |
5071 | tss_selector, reason, has_error_code, | 5071 | tss_selector, reason, has_error_code, |
5072 | error_code); | 5072 | error_code); |
5073 | 5073 | ||
@@ -5424,6 +5424,7 @@ int kvm_arch_vcpu_init(struct kvm_vcpu *vcpu) | |||
5424 | BUG_ON(vcpu->kvm == NULL); | 5424 | BUG_ON(vcpu->kvm == NULL); |
5425 | kvm = vcpu->kvm; | 5425 | kvm = vcpu->kvm; |
5426 | 5426 | ||
5427 | vcpu->arch.emulate_ctxt.ops = &emulate_ops; | ||
5427 | vcpu->arch.mmu.root_hpa = INVALID_PAGE; | 5428 | vcpu->arch.mmu.root_hpa = INVALID_PAGE; |
5428 | if (!irqchip_in_kernel(kvm) || kvm_vcpu_is_bsp(vcpu)) | 5429 | if (!irqchip_in_kernel(kvm) || kvm_vcpu_is_bsp(vcpu)) |
5429 | vcpu->arch.mp_state = KVM_MP_STATE_RUNNABLE; | 5430 | vcpu->arch.mp_state = KVM_MP_STATE_RUNNABLE; |