gdt: fix reload of GDT

The core problems were/are the following:

  - The flags of the segments were not entirely correct. Please recheck
    them against the spec!
  - The GDT pointer did not contain the address of the first (null) GTD
    entry, but the address of the stl::vector containing the GDT
    entries.
  - The far pointer must consist of:
    - the address to jump to
    - the byte index into the GDT for the desired segement descriptor to
      be loaded into CS.
  - The type of the "dummy" function we jump to was wrong (it's a
    function, we should declare it as such).
  - We cannot enable interrupts right now, since we die with a triple
    fault. This is caused by some initia fault which seems to lead to a
    general protection fault, which then triple faults since we cannot
    find the IDT.

Some FIXMEs have been added to the code. Please look at them carefully
and compare things against the specs.

1 files changed, 12 insertions, 13 deletions

diff --git a/arch/x86_64/src/context_switching/main.cpp b/arch/x86_64/src/context_switching/main.cpp
index 5d19f23..a5bd3fb 100644
--- a/arch/x86_64/src/context_switching/main.cpp
+++ b/arch/x86_64/src/context_switching/main.cpp
@@ -13,22 +13,21 @@ namespace teachos::arch::context_switching
     decltype(auto) global_descriptor_table = segment_descriptor_table::initialize_global_descriptor_table();
     decltype(auto) interrupt_descriptor_table = interrupt_descriptor_table::initialize_interrupt_descriptor_table();
 
-    interrupt_descriptor_table::segment_selector segment_selector{
-        1U, interrupt_descriptor_table::segment_selector::REQUEST_LEVEL_KERNEL};
-    kernel::cpu::far_pointer pointer{boot::reload_segment_register, segment_selector};
-    kernel::cpu::jmp(pointer);
+    kernel::cpu::far_pointer pointer{&boot::reload_segment_register, 1 * sizeof(segment_descriptor_table::segment_descriptor)};
+    asm volatile("rex64 lcall *%[far_function_pointer]" : : [far_function_pointer] "m" (pointer));
 
-    // Load task state segment descriptor from the last element in the global descriptor table, done by calculating
-    // offset in bytes to the start of the segment descriptor (5 * 16) = 80
-    uint16_t const tss_selector =
-        (global_descriptor_table.size() - 1) * sizeof(segment_descriptor_table::segment_descriptor);
-    kernel::cpu::load_task_register(tss_selector);
+    // // Load task state segment descriptor from the last element in the global descriptor table, done by calculating
+    // // offset in bytes to the start of the segment descriptor (5 * 16) = 80
+    // uint16_t const tss_selector =
+    //     (global_descriptor_table.size() - 1) * sizeof(segment_descriptor_table::segment_descriptor);
+    // kernel::cpu::load_task_register(tss_selector);
 
-    auto const stored_task_register = kernel::cpu::store_task_register();
-    arch::exception_handling::assert(tss_selector == stored_task_register,
-                                     "[Global Descriptor Table] Loaded TR value is not the same as the stored value.");
+    // auto const stored_task_register = kernel::cpu::store_task_register();
+    // arch::exception_handling::assert(tss_selector == stored_task_register,
+    //                                  "[Global Descriptor Table] Loaded TR value is not the same as the stored value.");
 
-    kernel::cpu::set_interrupt_flag();
+    // FIXME: We currently cannot enable interrupts, since for some reason, we will later run into what looks like a GP. Maybe because no IDT is loaded? Maybe our boot code segment is not set up correctly?
+    // kernel::cpu::set_interrupt_flag();
 
     descriptor_tables tables = {global_descriptor_table, interrupt_descriptor_table};
     return tables;