From owner-cvs-all@FreeBSD.ORG Thu Apr 10 12:48:07 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 7777437B401; Thu, 10 Apr 2003 12:48:07 -0700 (PDT) Received: from rwcrmhc53.attbi.com (rwcrmhc53.attbi.com [204.127.198.39]) by mx1.FreeBSD.org (Postfix) with ESMTP id D91A843F93; Thu, 10 Apr 2003 12:48:06 -0700 (PDT) (envelope-from julian@elischer.org) Received: from interjet.elischer.org (12-232-168-4.client.attbi.com[12.232.168.4]) by rwcrmhc53.attbi.com (rwcrmhc53) with ESMTP id <2003041019480605300do2ioe>; Thu, 10 Apr 2003 19:48:06 +0000 Received: from localhost (localhost.elischer.org [127.0.0.1]) by InterJet.elischer.org (8.9.1a/8.9.1) with ESMTP id MAA90527; Thu, 10 Apr 2003 12:48:05 -0700 (PDT) Message-Id: <200304101948.MAA90527@InterJet.elischer.org> Date: Thu, 10 Apr 2003 12:48:04 -0700 (PDT) From: Julian Elischer To: Jeff Roberson cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: Julian Elischer cc: cvs-all@FreeBSD.org cc: John Baldwin Subject: RE: cvs commit: src/sys/ddb db_ps.c src/sys/i386/i386 genassym.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: Thu, 10 Apr 2003 19:48:07 -0000 src/sys/kern init_main.c kern_fork.c kern_mutex.c kern_proc.c kern_thread.c sched_4bsd.c sched_ule.c subr_smp.c subr_witness.c src/sys/ In-Reply-To: <20030410152200.M37530-100000@mail.chesapeake.net> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII On Thu, 10 Apr 2003, Jeff Roberson wrote: > On Thu, 10 Apr 2003, Jeff Roberson wrote: > > > > > On Thu, 10 Apr 2003, John Baldwin wrote: > > > > > > > > On 10-Apr-2003 Julian Elischer wrote: > > > > julian 2003/04/10 10:35:45 PDT > > > > > > > > FreeBSD src repository > > > > > > > > Modified files: > > > > sys/ddb db_ps.c > > > > sys/i386/i386 genassym.c > > > > sys/kern init_main.c kern_fork.c kern_mutex.c > > > > kern_proc.c kern_thread.c sched_4bsd.c > > > > sched_ule.c subr_smp.c subr_witness.c > > > > sys/sys proc.h > > > > Log: > > > > Move the _oncpu entry from the KSE to the thread. > > > > The entry in the KSE still exists but it's purpose will change a bit > > > > when we add the ability to lock a KSE to a cpu. > > > > > > Why not add a ke_pincpu to hold the bound CPU? Since KSE's are in > > > theory a kind of virtual CPU abstraction the thread really seems to > > > be the wrong place for this information. > > > > > > > Er, this seems wrong to me. Regardless, please but the bound cpu > > Sorry, moving the information to the thread seems wrong. I'm not sure I > think it is such a good idea to so rigorously hide the kse structure. It > may be nice to limit its scope but I think it is not so necessary and it > leads to hacks like this where information is stored in a structure where > it does not logically make sense. It is a thread property. It is "what CPU am I running on now.." It is set when you are switched in and unset when you are switched out. Between these two points you can not switch KSE so the two are effectively the same. The only difference is that code that has a thread pointer has to get it from the KSE through indirection, where it can get it from the thread directly. Note that ALL cases of it being accessed did it via the thread pointer There is NO support for locking a KSE to a CPU yet. That is a completely different question. > > > information in the scheduler specific data. I already have an entry for > > it in ULE. > > > > Cheers, > > Jeff > > > >