diff options
| author | Konstantin Belousov <kib@FreeBSD.org> | 2008-12-29 12:45:11 +0000 |
|---|---|---|
| committer | Konstantin Belousov <kib@FreeBSD.org> | 2008-12-29 12:45:11 +0000 |
| commit | 22a448c4d91c826c96ddea12c82c82545e965ec9 (patch) | |
| tree | 3e48d98f96707e37fafe396ec8c27fa0b15ef680 /sys/kern/kern_proc.c | |
| parent | c990bf089616d4efa5f72b4e5fedad34a1b4dcf5 (diff) | |
| download | src-22a448c4d91c826c96ddea12c82c82545e965ec9.tar.gz src-22a448c4d91c826c96ddea12c82c82545e965ec9.zip | |
vm_map_lock_read() does not increment map->timestamp, so we should
compare map->timestamp with saved timestamp after map read lock is
reacquired, not with saved timestamp + 1. The only consequence of the +1
was unconditional lookup of the next map entry, though.
Tested by: pho
Approved by: des
MFC after: 2 weeks
Notes
Notes:
svn path=/head/; revision=186563
Diffstat (limited to 'sys/kern/kern_proc.c')
| -rw-r--r-- | sys/kern/kern_proc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/kern/kern_proc.c b/sys/kern/kern_proc.c index 9c6225f45322..3d994ba5f637 100644 --- a/sys/kern/kern_proc.c +++ b/sys/kern/kern_proc.c @@ -1500,7 +1500,7 @@ sysctl_kern_proc_ovmmap(SYSCTL_HANDLER_ARGS) vm_map_lock_read(map); if (error) break; - if (last_timestamp + 1 != map->timestamp) { + if (last_timestamp != map->timestamp) { vm_map_lookup_entry(map, addr - 1, &tmp_entry); entry = tmp_entry; } @@ -1677,7 +1677,7 @@ sysctl_kern_proc_vmmap(SYSCTL_HANDLER_ARGS) vm_map_lock_read(map); if (error) break; - if (last_timestamp + 1 != map->timestamp) { + if (last_timestamp != map->timestamp) { vm_map_lookup_entry(map, addr - 1, &tmp_entry); entry = tmp_entry; } |