diff options
| author | kib <kib@FreeBSD.org> | 2007-11-05 11:36:16 +0000 |
|---|---|---|
| committer | kib <kib@FreeBSD.org> | 2007-11-05 11:36:16 +0000 |
| commit | 9ae733819b7cdf0eef51ba1b18d39feb136a9dbf (patch) | |
| tree | cccec36134d2cab5ce1eabb67dcaab5981e9beb0 /sys/arm/xscale/i8134x | |
| parent | f3f033b9b90ea07350fbe7483af3d9636cb31d1d (diff) | |
| download | FreeBSD-src-9ae733819b7cdf0eef51ba1b18d39feb136a9dbf.zip FreeBSD-src-9ae733819b7cdf0eef51ba1b18d39feb136a9dbf.tar.gz | |
Fix for the panic("vm_thread_new: kstack allocation failed") and
silent NULL pointer dereference in the i386 and sparc64 pmap_pinit()
when the kmem_alloc_nofault() failed to allocate address space. Both
functions now return error instead of panicing or dereferencing NULL.
As consequence, vmspace_exec() and vmspace_unshare() returns the errno
int. struct vmspace arg was added to vm_forkproc() to avoid dealing
with failed allocation when most of the fork1() job is already done.
The kernel stack for the thread is now set up in the thread_alloc(),
that itself may return NULL. Also, allocation of the first process
thread is performed in the fork1() to properly deal with stack
allocation failure. proc_linkup() is separated into proc_linkup()
called from fork1(), and proc_linkup0(), that is used to set up the
kernel process (was known as swapper).
In collaboration with: Peter Holm
Reviewed by: jhb
Diffstat (limited to 'sys/arm/xscale/i8134x')
| -rw-r--r-- | sys/arm/xscale/i8134x/crb_machdep.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/arm/xscale/i8134x/crb_machdep.c b/sys/arm/xscale/i8134x/crb_machdep.c index d29693f..0c6475a 100644 --- a/sys/arm/xscale/i8134x/crb_machdep.c +++ b/sys/arm/xscale/i8134x/crb_machdep.c @@ -409,7 +409,7 @@ initarm(void *arg, void *arg2) #ifdef KSE proc_linkup(&proc0, &ksegrp0, &thread0); #else - proc_linkup(&proc0, &thread0); + proc_linkup0(&proc0, &thread0); #endif thread0.td_kstack = kernelstack.pv_va; thread0.td_pcb = (struct pcb *) |
