Changeset 9f9d725 in mainline

Timestamp:

2013-11-05T11:56:00Z (10 years ago)

Author:

Jakub Klama <jakub.klama@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

dbf4417

Parents:

3f596c9

Message:

• Properly implement inline_restores with respect to preserve invalid window variables
• Fix some next bugs in preemptible trap handler

File:

• kernel/arch/sparc32/src/trap_table.S (modified) (4 diffs)

kernel/arch/sparc32/src/trap_table.S

@@ -83 +83 @@
 /* Save next window to kernel stack or UWB */
 .macro inline_save_kernel
-        set 0x80000100, %g2
-        set 'k', %g1
-        sta %g1, [%g2] 0x1c
+        mov %wim, %g2
+        mov %g0, %wim
         save
         std %l0, [%sp +  0]
@@ -96 +95 @@
         std %i6, [%sp + 56]
         restore
+        mov %g2, %wim
 .endm
 
 .macro inline_save_uspace uwb
-        set 0x80000100, %g2
-        set 'u', %g1
-        sta %g1, [%g2] 0x1c
+        mov %wim, %g2
+        mov %g0, %wim
         save
-        ld [\uwb], %g1
-        std %l0, [%g1 +  0]
-        std %l2, [%g1 +  8]
-        std %l4, [%g1 + 16]
-        std %l6, [%g1 + 24]
-        std %i0, [%g1 + 32]
-        std %i2, [%g1 + 40]
-        std %i4, [%g1 + 48]
-        std %i6, [%g1 + 56]
+        std %l0, [\uwb +  0]
+        std %l2, [\uwb +  8]
+        std %l4, [\uwb + 16]
+        std %l6, [\uwb + 24]
+        std %i0, [\uwb + 32]
+        std %i2, [\uwb + 40]
+        std %i4, [\uwb + 48]
+        std %i6, [\uwb + 56]
         add \uwb, 64, \uwb
         restore
+        mov %g2, %wim
 .endm
 
 /* Restore window from kernel stack or UWB */
 .macro inline_restore_kernel
-        save
+        mov %wim, %l3
+        sll %l3, 7, %l4
+        srl %l3, 1, %l3
+        or  %l3, %l4, %l3
+        and %l3, 0xff, %l3
+        
+        mov %g0, %wim
+        restore
+        mov %l5, %g5
+        mov %l6, %g6
+        mov %l7, %g7
+        restore
+        mov %g5, %l5
+        mov %g6, %l6
+        mov %g7, %l7
+        save    
+
         ldd [%sp +  0], %l0
         ldd [%sp +  8], %l2
@@ -127 +142 @@
         ldd [%sp + 48], %i4
         ldd [%sp + 56], %i6
-        restore
+        save
+        
+        mov %l3, %wim
 .endm
 
 .macro inline_restore_uspace uwb
+        mov %wim, %l3
+        sll %l3, 7, %l4
+        srl %l3, 1, %l3
+        or  %l3, %l4, %l3
+        and %l3, 0xff, %l3
+
+        mov %g0, %wim
+        restore
+        mov %l5, %g5
+        mov %l6, %g6
+        mov %l7, %g7
+        restore
+        mov %g5, %l5
+        mov %g6, %l6
+        mov %g7, %l7
+        save    
+
+        ldd [\uwb +  0], %l0
+        ldd [\uwb +  8], %l2
+        ldd [\uwb + 16], %l4
+        ldd [\uwb + 24], %l6
+        ldd [\uwb + 32], %i0
+        ldd [\uwb + 40], %i2
+        ldd [\uwb + 48], %i4
+        ldd [\uwb + 56], %i6
+        sub \uwb, 64, \uwb
         save
-        ld [\uwb], %g1
-        std %l0, [%g1 +  0]
-        std %l2, [%g1 +  8]
-        std %l4, [%g1 + 16]
-        std %l6, [%g1 + 24]
-        std %i0, [%g1 + 32]
-        std %i2, [%g1 + 40]
-        std %i4, [%g1 + 48]
-        std %i6, [%g1 + 56]
-        sub \uwb, 64, \uwb
-        restore
+
+        mov %l3, %wim
 .endm
 
@@ -404 +438 @@
         nop
 
-        /* If trap originated from uspace, clear uspace window mark */
+        /* If trap originated from uspace, clear uspace window mark and save uwb address for future use */
         if_from_kernel 6f
         switch_to_invalid %g5, %g6
         clr %l7
+        mov %l6, %g7
         switch_back %g5, %g6
-
-6:      jmp %l1
+        mov %g7, %g1
+
+        /* If next window is invalid, do inline restore */
+6:      get_wim_number %g6
+        get_cwp %g7
+        inc %g7
+        cmp %g6, %g7
+        bne 8f
+
+        if_from_kernel 7f
+
+        inline_restore_uspace %g1
+        b 8f
+        nop
+
+7:      inline_restore_kernel
+
+8:      jmp %l1
         rett %l2