From owner-cvs-all@FreeBSD.ORG Mon Jun 16 21:48:25 2003 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 32F4037B401; Mon, 16 Jun 2003 21:48:25 -0700 (PDT) Received: from magic.adaptec.com (magic-mail.adaptec.com [208.236.45.100]) by mx1.FreeBSD.org (Postfix) with ESMTP id 6C41E43F75; Mon, 16 Jun 2003 21:48:24 -0700 (PDT) (envelope-from scottl@freebsd.org) Received: from redfish.adaptec.com (redfish.adaptec.com [162.62.50.11]) by magic.adaptec.com (8.11.6/8.11.6) with ESMTP id h5H4mA830652; Mon, 16 Jun 2003 21:48:10 -0700 Received: from freebsd.org (hollin.btc.adaptec.com [10.100.253.56]) by redfish.adaptec.com (8.8.8p2+Sun/8.8.8) with ESMTP id VAA19381; Mon, 16 Jun 2003 21:48:19 -0700 (PDT) Message-ID: <3EEE9D7F.2080406@freebsd.org> Date: Mon, 16 Jun 2003 22:47:59 -0600 From: Scott Long User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.3) Gecko/20030425 X-Accept-Language: en-us, en MIME-Version: 1.0 To: David Xu References: <200306150031.h5F0VPse055534@repoman.freebsd.org> <3EEE0F0C.3030205@freebsd.org> <001501c33465$04b11040$0701a8c0@tiger> In-Reply-To: <001501c33465$04b11040$0701a8c0@tiger> 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/alpha/alpha trap.c src/sys/alpha/linux src/sys/compat/svr4procfs_status.c src/sys/i386/i386 pmap.c ... X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Jun 2003 04:48:25 -0000 I apologize for coming on stronger than I meant. It likely comes down to personal style, so there is no use in wasting effort arguing about it. Scott David Xu wrote: > What does KSE mean ? I think kse is only useful for > scheduler, I would like it disappears in kse_thread.c > and only becomes a scheduler private data structure, > current kern_thread.c is forced to manage kse data > structure but no code is really using it in that file. > Julian even ever said ksegrp should be a subproc, > I think he is right. > If someone want to implement another SA, feel free to > do, but there is no reason that our SA implement > should be called P_KSE and a future not implemented yet > should be called P_SA or P_SAxxx. :( > > David Xu > > ----- Original Message ----- > From: "Scott Long" > To: "David Xu" > Cc: ; ; > Sent: Tuesday, June 17, 2003 2:40 AM > Subject: Re: cvs commit: src/sys/alpha/alpha trap.c src/sys/alpha/linux linux_sysvec.c src/sys/amd64/amd64 pmap.c trap.c src/sys/compat/svr4 svr4_sysvec.c src/sys/ddb db_ps.c src/sys/fs/procfs procfs_status.c src/sys/i386/i386 pmap.c ... > > > >>It would have been more appropriate to revert the name to P_KSE rather >>than give it yet another name. What happens when someone comes along >>and wants to experiment with another SA-like implementation? >> >>Scott >> >>David Xu wrote: >> >>>davidxu 2003/06/14 17:31:25 PDT >>> >>> FreeBSD src repository >>> >>> Modified files: >>> sys/alpha/alpha trap.c >>> sys/alpha/linux linux_sysvec.c >>> sys/amd64/amd64 pmap.c trap.c >>> sys/compat/svr4 svr4_sysvec.c >>> sys/ddb db_ps.c >>> sys/fs/procfs procfs_status.c >>> sys/i386/i386 pmap.c sys_machdep.c trap.c >>> sys/i386/linux linux_sysvec.c >>> sys/ia64/ia64 trap.c >>> sys/kern kern_clock.c kern_exec.c kern_exit.c >>> kern_fork.c kern_sig.c kern_switch.c >>> kern_synch.c kern_thread.c sched_4bsd.c >>> sched_ule.c subr_trap.c tty.c >>> sys/sparc64/sparc64 trap.c >>> sys/sys proc.h >>> Log: >>> Rename P_THREADED to P_SA. P_SA means a process is using scheduler >>> activations. >>> >>> Revision Changes Path >>> 1.115 +1 -1 src/sys/alpha/alpha/trap.c >>> 1.89 +1 -1 src/sys/alpha/linux/linux_sysvec.c >>> 1.411 +1 -1 src/sys/amd64/amd64/pmap.c >>> 1.261 +1 -1 src/sys/amd64/amd64/trap.c >>> 1.33 +1 -1 src/sys/compat/svr4/svr4_sysvec.c >>> 1.46 +3 -3 src/sys/ddb/db_ps.c >>> 1.49 +1 -1 src/sys/fs/procfs/procfs_status.c >>> 1.411 +1 -1 src/sys/i386/i386/pmap.c >>> 1.83 +1 -1 src/sys/i386/i386/sys_machdep.c >>> 1.254 +1 -1 src/sys/i386/i386/trap.c >>> 1.125 +1 -1 src/sys/i386/linux/linux_sysvec.c >>> 1.83 +1 -1 src/sys/ia64/ia64/trap.c >>> 1.160 +3 -3 src/sys/kern/kern_clock.c >>> 1.224 +2 -2 src/sys/kern/kern_exec.c >>> 1.217 +2 -2 src/sys/kern/kern_exit.c >>> 1.202 +3 -3 src/sys/kern/kern_fork.c >>> 1.242 +1 -1 src/sys/kern/kern_sig.c >>> 1.60 +5 -5 src/sys/kern/kern_switch.c >>> 1.226 +2 -2 src/sys/kern/kern_synch.c >>> 1.140 +6 -6 src/sys/kern/kern_thread.c >>> 1.20 +1 -1 src/sys/kern/sched_4bsd.c >>> 1.38 +1 -1 src/sys/kern/sched_ule.c >>> 1.256 +2 -2 src/sys/kern/subr_trap.c >>> 1.203 +1 -1 src/sys/kern/tty.c >>> 1.63 +1 -1 src/sys/sparc64/sparc64/trap.c >>> 1.337 +1 -1 src/sys/sys/proc.h >>> >> >> > >