aboutsummaryrefslogtreecommitdiffstats
path: root/arch/i386/boot/pm.c
diff options
context:
space:
mode:
authorH. Peter Anvin <hpa@zytor.com>2007-07-14 19:47:13 -0400
committerH. Peter Anvin <hpa@zytor.com>2007-07-18 14:36:17 -0400
commit5593eaa854d0b23c3b270933a93b9b82946df729 (patch)
treef1efc4a66fcc25749db43eaa77e3075c65b2a1ab /arch/i386/boot/pm.c
parent9aa3909c0ea33da41755e15182fa4f88ae036d83 (diff)
[x86 setup] Fix assembly constraints
Fix incorrect assembly constraints. In particular, fix memory constraints used inside push..pop, which can cause invalid operation since gcc may generate %esp-relative references. Additionally: outl() should have "dN" not "dn". query_mca() shouldn't listen 16/32-bit registers in an 8-bit only context. has_eflag(): the "mask" is only used well after both the stack pointer and the output registers have been touched; this requires the output registers to be earlyclobbers (=&) and the input to exclude memory (so "ri", not "g"). Thanks to Etienne Lorrain and Chuck Ebbert for prompting this review. Cc: Etienne Lorrain <etienne_lorrain@yahoo.fr> Cc: Chuck Ebbert <cebbert@redhat.com> Signed-off-by: H. Peter Anvin <hpa@zytor.com>
Diffstat (limited to 'arch/i386/boot/pm.c')
-rw-r--r--arch/i386/boot/pm.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/i386/boot/pm.c b/arch/i386/boot/pm.c
index 3fa53e15ed77..1df025c73261 100644
--- a/arch/i386/boot/pm.c
+++ b/arch/i386/boot/pm.c
@@ -65,7 +65,7 @@ static void move_kernel_around(void)
65 "popw %%ds ; " 65 "popw %%ds ; "
66 "popw %%es" 66 "popw %%es"
67 : "+c" (dwords) 67 : "+c" (dwords)
68 : "rm" (dst_seg), "rm" (src_seg) 68 : "r" (dst_seg), "r" (src_seg)
69 : "esi", "edi"); 69 : "esi", "edi");
70 70
71 syssize -= paras; 71 syssize -= paras;