From owner-cvs-src@FreeBSD.ORG  Thu Jan 29 04:35:12 2004
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 734E616A4CE; Thu, 29 Jan 2004 04:35:12 -0800 (PST)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 9B2C043D45; Thu, 29 Jan 2004 04:35:11 -0800 (PST)
	(envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.10/8.12.10) with ESMTP id i0TCZB0B036161;
	Thu, 29 Jan 2004 04:35:11 -0800 (PST)
	(envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.12.10/8.12.10/Submit) id i0TCZBDG036160;
	Thu, 29 Jan 2004 04:35:11 -0800 (PST)
	(envelope-from bde)
Message-Id: <200401291235.i0TCZBDG036160@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Thu, 29 Jan 2004 04:35:11 -0800 (PST)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/vm vm_glue.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 29 Jan 2004 12:35:12 -0000

bde         2004/01/29 04:35:11 PST

  FreeBSD src repository

  Modified files:
    sys/vm               vm_glue.c 
  Log:
  Fixed breakage of scheduling in rev.1.29 of subr_4bsd.c.  The
  "scheduler" here has very little to do with scheduling.  It is actually
  the swapper, and it really must be the last SYSINIT'ed item like its
  comment says, since proc0 metamorphoses into swapper by calling
  scheduler() last in mi_start(), and scheduler() never returns..  Rev.1.29
  of subr_4bsd.c broke this by adding another SI_ORDER_FIRST item
  (kproc_start() for schedcpu_thread() onto the SI_SUB_RUN_SCHEDULER_LIST.
  The sorting of SYSINITs with identical orders (at all levels) is
  apparently nondeterministic, so this resulted in schedule() sometimes
  being called second last and schedcpu_thread() not being called at all.
  
  This quick fix just changes the code to almost match the comment
  (SI_ORDER_FIRST -> SI_ORDER_ANY).  "LAST" is misspelled "ANY", and
  there is no way to ensure that there is only 1 very lst SYSINIT.
  A more complete fix would remove the SYSINIT obfuscation.
  
  Revision  Changes    Path
  1.188     +1 -1      src/sys/vm/vm_glue.c