mirror of https://github.com/ipxe/ipxe.git
[build] Prevent use of MMX and SSE registers
The existence of MMX and SSE is required by the System V x86_64 ABI and so is assumed by gcc, but these registers are not preserved by our own interrupt handlers and are unlikely to be preserved by other context switch handlers in a boot firmware environment. Explicitly prevent gcc from using MMX or SSE registers to avoid potential problems due to silent register corruption. We must remove the %xmm0-%xmm5 clobbers from the x86_64 version of hv_call() since otherwise gcc will complain about unknown register names. Theoretically, we should probably add code to explicitly preserve the %xmm0-%xmm5 registers across a hypercall, in order to guarantee to external code that these registers remain unchanged. In practice this is difficult since SSE registers are disabled by default: for background information see commitspull/71/head71560d1
("[librm] Preserve FPU, MMX and SSE state across calls to virt_call()") anddd9a14d
("[librm] Conditionalize the workaround for the Tivoli VMM's SSE garbling"). Signed-off-by: Michael Brown <mcb30@ipxe.org>
parent
0d35411f88
commit
1df3b53051
|
@ -13,6 +13,10 @@ CFLAGS += -m64
|
|||
ASFLAGS += --64
|
||||
LDFLAGS += -m elf_x86_64
|
||||
|
||||
# Prevent use of MMX and SSE registers
|
||||
#
|
||||
CFLAGS += -mno-mmx -mno-sse
|
||||
|
||||
# EFI requires -fshort-wchar, and nothing else currently uses wchar_t
|
||||
#
|
||||
CFLAGS += -fshort-wchar
|
||||
|
|
|
@ -44,8 +44,7 @@ hv_call ( struct hv_hypervisor *hv, unsigned int code, const void *in,
|
|||
: "=a" ( result ), "+r" ( rcx ), "+r" ( rdx ),
|
||||
"+r" ( r8 )
|
||||
: "m" ( hypercall )
|
||||
: "r9", "r10", "r11", "xmm0", "xmm1", "xmm2",
|
||||
"xmm3", "xmm4", "xmm5" );
|
||||
: "r9", "r10", "r11" );
|
||||
return result;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue