Cleanup implementation of call16().

diff --git a/TODO b/TODO
index d8d0128a75c19efea7990ea80f2e4214c3c3a522..08aaf91d8b55fc9b80eeb373852b161ebc9719b1 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,7 +1,5 @@
 Find out why ubuntu compiles are failing.  Find work around.
 
-Fixup __call16 usage from util.h / rombios32.lds.S
-
 See if it is better to tell gcc that call16 clobbers all registers
 instead of having the code call pushal/popal.
 

diff --git a/src/rombios32.lds.S b/src/rombios32.lds.S
index 5d7e77247eb17e34c5b46e189164c6f51c94b3cd..78546cf30e771e490cb290469f456684e50ba5f6 100644 (file)
--- a/src/rombios32.lds.S
+++ b/src/rombios32.lds.S
@@ -23,7 +23,7 @@ SECTIONS
         __bss_start = . ;
         .bss      : { *(.bss) *(COMMON) }
         _end = . ;
-        __call16 = (0xf0000 | OFFSET___call16_from32) ;
+        __call16_from32 = (0xf0000 | OFFSET___call16_from32) ;
         /DISCARD/ : { *(.stab)
                      *(.stabstr)
                      *(.comment)

diff --git a/src/util.h b/src/util.h
index cdb8b1bd8429116b9c9205be4ec3827cf650ff39..7de577bb0f9e6ec97f028f3d92759d7c0679688d 100644 (file)
--- a/src/util.h
+++ b/src/util.h
@@ -67,29 +67,33 @@ void call16(struct bregs *callregs)
     asm volatile(
         "pushfl\n"   // Save flags
         "pushal\n"   // Save registers
+#ifdef MODE16
         "calll __call16\n"
+#else
+        "calll __call16_from32\n"
+#endif
         "popal\n"
         "popfl\n"
         : : "a" (callregs), "m" (*callregs));
 }
 
-// XXX - this is ugly.
+static inline
+void __call16_int(struct bregs *callregs, u16 offset)
+{
+    callregs->cs = 0xf000;
+    callregs->ip = offset;
+    call16(callregs);
+}
+
 #ifdef MODE16
-#define call16_int(nr, callregs) do {           \
-        struct bregs *__br = (callregs);        \
-        extern void irq_trampoline_ ##nr ();    \
-        __br->cs = 0xf000;                      \
-        __br->ip = (u16)&irq_trampoline_ ##nr;  \
-        call16(__br);                           \
+#define call16_int(nr, callregs) do {                           \
+        extern void irq_trampoline_ ##nr ();                    \
+        __call16_int((callregs), (u16)&irq_trampoline_ ##nr );  \
     } while (0)
 #else
 #include "../out/rom16.offset.auto.h"
-#define call16_int(nr, callregs) do {           \
-        struct bregs *__br = (callregs);        \
-        __br->cs = 0xf000;                      \
-        __br->ip = OFFSET_irq_trampoline_ ##nr; \
-        call16(__br);                           \
-    } while (0)
+#define call16_int(nr, callregs)                                \
+    __call16_int((callregs), OFFSET_irq_trampoline_ ##nr )
 #endif
 
 // output.c