Changeset 83001 in vbox for trunk/src/VBox/Devices/Graphics/BIOS/vgabios.c

Timestamp:

Feb 6, 2020 7:38:51 AM (5 years ago)

Author:

vboxsync

Message:

VGABIOS: Do not recursively invoke INT 10h C handlers to conserve stack space; also do not recursively invoke INT 10h, rather use INT 6Dh which should not be hooked (see bugref:6549).

File:

• trunk/src/VBox/Devices/Graphics/BIOS/vgabios.c (modified) (3 diffs)

trunk/src/VBox/Devices/Graphics/BIOS/vgabios.c

@@ -230 +230 @@
 #endif
     set_int_vector(0x10, vgabios_int10_handler);
+    set_int_vector(0x6D, vgabios_int10_handler);
 #ifdef CIRRUS
     cirrus_init();
@@ -854 +855 @@
 }
 
-/// @todo Evaluate whether executing INT 10h is the right thing here
+/// Recursive BIOS invocation, uses
 extern void vga_font_set(uint8_t function, uint8_t data);
 #pragma aux vga_font_set =  \
     "mov    ah, 11h"        \
-    "int    10h"            \
+    "int    6Dh"            \
     parm [al] [bl];
 
@@ -1070 +1071 @@
  if(vga_modes[line].class==TEXT)
   {
-     vga_font_set(0x04, 0);     /* Load 8x16 font into page 0. */
-     vga_font_set(0x03, 0);     /* Select font page mode 0. */
+     biosfn_load_text_8_16_pat(0x04, 0);    /* Load 8x16 font into page 0. */
+     vga_font_set(0x03, 0);                 /* Select font page mode 0. */
   }