Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 25 Aug 2018 15:31:23 +0000 (UTC)
From:      Konstantin Belousov <kib@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r338313 - head/sys/i386/i386
Message-ID:  <201808251531.w7PFVNp8074507@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kib
Date: Sat Aug 25 15:31:23 2018
New Revision: 338313
URL: https://svnweb.freebsd.org/changeset/base/338313

Log:
  Remove dead code in i386 cpu_throw().
  
  Curpmap must be already valid when cpu_throw() is called, even for early
  AP startup.
  
  Suggested by:	alc
  Reviewed by:	alc, markj
  Sponsored by:	The FreeBSD Foundation
  Approved by:	re (marius)
  MFC after:	1 week
  Differential revision:	https://reviews.freebsd.org/D16893

Modified:
  head/sys/i386/i386/swtch.s

Modified: head/sys/i386/i386/swtch.s
==============================================================================
--- head/sys/i386/i386/swtch.s	Sat Aug 25 15:21:28 2018	(r338312)
+++ head/sys/i386/i386/swtch.s	Sat Aug 25 15:31:23 2018	(r338313)
@@ -74,16 +74,12 @@
  */
 ENTRY(cpu_throw)
 	movl	PCPU(CPUID), %esi
-	movl	4(%esp),%ecx			/* Old thread */
-	testl	%ecx,%ecx			/* no thread? */
-	jz	1f
 	/* release bit from old pm_active */
 	movl	PCPU(CURPMAP), %ebx
 #ifdef SMP
 	lock
 #endif
 	btrl	%esi, PM_ACTIVE(%ebx)		/* clear old */
-1:
 	movl	8(%esp),%ecx			/* New thread */
 	movl	TD_PCB(%ecx),%edx
 	/* set bit in new pm_active */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201808251531.w7PFVNp8074507>