From: Ben Pfaff <blp@cs.stanford.edu>
Date: Sun, 9 Apr 2006 19:52:03 +0000 (+0000)
Subject: Remove unnecessary test for switching from null current thread.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=424970679fb7dec648531888ae26919b784de1b6;p=pintos-anon

Remove unnecessary test for switching from null current thread.
Thanks to "Waqar Mohsin" <wmohsin@gmail.com> for reporting this.
---

diff --git a/src/threads/switch.S b/src/threads/switch.S
index 6cb70aa..48fc588 100644
--- a/src/threads/switch.S
+++ b/src/threads/switch.S
@@ -33,10 +33,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