[SeaBIOS] [PATCH 2/2] Eliminate FUNCFSEG - only force portions of inline asm to f-segment
Kevin O'Connor
kevin at koconnor.net
Wed Dec 3 19:01:48 CET 2014
The FUNCFSEG macro was introduced to force a C function into the
f-segment. This was needed for some C functions that used inline
assembler that contained some 16bit code. Instead of forcing the
entire C function into the f-segment, just force the small subset of
inline assembler into the f-segment.
Signed-off-by: Kevin O'Connor <kevin at koconnor.net>
---
src/stacks.c | 12 ++++++------
src/types.h | 4 ----
2 files changed, 6 insertions(+), 10 deletions(-)
diff --git a/src/stacks.c b/src/stacks.c
index c6b7e8b..b2e2447 100644
--- a/src/stacks.c
+++ b/src/stacks.c
@@ -63,8 +63,8 @@ call32_smm_post(void)
inb(PORT_CMOS_DATA);
}
-#define ASM32_SWITCH16 " .code16\n"
-#define ASM32_BACK32 " .code32\n"
+#define ASM32_SWITCH16 " .pushsection .text.fseg." UNIQSEC "\n .code16\n"
+#define ASM32_BACK32 " .popsection\n .code32\n"
#define ASM16_SWITCH32 " .code32\n"
#define ASM16_BACK16 " .code16gcc\n"
@@ -127,7 +127,7 @@ call16_smm_helper(u32 eax, u32 edx, u32 (*func)(u32 eax, u32 edx))
return ret;
}
-u32 FUNCFSEG
+static u32
call16_smm(u32 eax, u32 edx, void *func)
{
ASSERT32FLAT();
@@ -265,7 +265,7 @@ call16_sloppy_helper(u32 eax, u32 edx, u32 (*func)(u32 eax, u32 edx))
}
// Jump back to 16bit mode while in 32bit mode from call32_sloppy()
-u32 FUNCFSEG
+static u32
call16_sloppy(u32 eax, u32 edx, void *func)
{
ASSERT32FLAT();
@@ -313,7 +313,7 @@ call32(void *func, u32 eax, u32 errret)
}
// Call a 16bit SeaBIOS function from a 32bit SeaBIOS function.
-u32 FUNCFSEG
+static u32
call16(u32 eax, u32 edx, void *func)
{
ASSERT32FLAT();
@@ -340,7 +340,7 @@ call16(u32 eax, u32 edx, void *func)
}
// Call a 16bit SeaBIOS function in "big real" mode.
-u32 FUNCFSEG
+static u32
call16big(u32 eax, u32 edx, void *func)
{
ASSERT32FLAT();
diff --git a/src/types.h b/src/types.h
index 6dd8c43..097372c 100644
--- a/src/types.h
+++ b/src/types.h
@@ -70,8 +70,6 @@ extern void __force_link_error__only_in_16bit(void) __noreturn;
# define VARFSEG __section(".discard.varfseg." UNIQSEC) __VISIBLE __weak
// Designate a variable at a specific address in the f-segment.
# define VARFSEGFIXED(addr) __section(".discard.varfixed." UNIQSEC) __VISIBLE __weak
-// Notes a 32bit flat function that must reside in the f-segment.
-# define FUNCFSEG __section(".discard.32fseg." UNIQSEC) __VISIBLE __weak
// Verify a variable is only accessable via 32bit "init" functions
# define VARVERIFY32INIT __section(".discard.varinit." UNIQSEC)
// Designate top-level assembler as 16bit only.
@@ -92,7 +90,6 @@ extern void __force_link_error__only_in_16bit(void) __noreturn;
# define VARLOW __section(".discard.varlow." UNIQSEC) __VISIBLE __weak
# define VARFSEG __section(".discard.varfseg." UNIQSEC) __VISIBLE __weak
# define VARFSEGFIXED(addr) __section(".discard.varfixed." UNIQSEC) __VISIBLE __weak
-# define FUNCFSEG __section(".discard.32fseg." UNIQSEC) __VISIBLE __weak
# define VARVERIFY32INIT __section(".discard.varinit." UNIQSEC)
# define ASM16(code)
# define ASM32FLAT(code)
@@ -109,7 +106,6 @@ extern void __force_link_error__only_in_16bit(void) __noreturn;
# define VARLOW __section(".data.varlow." UNIQSEC) __VISIBLE __weak
# define VARFSEG __section(".data.varfseg." UNIQSEC) __VISIBLE
# define VARFSEGFIXED(addr) __section(".fixedaddr." __stringify(addr)) __VISIBLE __aligned(1)
-# define FUNCFSEG __section(".text.32fseg." UNIQSEC) __VISIBLE
# define VARVERIFY32INIT __section(".data.varinit." UNIQSEC)
# define ASM16(code)
# define ASM32FLAT(code) __ASM(code)
--
1.9.3
More information about the SeaBIOS
mailing list