From owner-cvs-all  Wed Apr 28 15: 0:42 1999
Delivered-To: cvs-all@freebsd.org
Received: from godzilla.zeta.org.au (godzilla.zeta.org.au [203.26.10.9])
	by hub.freebsd.org (Postfix) with ESMTP
	id 4B99215520; Wed, 28 Apr 1999 15:00:36 -0700 (PDT)
	(envelope-from bde@godzilla.zeta.org.au)
Received: (from bde@localhost)
	by godzilla.zeta.org.au (8.8.7/8.8.7) id IAA05572;
	Thu, 29 Apr 1999 08:00:33 +1000
Date: Thu, 29 Apr 1999 08:00:33 +1000
From: Bruce Evans <bde@zeta.org.au>
Message-Id: <199904282200.IAA05572@godzilla.zeta.org.au>
To: dfr@nlsystems.com, luoqi@watermarkgroup.com,
	tejblum@arc.hq.cti.ru
Subject: Re: cvs commit: src/sys/alpha/alpha pmap.c
Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org, dt@FreeBSD.org,
	luoqi@FreeBSD.org
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk

>> 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