diff options
author | dillon <dillon@FreeBSD.org> | 2002-03-27 05:39:23 +0000 |
---|---|---|
committer | dillon <dillon@FreeBSD.org> | 2002-03-27 05:39:23 +0000 |
commit | dc5aafeb94ddee4f835e390dffaecbb0eec5d5e2 (patch) | |
tree | 8233f61cf29e01829b91c6a5cf27defe60e6b8d8 /sys/ia64 | |
parent | 9b5143f94f573dc8954cb0913f3edb055e6caf0f (diff) | |
download | FreeBSD-src-dc5aafeb94ddee4f835e390dffaecbb0eec5d5e2.zip FreeBSD-src-dc5aafeb94ddee4f835e390dffaecbb0eec5d5e2.tar.gz |
Compromise for critical*()/cpu_critical*() recommit. Cleanup the interrupt
disablement assumptions in kern_fork.c by adding another API call,
cpu_critical_fork_exit(). Cleanup the td_savecrit field by moving it
from MI to MD. Temporarily move cpu_critical*() from <arch>/include/cpufunc.h
to <arch>/<arch>/critical.c (stage-2 will clean this up).
Implement interrupt deferral for i386 that allows interrupts to remain
enabled inside critical sections. This also fixes an IPI interlock bug,
and requires uses of icu_lock to be enclosed in a true interrupt disablement.
This is the stage-1 commit. Stage-2 will occur after stage-1 has stabilized,
and will move cpu_critical*() into its own header file(s) + other things.
This commit may break non-i386 architectures in trivial ways. This should
be temporary.
Reviewed by: core
Approved by: core
Diffstat (limited to 'sys/ia64')
-rw-r--r-- | sys/ia64/ia64/critical.c | 58 | ||||
-rw-r--r-- | sys/ia64/include/cpufunc.h | 17 | ||||
-rw-r--r-- | sys/ia64/include/proc.h | 1 |
3 files changed, 64 insertions, 12 deletions
diff --git a/sys/ia64/ia64/critical.c b/sys/ia64/ia64/critical.c new file mode 100644 index 0000000..77a6c83 --- /dev/null +++ b/sys/ia64/ia64/critical.c @@ -0,0 +1,58 @@ +/*- + * Copyright (c) 2001 Matthew Dillon. This code is distributed under + * the BSD copyright, /usr/src/COPYRIGHT. + * + * $FreeBSD$ + */ + +#include <sys/param.h> +#include <sys/systm.h> +#include <sys/pcpu.h> +#include <sys/eventhandler.h> /* XX */ +#include <sys/ktr.h> /* XX */ +#include <sys/signalvar.h> +#include <sys/sysproto.h> /* XX */ +#include <sys/kernel.h> +#include <sys/proc.h> +#include <sys/lock.h> +#include <sys/mutex.h> +#include <sys/sysctl.h> +#include <sys/ucontext.h> + +void +cpu_critical_enter(void) +{ + struct thread *td = curthread; + + td->td_md.md_savecrit = intr_disable(); +} + +void +cpu_critical_exit(void) +{ + struct thread *td = curthread; + + intr_restore(td->td_md.md_savecrit); +} + +/* + * cpu_critical_fork_exit() - cleanup after fork + */ +void +cpu_critical_fork_exit(void) +{ + struct thread *td = curthread; + + td->td_critnest = 1; + td->td_md.md_savecrit = (ia64_get_psr() | IA64_PSR_I); +} + +/* + * cpu_thread_link() - thread linkup, initialize machine-dependant fields + */ +void +cpu_thread_link(struct thread *td) +{ + td->td_md.md_savecrit = 0; +} + diff --git a/sys/ia64/include/cpufunc.h b/sys/ia64/include/cpufunc.h index 713ecf4..e7cf818 100644 --- a/sys/ia64/include/cpufunc.h +++ b/sys/ia64/include/cpufunc.h @@ -34,7 +34,7 @@ #include <sys/types.h> #include <machine/ia64_cpu.h> -#define CRITICAL_FORK (ia64_get_psr() | IA64_PSR_I) +struct thread; #ifdef __GNUC__ @@ -300,17 +300,10 @@ intr_restore(critical_t psr) __asm __volatile ("mov psr.l=%0;; srlz.d" :: "r" (psr)); } -static __inline critical_t -cpu_critical_enter(void) -{ - return (intr_disable()); -} - -static __inline void -cpu_critical_exit(critical_t psr) -{ - intr_restore(psr); -} +void cpu_critical_enter(void); +void cpu_critical_exit(void); +void cpu_critical_fork_exit(void); +void cpu_thread_link(struct thread *td); #endif /* _KERNEL */ diff --git a/sys/ia64/include/proc.h b/sys/ia64/include/proc.h index c9ae251..4daead6 100644 --- a/sys/ia64/include/proc.h +++ b/sys/ia64/include/proc.h @@ -39,6 +39,7 @@ struct mdthread { u_long md_flags; void *md_kstackvirt; /* virtual address of td_kstack */ vm_offset_t md_bspstore; /* initial ar.bspstore */ + register_t md_savecrit; }; #define MDP_FPUSED 0x0001 /* Process used the FPU */ |