Changes in kernel/arch/ia32/src/asm.S [d029434:8078180] in mainline

File:

• kernel/arch/ia32/src/asm.S (modified) (8 diffs)

kernel/arch/ia32/src/asm.S

@@ -1 +1 @@
 /*
- * Copyright (c) 2010 Jakub Jermar
+ * Copyright (c) 2001 Jakub Jermar
  * All rights reserved.
  *
@@ -32 +32 @@
 
 #include <arch/pm.h>
-#include <arch/cpu.h>
 #include <arch/mm/page.h>
 
@@ -191 +190 @@
 .global sysenter_handler
 sysenter_handler:
+        sti
         subl $(ISTATE_REAL_SIZE), %esp
 
@@ -227 +227 @@
          * Switch to kernel selectors.
          */
-        movw $(gdtselector(KDATA_DES)), %ax
+        movw $16, %ax
         movw %ax, %ds
         movw %ax, %es
         
-        /*
-         * Sanitize EFLAGS.
-         *
-         * SYSENTER does not clear the NT flag, which could thus proliferate
-         * from here to the IRET instruction via a context switch and result
-         * in crash.
-         *
-         * SYSENTER does not clear DF, which the ABI assumes to be cleared.
-         *
-         * SYSENTER clears IF, which we would like to be set for syscalls.
-         *
-         */
-        pushl $(EFLAGS_IF)  /* specify EFLAGS bits that we want to set */
-        popfl               /* set bits from the mask, clear or ignore others */
-
+        cld
         call syscall_handler
         
@@ -308 +294 @@
          * Switch to kernel selectors.
          */
-        movl $(gdtselector(KDATA_DES)), %eax
+        movl $16, %eax
         movl %eax, %ds
         movl %eax, %es
@@ -372 +358 @@
                 .if (1 << \i) & ERROR_WORD_INTERRUPT_LIST
                         /*
-                         * Exception with error word.
+                         * Exception with error word: do nothing
                          */
-                        subl $ISTATE_SOFT_SIZE, %esp
                 .else
                         /*
                          * Exception without error word: fake up one
                          */
-                        subl $(ISTATE_SOFT_SIZE + 4), %esp
+                        pushl $0
                 .endif
         .else
                 /*
-                 * Interrupt: fake up an error word 
+                 * Interrupt: fake up one
                  */
-                subl $(ISTATE_SOFT_SIZE + 4), %esp
+                pushl $0
         .endif
+        
+        subl $ISTATE_SOFT_SIZE, %esp
         
         /*
@@ -417 +404 @@
          * Switch to kernel selectors.
          */
-        movl $(gdtselector(KDATA_DES)), %eax
+        movl $16, %eax
         movl %eax, %ds
         movl %eax, %es
@@ -426 +413 @@
          */
         xorl %eax, %eax
-        cmpl $(gdtselector(KTEXT_DES)), ISTATE_OFFSET_CS(%esp)
-        cmovnzl %eax, %ebp
+        cmpl $8, ISTATE_OFFSET_CS(%esp)
+        cmovl %eax, %ebp
 
         movl %ebp, ISTATE_OFFSET_EBP_FRAME(%esp)