diff options
| author | das <das@FreeBSD.org> | 2004-09-19 18:34:17 +0000 |
|---|---|---|
| committer | das <das@FreeBSD.org> | 2004-09-19 18:34:17 +0000 |
| commit | 8b64b8f0288600f31ffc708cf66e8b75e3430949 (patch) | |
| tree | a883b85688bf1e9cee8a5852cea1fe606fa15bb0 /sys/vm/vm_glue.c | |
| parent | ca50096305a4f44d3cd3853b12eeb36b16115ab0 (diff) | |
| download | FreeBSD-src-8b64b8f0288600f31ffc708cf66e8b75e3430949.zip FreeBSD-src-8b64b8f0288600f31ffc708cf66e8b75e3430949.tar.gz | |
The zone from which proc structures are allocated is marked
UMA_ZONE_NOFREE to guarantee type stability, so proc_fini() should
never be called. Move an assertion from proc_fini() to proc_dtor()
and garbage-collect the rest of the unreachable code. I have retained
vm_proc_dispose(), since I consider its disuse a bug.
Diffstat (limited to 'sys/vm/vm_glue.c')
| -rw-r--r-- | sys/vm/vm_glue.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sys/vm/vm_glue.c b/sys/vm/vm_glue.c index 9d5fa42..af0ed3e 100644 --- a/sys/vm/vm_glue.c +++ b/sys/vm/vm_glue.c @@ -282,7 +282,10 @@ vm_proc_new(struct proc *p) /* * Dispose the U area for a process that has exited. * This routine directly impacts the exit perf of a process. - * XXX proc_zone is marked UMA_ZONE_NOFREE, so this should never be called. + * + * XXX UNUSED + * U areas of free proc structures are no longer freed and are never + * swapped out. Ideally we would free U areas lazily, when low on memory. */ void vm_proc_dispose(struct proc *p) |
