On Mon, Nov 21, 2011 at 10:36:57AM +0000, Sergey Kandaurov wrote: > Author: pluknet > Date: Mon Nov 21 10:36:57 2011 > New Revision: 227784 > URL: http://svn.freebsd.org/changeset/base/227784 > > Log: > Use the acquired reference to the vmspace instead of direct dereferencing > of p->p_vmspace like it is done in sysctl_kern_proc_vmmap(). > > Modified: > head/sys/kern/kern_proc.c > > Modified: head/sys/kern/kern_proc.c > ============================================================================== > --- head/sys/kern/kern_proc.c Mon Nov 21 08:12:36 2011 (r227783) > +++ head/sys/kern/kern_proc.c Mon Nov 21 10:36:57 2011 (r227784) > @@ -1528,7 +1528,7 @@ sysctl_kern_proc_ovmmap(SYSCTL_HANDLER_A > } > kve = malloc(sizeof(*kve), M_TEMP, M_WAITOK); > > - map = &p->p_vmspace->vm_map; /* XXXRW: More locking required? */ > + map = &vm->vm_map; /* XXXRW: More locking required? */ It makes sense to remove the XXXRW comment, from both places.
> vm_map_lock_read(map); > for (entry = map->header.next; entry != &map->header; > entry = entry->next) {
pgpeZ70gunFsP.pgp
Description: PGP signature