Date: Thu, 29 Apr 1999 08:00:33 +1000 From: Bruce Evans <bde@zeta.org.au> To: dfr@nlsystems.com, luoqi@watermarkgroup.com, tejblum@arc.hq.cti.ru Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org, dt@FreeBSD.org, luoqi@FreeBSD.org Subject: Re: cvs commit: src/sys/alpha/alpha pmap.c Message-ID: <199904282200.IAA05572@godzilla.zeta.org.au>
next in thread | raw e-mail | index | archive | help
>> That's because Luoqi removed "curproc = p; /* XXX redundant */" from >> init_proc0(). >> Any idea where it should be done nowdays? Only after proc0 is fully initialised. >On i386, this is done earlier in init386(), probably you could add this line It is (was?) actually done in locore.s just before calling init386(). I removed it to fix the (curproc != NULL) sanity checks in trap_pfault() (traps can happen when early when unmapped memory is accessed in ddb). I'll try setting curproc at the end of proc0_init() (it should not be set until after after proc0.p_stats has been initialised, in case hardclock() is called; hardclock() currently isn't called that early). The comment about initialising curproc is now misplaced; previously it was backwards. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199904282200.IAA05572>