From owner-freebsd-current Fri May 31 5: 9:29 2002 Delivered-To: freebsd-current@freebsd.org Received: from srv1.cosmo-project.de (srv1.cosmo-project.de [213.83.6.106]) by hub.freebsd.org (Postfix) with ESMTP id 36A9037B405 for ; Fri, 31 May 2002 05:09:23 -0700 (PDT) Received: from cicely5.cicely.de (cicely5.cicely.de [IPv6:3ffe:400:8d0:301:200:92ff:fe9b:20e7]) (authenticated bits=0) by srv1.cosmo-project.de (8.12.3/8.12.3) with ESMTP id g4VC9AHc052508 (version=TLSv1/SSLv3 cipher=EDH-RSA-DES-CBC3-SHA bits=168 verify=OK); Fri, 31 May 2002 14:09:13 +0200 (CEST) (envelope-from ticso@cicely5.cicely.de) Received: from cicely5.cicely.de (localhost [IPv6:::1]) by cicely5.cicely.de (8.12.1/8.12.1) with ESMTP id g4VC9QSA014181 (version=TLSv1/SSLv3 cipher=EDH-RSA-DES-CBC3-SHA bits=168 verify=NO); Fri, 31 May 2002 14:09:27 +0200 (CEST)?g (envelope-from ticso@cicely5.cicely.de) Received: (from ticso@localhost) by cicely5.cicely.de (8.12.1/8.12.1/Submit) id g4VC9PaT014180; Fri, 31 May 2002 14:09:25 +0200 (CEST)?g (envelope-from ticso) Date: Fri, 31 May 2002 14:09:25 +0200 From: Bernd Walter To: Julian Elischer Cc: FreeBSD current users Subject: Re: Seeking OK to commit KSE MIII-again Message-ID: <20020531120925.GL5414@cicely5.cicely.de> Reply-To: ticso@cicely.de References: <20020530191432.GE5414@cicely5.cicely.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020530191432.GE5414@cicely5.cicely.de> User-Agent: Mutt/1.3.26i X-Operating-System: FreeBSD cicely5.cicely.de 5.0-CURRENT i386 Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Thu, May 30, 2002 at 09:14:33PM +0200, Bernd Walter wrote: > On Thu, May 30, 2002 at 09:20:57AM -0700, Julian Elischer wrote: > > ok, but does anyone other than john (who has commented) have any comments > > about the logic and work in the change? > > > > I'm working on his comments but comments by others would sure be > > appreciated.. > > especially if they actually comment on what I'm trying to do.. > > > > If I can get the changes for the other architectures done, > > I'd like to commit this weekend. HOPEFULLY it shouldn't > > affect normal operations but of course the testing done by two people > > can't hope to equal that which will be done in teh first 24 hours > > once it's committed :-) > > > > once again: > > > > the diffs are at: > > http://people.freebsd.org/~peter/kse.diff > > and > > http://people.freebsd.org/~julian/thediff > > and the diffs I need for other architectures are versions of: > > > > sys/i386/i386/genassym.c (small) > > sys/i386/i386/machdep.c (1 line) > > sys/i386/i386/swtch.s (a few lines) > > sys/i386/i386/trap.c (small) > > sys/i386/i386/vm_machdep.c (largly new functions, we could stub them) > > sys/i386/include/kse.h (new file) > > sys/i386/linux/linux_machdep.c (one line) > > > > Largely these need to be written by someone who is intimately aquainted > > with the register set of the machine in question and knows > > what registers need to be saved to restore a user context correctly. > > I can do the alpha part tomorrow unless someone else already startet. There are problems with the patchset: ../../../kern/kern_proc.c: In function `fill_kinfo_proc': ../../../kern/kern_proc.c:731: `TDS_RUN' undeclared (first use in this function) ../../../kern/kern_proc.c:731: (Each undeclared identifier is reported only once ../../../kern/kern_proc.c:731: for each function it appears in.) ../../../kern/kern_proc.c:733: `SRUN' undeclared (first use in this function) ../../../kern/kern_proc.c:734: `TDS_SLEEP' undeclared (first use in this function) ../../../kern/kern_proc.c:735: `SSLEEP' undeclared (first use in this function) ../../../kern/kern_proc.c:737: `SSTOP' undeclared (first use in this function) ../../../kern/kern_proc.c:739: `SMTX' undeclared (first use in this function) ../../../kern/kern_proc.c:741: `SWAIT' undeclared (first use in this function) ../../../kern/kern_proc.c:743: syntax error before '{' token ../../../kern/kern_proc.c:744: `SZOMB' undeclared (first use in this function) ../../../kern/kern_proc.c:745: syntax error before "else" ../../../kern/kern_proc.c:663: warning: unused variable `tp' ../../../kern/kern_proc.c:664: warning: unused variable `sp' ../../../kern/kern_proc.c:665: warning: unused variable `tv' [...] Line 731 should be TDS_RUNQ I guess. TDS_SLEEP - where is it defined - intended to be TDS_SLP? SRUN, SSLEEP and others: +#ifndef _KERNEL /* Only usable by libkvm for legacy apps */ +#define SIDL 1 /* Process being created by fork. */ +#define SRUN 2 /* Currently runnable. */ +#define SSLEEP 3 /* Sleeping on an address. */ +#define SSTOP 4 /* Process debugging or suspension. */ +#define SZOMB 5 /* Awaiting collection by parent. */ +#define SWAIT 6 /* Waiting for interrupt. */ +#define SMTX 7 /* Blocked on a mutex. */ +#endif Why are they (newly) used in kern_proc.c? IMO they need to be changed to TDS_SLP, ... -- B.Walter COSMO-Project http://www.cosmo-project.de ticso@cicely.de Usergroup info@cosmo-project.de To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message