From: Gabe Black <gblack@eecs.umich.edu>
Date: Wed, 25 Feb 2009 18:18:16 +0000 (-0800)
Subject: X86: Move where CS is set so CPL checks work out.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=897c3748925e83301027d85dbc4c0479d972fda4;p=gem5.git

X86: Move where CS is set so CPL checks work out.
---

diff --git a/src/arch/x86/isa/insts/romutil.py b/src/arch/x86/isa/insts/romutil.py
index a39ba202f..93276addc 100644
--- a/src/arch/x86/isa/insts/romutil.py
+++ b/src/arch/x86/isa/insts/romutil.py
@@ -124,6 +124,18 @@ def rom
     ##
     wrip t0, t9, dataSize=8
 
+    #
+    # Set up the target code segment. Do this now so we have the right
+    # permissions when setting up the stack frame.
+    #
+    srli t5, t4, 16, dataSize=8
+    andi t5, t5, 0xFF, dataSize=8
+    wrdl cs, t3, t5, dataSize=8
+    # Tuck away the old CS for use below
+    limm t10, 0, dataSize=8
+    rdsel t10, cs, dataSize=2
+    wrsel cs, t5, dataSize=2
+
 
     #
     # Build up the interrupt stack frame
@@ -133,9 +145,7 @@ def rom
     # Write out the contents of memory
     %(errorCodeCode)s
     st t7, hs, [1, t0, t6], %(errorCodeSize)d, dataSize=8, addressSize=8
-    limm t5, 0, dataSize=8
-    rdsel t5, cs, dataSize=2
-    st t5, hs, [1, t0, t6], 8 + %(errorCodeSize)d, dataSize=8, addressSize=8
+    st t10, hs, [1, t0, t6], 8 + %(errorCodeSize)d, dataSize=8, addressSize=8
     rflags t10, dataSize=8
     st t10, hs, [1, t0, t6], 16 + %(errorCodeSize)d, dataSize=8, addressSize=8
     st rsp, hs, [1, t0, t6], 24 + %(errorCodeSize)d, dataSize=8, addressSize=8
@@ -146,14 +156,6 @@ def rom
     mov rsp, rsp, t6, dataSize=8
     wrsel ss, t11, dataSize=2
 
-    #
-    # Set up the target code segment
-    #
-    srli t5, t4, 16, dataSize=8
-    andi t5, t5, 0xFF, dataSize=8
-    wrdl cs, t3, t5, dataSize=8
-    wrsel cs, t5, dataSize=2
-
     #
     # Adjust rflags which is still in t10 from above
     #