Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 22 Feb 2001 16:04:35 +0000 (GMT)
From:      Doug Rabson <dfr@nlsystems.com>
To:        John Baldwin <jhb@FreeBSD.org>
Cc:        <cvs-all@FreeBSD.org>, <cvs-committers@FreeBSD.org>
Subject:   Re: cvs commit: src/sys/alpha/alpha swtch.s
Message-ID:  <Pine.BSF.4.33.0102221602230.32872-100000@herring.nlsystems.com>
In-Reply-To: <XFMail.010222075403.jhb@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 22 Feb 2001, John Baldwin wrote:

>
> On 22-Feb-01 Doug Rabson wrote:
> > On Thu, 22 Feb 2001, John Baldwin wrote:
> >
> >> jhb         2001/02/22 07:06:20 PST
> >>
> >>   Modified files:
> >>     sys/alpha/alpha      swtch.s
> >>   Log:
> >>   Cleanup and clarify the comments above switch_trampoline().
> >
> > Should we unify the name for this thing? Alpha and ia64 use
> > switch_trampoline (derived from the NetBSD/alpha code) and i386 uses
> > fork_trampoline, which is probably a slightly better name.
>
> I've thought about doing that, yes.  If you want, I can stick it on my todo
> list.  Also, I have managed to make userret() and ast() (modulo a few commits
> coming up in the next 15 minutes or so) the same across all architectures.
> Do you think that given that it would be feasible to move those two functions
> to sys/kern/subr_trap.c and let them be MI?

That seems like a good direction to take.

>
> Also, as a sidenote, I think that addupc_task() should take a uniptr_t, not a
> uint64_t as its second argument..

It should certainly be uintptr_t.

-- 
Doug Rabson				Mail:  dfr@nlsystems.com
					Phone: +44 20 8348 6160



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?Pine.BSF.4.33.0102221602230.32872-100000>