diff options
| author |   Glauber Costa <gcosta@redhat.com> | 2008-06-10 10:46:53 -0300 |
|---|---|---|
| committer |   Avi Kivity <avi@qumranet.com> | 2008-07-20 12:42:32 +0300 |
| commit | 25be46080f1a446cb2bda3daadbd22a5682b955e (patch) | |
| tree | d736f1c5982d4eefa5da40dac5d8e8074dfa0a76 | |
| parent | KVM: only abort guest entry if timer count goes from 0->1 (diff) | |
| download | linux-dev-25be46080f1a446cb2bda3daadbd22a5682b955e.tar.xz linux-dev-25be46080f1a446cb2bda3daadbd22a5682b955e.zip | |
KVM: Do not calculate linear rip in emulation failure report
If we're not gonna do anything (case in which failure is already
reported), we do not need to even bother with calculating the linear rip.
Signed-off-by: Glauber Costa <gcosta@redhat.com>
Signed-off-by: Avi Kivity <avi@qumranet.com>
| -rw-r--r-- | arch/x86/kvm/x86.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index d731d4fff1ae..5d21bb69d88c 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -2081,11 +2081,11 @@ void kvm_report_emulation_failure(struct kvm_vcpu *vcpu, const char *context) unsigned long rip = vcpu->arch.rip; unsigned long rip_linear; - rip_linear = rip + get_segment_base(vcpu, VCPU_SREG_CS); - if (reported) return; + rip_linear = rip + get_segment_base(vcpu, VCPU_SREG_CS); + emulator_read_std(rip_linear, (void *)opcodes, 4, vcpu); printk(KERN_ERR "emulation failed (%s) rip %lx %02x %02x %02x %02x\n", |