Fixed #2: gdtr.base_address in stage2.asm is not being respected.

Author: scopeInfinity

src/bootloader/stage2.asm

@@ -24,12 +24,7 @@ global label_exit
         mov ebp, esp
 
         ; Load GDT Table
-            ; Workaround for Issue #3
-            mov eax, [esp+8]   ; gdtr_address
-            mov ebx, eax
-            add ebx, 8         ; &gdtr_table
-            mov [eax+2], ebx   ; gdtr.base_address = &gdtr_table
-
+        mov eax, [esp+8]   ; gdtr_address
         lgdt [eax]         ; Load GDT Table
 
         ; Enter Protected mode

src/bootloader/stage2.c

@@ -26,7 +26,6 @@ struct GDTEntry {
 };
 #pragma pack(pop)
 
-// Issue#2: gdt_table must immediately after gdtr.
 struct GDTReference gdtr;
 struct GDTEntry gdt_table[GDT_TABLE_SIZE];
 
@@ -71,9 +70,6 @@ int populate_gdt_table() {
         0b0100,  // 32-bit protected mode
         0x92);
 
-    // Issue#2: For some reason gdtr.base_address assigned is not
-    // being respected later on. So, as a bad workaround hardcoding
-    // base_address as &gdtr+8 in stage2.asm.
     gdtr.base_address = (int)gdt_table;
     gdtr.size = (sizeof(gdt_table));