|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH v6.5 07/26] x86/hvm: Use SAVE_ALL to construct the cpu_user_regs frame after VMExit
No practical change.
One side effect in debug builds is that %rbp is inverted in the manner
expected by the stack unwinder to indicate a interrupt frame.
Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Wei Liu <wei.liu2@xxxxxxxxxx>
---
xen/arch/x86/hvm/svm/entry.S | 22 ++++------------------
xen/arch/x86/hvm/vmx/entry.S | 17 ++---------------
2 files changed, 6 insertions(+), 33 deletions(-)
diff --git a/xen/arch/x86/hvm/svm/entry.S b/xen/arch/x86/hvm/svm/entry.S
index 4a72e38..df86da0 100644
--- a/xen/arch/x86/hvm/svm/entry.S
+++ b/xen/arch/x86/hvm/svm/entry.S
@@ -98,24 +98,10 @@ UNLIKELY_END(svm_trace)
VMRUN
- GET_CURRENT(ax)
- push %rdi
- push %rsi
- push %rdx
- push %rcx
- mov VCPU_svm_vmcb(%rax),%rcx
- push %rax
- push %r8
- push %r9
- push %r10
- push %r11
- push %rbx
- mov %rax,%rbx
- push %rbp
- push %r12
- push %r13
- push %r14
- push %r15
+ SAVE_ALL
+
+ GET_CURRENT(bx)
+ mov VCPU_svm_vmcb(%rbx),%rcx
movb $0,VCPU_svm_vmcb_in_sync(%rbx)
mov VMCB_rax(%rcx),%rax
diff --git a/xen/arch/x86/hvm/vmx/entry.S b/xen/arch/x86/hvm/vmx/entry.S
index 47cd674..b2f98be 100644
--- a/xen/arch/x86/hvm/vmx/entry.S
+++ b/xen/arch/x86/hvm/vmx/entry.S
@@ -30,23 +30,10 @@
#define VMLAUNCH .byte 0x0f,0x01,0xc2
ENTRY(vmx_asm_vmexit_handler)
- push %rdi
- push %rsi
- push %rdx
- push %rcx
- push %rax
+ SAVE_ALL
+
mov %cr2,%rax
- push %r8
- push %r9
- push %r10
- push %r11
- push %rbx
GET_CURRENT(bx)
- push %rbp
- push %r12
- push %r13
- push %r14
- push %r15
movb $1,VCPU_vmx_launched(%rbx)
mov %rax,VCPU_hvm_guest_cr2(%rbx)
--
2.1.4
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |