Skip to content

Commit f0b97e1

Browse files
kraxelmergify[bot]
authored andcommitted
Revert "OvmfPkg/Sec: fix stack switch"
This reverts commit ff36b25. Has no effect because GCC_IA32_CC_FLAGS and GCC_X64_CC_FLAGS are unused. Signed-off-by: Gerd Hoffmann <[email protected]> Reviewed-by: Jiewen Yao <[email protected]>
1 parent ccc2697 commit f0b97e1

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

BaseTools/Conf/tools_def.template

+3-3
Original file line numberDiff line numberDiff line change
@@ -1849,9 +1849,9 @@ NOOPT_*_*_OBJCOPY_ADDDEBUGFLAG = --add-gnu-debuglink=$(DEBUG_DIR)/$(MODULE_N
18491849
*_*_*_DTC_PATH = DEF(DTC_BIN)
18501850

18511851
DEFINE GCC_ALL_CC_FLAGS = -g -Os -fshort-wchar -fno-builtin -fno-strict-aliasing -Wall -Werror -Wno-array-bounds -include AutoGen.h -fno-common
1852-
DEFINE GCC_IA32_CC_FLAGS = DEF(GCC_ALL_CC_FLAGS) -m32 -malign-double -freorder-blocks -freorder-blocks-and-partition -O2 -mno-stack-arg-probe -fno-omit-frame-pointer
1853-
DEFINE GCC_X64_CC_FLAGS = DEF(GCC_ALL_CC_FLAGS) -mno-red-zone -Wno-address -mno-stack-arg-probe -fomit-frame-pointer -DOMIT_FRAME_POINTER=1
1854-
DEFINE GCC_ARM_CC_FLAGS = DEF(GCC_ALL_CC_FLAGS) -mlittle-endian -mabi=aapcs -fno-short-enums -funsigned-char -ffunction-sections -fdata-sections -fomit-frame-pointer -DOMIT_FRAME_POINTER=1 -Wno-address -mthumb -mfloat-abi=soft -fno-pic -fno-pie
1852+
DEFINE GCC_IA32_CC_FLAGS = DEF(GCC_ALL_CC_FLAGS) -m32 -malign-double -freorder-blocks -freorder-blocks-and-partition -O2 -mno-stack-arg-probe
1853+
DEFINE GCC_X64_CC_FLAGS = DEF(GCC_ALL_CC_FLAGS) -mno-red-zone -Wno-address -mno-stack-arg-probe
1854+
DEFINE GCC_ARM_CC_FLAGS = DEF(GCC_ALL_CC_FLAGS) -mlittle-endian -mabi=aapcs -fno-short-enums -funsigned-char -ffunction-sections -fdata-sections -fomit-frame-pointer -Wno-address -mthumb -mfloat-abi=soft -fno-pic -fno-pie
18551855
DEFINE GCC_ARM_CC_XIPFLAGS = -mno-unaligned-access
18561856
DEFINE GCC_AARCH64_CC_FLAGS = DEF(GCC_ALL_CC_FLAGS) -mlittle-endian -fno-short-enums -fverbose-asm -funsigned-char -ffunction-sections -fdata-sections -Wno-address -fno-asynchronous-unwind-tables -fno-unwind-tables -fno-pic -fno-pie -ffixed-x18
18571857
DEFINE GCC_AARCH64_CC_XIPFLAGS = -mstrict-align -mgeneral-regs-only

OvmfPkg/Sec/SecMain.c

-4
Original file line numberDiff line numberDiff line change
@@ -1052,15 +1052,11 @@ TemporaryRamMigration (
10521052
if (SetJump (&JumpBuffer) == 0) {
10531053
#if defined (MDE_CPU_IA32)
10541054
JumpBuffer.Esp = JumpBuffer.Esp + DebugAgentContext.StackMigrateOffset;
1055-
#ifndef OMIT_FRAME_POINTER
10561055
JumpBuffer.Ebp = JumpBuffer.Ebp + DebugAgentContext.StackMigrateOffset;
10571056
#endif
1058-
#endif
10591057
#if defined (MDE_CPU_X64)
10601058
JumpBuffer.Rsp = JumpBuffer.Rsp + DebugAgentContext.StackMigrateOffset;
1061-
#ifndef OMIT_FRAME_POINTER
10621059
JumpBuffer.Rbp = JumpBuffer.Rbp + DebugAgentContext.StackMigrateOffset;
1063-
#endif
10641060
#endif
10651061
LongJump (&JumpBuffer, (UINTN)-1);
10661062
}

0 commit comments

Comments
 (0)