|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [linux-2.6.18-xen] x86-64: fix HYPERVISOR_iret unwind annotation
# HG changeset patch
# User Jan Beulich <jbeulich@xxxxxxxx>
# Date 1362388145 -3600
# Node ID 805186ebb4ab418f1ff968a634a59d7658ba126c
# Parent 28e95bbec82172230b27c9cdb8bfa8aa0d342bbc
x86-64: fix HYPERVISOR_iret unwind annotation
Minimally for %rax this needs to reflect that the value is no longer
saved: Since the whole hypercall page is covered by a single FDE,
subsequent entries would otherwise retain the state of %rax being saved
on the stack, yet the offset it's being saved at is out of bounds for
the "normal" hypercall stubs.
Breaking up the single FDE into per stub ones would increase the size
of the resulting .eh_frame section, which is already quite big for this
simple a piece of code.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
---
diff -r 28e95bbec821 -r 805186ebb4ab arch/x86_64/kernel/head-xen.S
--- a/arch/x86_64/kernel/head-xen.S Wed Feb 27 11:21:03 2013 +0100
+++ b/arch/x86_64/kernel/head-xen.S Mon Mar 04 10:09:05 2013 +0100
@@ -98,6 +98,9 @@ NEXT_PAGE(hypercall_page)
.skip 2 /* syscall */
.if i == __HYPERVISOR_iret
CFI_ADJUST_CFA_OFFSET -3*8
+ CFI_SAME_VALUE rax
+ CFI_SAME_VALUE r11
+ CFI_SAME_VALUE rcx
.else
.skip 2 /* pop %r11 */
CFI_ADJUST_CFA_OFFSET -8
diff -r 28e95bbec821 -r 805186ebb4ab include/asm-x86_64/dwarf2.h
--- a/include/asm-x86_64/dwarf2.h Wed Feb 27 11:21:03 2013 +0100
+++ b/include/asm-x86_64/dwarf2.h Mon Mar 04 10:09:05 2013 +0100
@@ -28,6 +28,7 @@
#define CFI_REMEMBER_STATE .cfi_remember_state
#define CFI_RESTORE_STATE .cfi_restore_state
#define CFI_UNDEFINED .cfi_undefined
+#define CFI_SAME_VALUE .cfi_same_value
#else
@@ -45,6 +46,7 @@
#define CFI_REMEMBER_STATE #
#define CFI_RESTORE_STATE #
#define CFI_UNDEFINED #
+#define CFI_SAME_VALUE #
#endif
_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |