Changeset 74613 in vbox for trunk

Timestamp:

Oct 4, 2018 1:44:55 PM (6 years ago)

Author:

vboxsync

Message:

BIOS: Inlined write_word, read_word and friends. Saves over 600 bytes of code.

Location:

trunk/src/VBox/Devices/PC/BIOS

Files:

• bios.c (modified) (1 diff)
• floppy.c (modified) (1 diff)
• inlines.h (modified) (1 diff)

trunk/src/VBox/Devices/PC/BIOS/bios.c

@@ -62 +62 @@
 
 static  const char  bios_cvs_version_string[] = "VirtualBox " VBOX_VERSION_STRING;
-
-uint8_t read_byte(uint16_t seg, uint16_t offset)
-{
-    return( *(seg:>(uint8_t *)offset) );
-}
-
-void write_byte(uint16_t seg, uint16_t offset, uint8_t data)
-{
-    *(seg:>(uint8_t *)offset) = data;
-}
-
-uint16_t read_word(uint16_t seg, uint16_t offset)
-{
-    return( *(seg:>(uint16_t *)offset) );
-}
-
-void write_word(uint16_t seg, uint16_t offset, uint16_t data)
-{
-    *(seg:>(uint16_t *)offset) = data;
-}
-
-uint32_t read_dword(uint16_t seg, uint16_t offset)
-{
-    return( *(seg:>(uint32_t *)offset) );
-}
-
-void write_dword(uint16_t seg, uint16_t offset, uint32_t data)
-{
-    *(seg:>(uint32_t *)offset) = data;
-}
 
 uint8_t inb_cmos(uint8_t cmos_reg)

trunk/src/VBox/Devices/PC/BIOS/floppy.c

@@ -59 +59 @@
 
 extern uint16_t get_floppy_dpt(uint8_t drive_type);
-
-// Local copies to slihgtly reduce stack usage.
-inline uint8_t read_byte(uint16_t seg, uint16_t offset)
-{
-    return( *(seg:>(uint8_t *)offset) );
-}
-
-inline void write_byte(uint16_t seg, uint16_t offset, uint8_t data)
-{
-    *(seg:>(uint8_t *)offset) = data;
-}
-
 
 //////////////////////

trunk/src/VBox/Devices/PC/BIOS/inlines.h

@@ -27 +27 @@
 #define outw(p, v)  outpw(p, v)
 
-extern  uint8_t     read_byte(uint16_t seg, uint16_t offset);
-extern  uint16_t    read_word(uint16_t seg, uint16_t offset);
-extern  uint32_t    read_dword(uint16_t seg, uint16_t offset);
-extern  void        write_byte(uint16_t seg, uint16_t offset, uint8_t data);
-extern  void        write_word(uint16_t seg, uint16_t offset, uint16_t data);
-extern  void        write_dword(uint16_t seg, uint16_t offset, uint32_t data);
+/* Far byte/word/dword access routines. */
+
+inline uint8_t read_byte(uint16_t seg, uint16_t offset)
+{
+    return( *(seg:>(uint8_t *)offset) );
+}
+
+inline void write_byte(uint16_t seg, uint16_t offset, uint8_t data)
+{
+    *(seg:>(uint8_t *)offset) = data;
+}
+
+inline uint16_t read_word(uint16_t seg, uint16_t offset)
+{
+    return( *(seg:>(uint16_t *)offset) );
+}
+
+inline void write_word(uint16_t seg, uint16_t offset, uint16_t data)
+{
+    *(seg:>(uint16_t *)offset) = data;
+}
+
+inline uint32_t read_dword(uint16_t seg, uint16_t offset)
+{
+    return( *(seg:>(uint32_t *)offset) );
+}
+
+inline void write_dword(uint16_t seg, uint16_t offset, uint32_t data)
+{
+    *(seg:>(uint32_t *)offset) = data;
+}
+
 
 void int_enable(void);