From owner-cvs-src@FreeBSD.ORG  Mon May  3 08:10:38 2004
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from green.homeunix.org (freefall.freebsd.org [216.136.204.21])
	by hub.freebsd.org (Postfix) with ESMTP
	id 7330C16A4CE; Mon,  3 May 2004 08:10:38 -0700 (PDT)
Received: from localhost (green@localhost [127.0.0.1])
	by green.homeunix.org (8.12.11/8.12.11) with ESMTP id i43FAced005343;
	Mon, 3 May 2004 11:10:38 -0400 (EDT)
	(envelope-from green@green.homeunix.org)
Message-Id: <200405031510.i43FAced005343@green.homeunix.org>
X-Mailer: exmh version 2.6.3 04/04/2003 with nmh-1.0.4
To: Colin Percival <cperciva@FreeBSD.org>
In-Reply-To: Message from Colin Percival <cperciva@FreeBSD.org> 
	<200405030048.i430mBDq031197@repoman.freebsd.org> 
From: Brian Fundakowski Feldman <green@FreeBSD.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Date: Mon, 03 May 2004 11:10:38 -0400
Sender: green@green.homeunix.org
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/kern kern_clock.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, 03 May 2004 15:10:38 -0000

Colin Percival <cperciva@FreeBSD.org> wrote:
> cperciva    2004/05/02 17:48:11 PDT
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/kern             kern_clock.c 
>   Log:
>   Fix a race condition which could result in profprocs being decremented
>   more than once if stopprofclock is called multiple times on the same
>   process.

FWIW, the sched_lock enclosing the p->p_flag modification seems pretty bogus.

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green@FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\