diff options
author | Avi Kivity <avi@qumranet.com> | 2008-03-16 12:48:26 -0400 |
---|---|---|
committer | Avi Kivity <avi@qumranet.com> | 2008-03-25 04:22:16 -0400 |
commit | 5dc832628229d2736fab10523566855c3cda622d (patch) | |
tree | 9f2024d364073ba42a6fed317a5e66dd9999e28d /arch/x86/kvm | |
parent | a4083c9271e0a697278e089f2c0b9a95363ada0a (diff) |
KVM: VMX: Restore tss even on x86_64
The vmx hardware state restore restores the tss selector and base address, but
not its length. Usually, this does not matter since most of the tss contents
is within the default length of 0x67. However, if a process is using ioperm()
to grant itself I/O port permissions, an additional bitmap within the tss,
but outside the default length is consulted. The effect is that the process
will receive a SIGSEGV instead of transparently accessing the port.
Fix by restoring the tss length. Note that i386 had this working already.
Closes bugzilla 10246.
Signed-off-by: Avi Kivity <avi@qumranet.com>
Diffstat (limited to 'arch/x86/kvm')
-rw-r--r-- | arch/x86/kvm/vmx.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c index 94ea724638fd..f2df03c12f16 100644 --- a/arch/x86/kvm/vmx.c +++ b/arch/x86/kvm/vmx.c | |||
@@ -349,8 +349,6 @@ static void update_exception_bitmap(struct kvm_vcpu *vcpu) | |||
349 | 349 | ||
350 | static void reload_tss(void) | 350 | static void reload_tss(void) |
351 | { | 351 | { |
352 | #ifndef CONFIG_X86_64 | ||
353 | |||
354 | /* | 352 | /* |
355 | * VT restores TR but not its size. Useless. | 353 | * VT restores TR but not its size. Useless. |
356 | */ | 354 | */ |
@@ -361,7 +359,6 @@ static void reload_tss(void) | |||
361 | descs = (void *)gdt.base; | 359 | descs = (void *)gdt.base; |
362 | descs[GDT_ENTRY_TSS].type = 9; /* available TSS */ | 360 | descs[GDT_ENTRY_TSS].type = 9; /* available TSS */ |
363 | load_TR_desc(); | 361 | load_TR_desc(); |
364 | #endif | ||
365 | } | 362 | } |
366 | 363 | ||
367 | static void load_transition_efer(struct vcpu_vmx *vmx) | 364 | static void load_transition_efer(struct vcpu_vmx *vmx) |