From owner-cvs-src@FreeBSD.ORG  Mon Jun 21 14:49:51 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 4849E16A4CE; Mon, 21 Jun 2004 14:49:51 +0000 (GMT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 423AD43D49; Mon, 21 Jun 2004 14:49:51 +0000 (GMT)
	(envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.11/8.12.11) with ESMTP id i5LEnpum031425;
	Mon, 21 Jun 2004 14:49:51 GMT
	(envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.12.11/8.12.11/Submit) id i5LEnpu3031424;
	Mon, 21 Jun 2004 14:49:51 GMT
	(envelope-from bde)
Message-Id: <200406211449.i5LEnpu3031424@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Mon, 21 Jun 2004 14:49:51 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/kern kern_exit.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: Mon, 21 Jun 2004 14:49:51 -0000

bde         2004-06-21 14:49:50 UTC

  FreeBSD src repository

  Modified files:
    sys/kern             kern_exit.c 
  Log:
  (1) Removed the bogus condition "p->p_pid != 1" on calling sched_exit()
      from exit1().  sched_exit() must be called unconditionally from exit1().
      It was called almost unconditionally because the only exits on system
      shutdown if at all.
  
  (2) Removed the comment that presumed to know what sched_exit() does.
      sched_exit() does different things for the ULE case.  The call became
      essential when it started doing load average stuff, but its caller
      should not know that.
  
  (3) Didn't fix bugs caused by bitrot in the condition.  The condition was
      last correct in rev.1.208 when it was in wait1().  There p was spelled
      curthread->td_proc and was for the waiting parent; now p is for the
      exiting child.  The condition was to avoid lowering init's priority.
      It should be in sched_exit() itself.  Lowering of priorities is broken
      in other ways in at least the 4BSD scheduler, and doing it for init
      causes less noticeable problems than doing it for for shells.
  
  Noticed by:     julian (1)
  
  Revision  Changes    Path
  1.238     +1 -7      src/sys/kern/kern_exit.c