diff options
author | H. Peter Anvin <hpa@zytor.com> | 2007-07-14 19:47:13 -0400 |
---|---|---|
committer | H. Peter Anvin <hpa@zytor.com> | 2007-07-18 14:36:17 -0400 |
commit | 5593eaa854d0b23c3b270933a93b9b82946df729 (patch) | |
tree | f1efc4a66fcc25749db43eaa77e3075c65b2a1ab /arch/i386/boot/pm.c | |
parent | 9aa3909c0ea33da41755e15182fa4f88ae036d83 (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.c | 2 |
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; |