POST: Move cpu caching and dma setup to platform_hardware_setup().

The CPU cache is enabled and DMA is disabled on all real-world POST
entry situations, so no need to do this in the low-level "pre-init"
phase.  Instead, move it to the platform hardware setup stage.

Also, move the setting of the reboot flags (which control reset-vector
entry point handling) to ivt_init().

Signed-off-by: Kevin O'Connor <kevin@koconnor.net>

1 files changed, 10 insertions, 10 deletions

diff --git a/src/post.c b/src/post.c
index ae5e9230..f2fabbb6 100644
--- a/src/post.c
+++ b/src/post.c
@@ -37,6 +37,10 @@ ivt_init(void)
 {
     dprintf(3, "init ivt\n");
 
+    // Setup reset-vector entry point (controls legacy reboots).
+    HaveRunPost = 1;
+    outb_cmos(0, CMOS_RESET_CODE);
+
     // Initialize all vectors to the default handler.
     int i;
     for (i=0; i<256; i++)
@@ -145,6 +149,12 @@ device_hardware_setup(void)
 static void
 platform_hardware_setup(void)
 {
+    // Enable CPU caching
+    setcr0(getcr0() & ~(CR0_CD|CR0_NW));
+
+    // Make sure legacy DMA isn't running.
+    dma_setup();
+
     // Init base pc hardware.
     pic_setup();
     timer_setup();
@@ -305,16 +315,6 @@ reloc_preinit(void *f, void *arg)
 void VISIBLE32INIT
 dopost(void)
 {
-    // Set reboot flags.
-    HaveRunPost = 1;
-    outb_cmos(0, CMOS_RESET_CODE);
-
-    // Enable CPU caching
-    setcr0(getcr0() & ~(CR0_CD|CR0_NW));
-
-    // Make sure legacy DMA isn't running.
-    dma_preinit();
-
     // Check if we are running under Xen.
     xen_preinit();