From owner-cvs-src@FreeBSD.ORG Sun Sep 19 22:53:01 2004 Return-Path: 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 ADB9516A4CE; Sun, 19 Sep 2004 22:53:01 +0000 (GMT) Received: from ylpvm29.prodigy.net (ylpvm29-ext.prodigy.net [207.115.57.60]) by mx1.FreeBSD.org (Postfix) with ESMTP id 30C7143D4C; Sun, 19 Sep 2004 22:53:01 +0000 (GMT) (envelope-from julian@elischer.org) Received: from elischer.org (adsl-216-100-134-94.dsl.snfc21.pacbell.net [216.100.134.94])i8JMqjIV022416; Sun, 19 Sep 2004 18:52:46 -0400 Message-ID: <414E0DCA.5090601@elischer.org> Date: Sun, 19 Sep 2004 15:52:58 -0700 From: Julian Elischer User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.4b) Gecko/20030524 X-Accept-Language: en, hu MIME-Version: 1.0 To: David Schultz References: <200409191834.i8JIYHXU089517@repoman.freebsd.org> In-Reply-To: <200409191834.i8JIYHXU089517@repoman.freebsd.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_proc.c kern_switch.c src/sys/sys sched.h 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 19 Sep 2004 22:53:01 -0000 David Schultz wrote: > das 2004-09-19 18:34:17 UTC > > FreeBSD src repository > > Modified files: > sys/kern kern_proc.c kern_switch.c > sys/sys sched.h > sys/vm vm_glue.c > Log: > The zone from which proc structures are allocated is marked > UMA_ZONE_NOFREE to guarantee type stability, so proc_fini() should > never be called. Move an assertion from proc_fini() to proc_dtor() > and garbage-collect the rest of the unreachable code. I have retained > vm_proc_dispose(), since I consider its disuse a bug. well we do aim to one day remove the requirement for UMA_ZONE_NOFREE. In fact I have a gague feeling we mayhave already done so. I think it had to do with what page tables the kernel ran on after a thread went away. Peter may have a better memory as to why that was required. > > Revision Changes Path > 1.217 +5 -16 src/sys/kern/kern_proc.c > 1.95 +0 -16 src/sys/kern/kern_switch.c > 1.22 +1 -1 src/sys/sys/sched.h > 1.204 +4 -1 src/sys/vm/vm_glue.c