]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Change the second (and last) argument of cpu_set_upcall(). Previously
authorMarcel Moolenaar <marcel@FreeBSD.org>
Wed, 4 Jun 2003 21:13:21 +0000 (21:13 +0000)
committerMarcel Moolenaar <marcel@FreeBSD.org>
Wed, 4 Jun 2003 21:13:21 +0000 (21:13 +0000)
commit11e0f8e16d4180acd1ef277011e9a30fc72ab6c7
tree19ddbe55bfa5e54d0c6ec4f04832d6c8fdba3e5b
parentba90ccc69ae68ad013e097fc9cc2e4405af74b3f
Change the second (and last) argument of cpu_set_upcall(). Previously
we were passing in a void* representing the PCB of the parent thread.
Now we pass a pointer to the parent thread itself.
The prime reason for this change is to allow cpu_set_upcall() to copy
(parts of) the trapframe instead of having it done in MI code in each
caller of cpu_set_upcall(). Copying the trapframe cannot always be
done with a simply bcopy() or may not always be optimal that way. On
ia64 specifically the trapframe contains information that is specific
to an entry into the kernel and can only be used by the corresponding
exit from the kernel. A trapframe copied verbatim from another frame
is in most cases useless without some additional normalization.

Note that this change removes the assignment to td->td_frame in some
implementations of cpu_set_upcall(). The assignment is redundant.
A previous call to cpu_thread_setup() already did the exact same
assignment. An added benefit of removing the redundant assignment is
that we can now change td_pcb without nasty side-effects.

This change officially marks the ability on ia64 for 1:1 threading.

Not tested on: amd64, powerpc
Compile & boot tested on: alpha, sparc64
Functionally tested on: i386, ia64
sys/alpha/alpha/vm_machdep.c
sys/i386/i386/vm_machdep.c
sys/ia64/ia64/vm_machdep.c
sys/kern/kern_kse.c
sys/kern/kern_thr.c
sys/kern/kern_thread.c
sys/powerpc/aim/vm_machdep.c
sys/powerpc/powerpc/vm_machdep.c
sys/sparc64/sparc64/vm_machdep.c
sys/sys/proc.h