Rename schedule_tail() to thread_schedule_tail().
[pintos-anon] / src / threads / switch.S
index 374da9e4a11cfd64a79816c3d358a177b224ff09..feca86cd16d02e3d6f31d1dde25cff379c0861a4 100644 (file)
@@ -12,7 +12,8 @@
 #### restore the registers.  As part of switching stacks we record the
 #### current stack pointer in CUR's thread structure.
 
-.globl switch_threads 
+.globl switch_threads
+.func switch_threads
 switch_threads:
        # Save caller's register state.
        #
@@ -20,7 +21,8 @@ switch_threads:
        # but requires us to preserve %ebx, %ebp, %esi, %edi.  See
        # [SysV-ABI-386] pages 3-11 and 3-12 for details.
        #
-       # This stack frame must match the one set up by thread_create().
+       # This stack frame must match the one set up by thread_create()
+       # in size.
        pushl %ebx
        pushl %ebp
        pushl %esi
@@ -32,10 +34,7 @@ switch_threads:
 
        # Save current stack pointer to old thread's stack, if any.
        movl SWITCH_CUR(%esp), %eax
-       test %eax, %eax
-       jz 1f
        movl %esp, (%eax,%edx,1)
-1:
 
        # Restore stack pointer from new thread's stack.
        movl SWITCH_NEXT(%esp), %ecx
@@ -47,17 +46,20 @@ switch_threads:
        popl %ebp
        popl %ebx
         ret
+.endfunc
 
 .globl switch_entry
+.func switch_entry
 switch_entry:
        # Discard switch_threads() arguments.
        addl $8, %esp
 
-       # Call schedule_tail(prev).
+       # Call thread_schedule_tail(prev).
        pushl %eax
-.globl schedule_tail
-       call schedule_tail
+.globl thread_schedule_tail
+       call thread_schedule_tail
        addl $4, %esp
 
        # Start thread proper.
        ret
+.endfunc