diff options
| author |   beck <beck@openbsd.org> | 2007-10-18 20:26:55 +0000 |
|---|---|---|
| committer | beck <beck@openbsd.org> | 2007-10-18 20:26:55 +0000 |
| commit | 8059b48d83fad5deefc5a6bb0e8ce0c55bb46997 (patch) | |
| tree | 0c15020013d00cc6ace2ae5791369d57207aa7ba /sys | |
| parent | sync (diff) | |
| download | wireguard-openbsd-8059b48d83fad5deefc5a6bb0e8ce0c55bb46997.tar.xz wireguard-openbsd-8059b48d83fad5deefc5a6bb0e8ce0c55bb46997.zip | |
Correct possible spl problem in buffer cleaning daemon - the buffer cleaning
daemon requires splbio when doing dirty buffer queue manipulation. Since
version 1.88 of vfs_bio.c, it was possible to break out of the processing
loop when the cleaner had been running long enough, and this early exit would
mean a future pass through would manipulate the buffer queues not at splbio.
This change corrects this.
ok krw@, deraadt@, tedu@, thib@
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/kern/vfs_bio.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/kern/vfs_bio.c b/sys/kern/vfs_bio.c index b40113a0844..fffb0a5db25 100644 --- a/sys/kern/vfs_bio.c +++ b/sys/kern/vfs_bio.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_bio.c,v 1.100 2007/09/15 10:10:37 martin Exp $ */ +/* $OpenBSD: vfs_bio.c,v 1.101 2007/10/18 20:26:55 beck Exp $ */ /* $NetBSD: vfs_bio.c,v 1.44 1996/06/11 11:15:36 pk Exp $ */ /*- @@ -1122,10 +1122,10 @@ buf_daemon(struct proc *p) /* Never allow processing to run for more than 1 sec */ getmicrouptime(&tv); timersub(&tv, &starttime, &timediff); + s = splbio(); if (timediff.tv_sec) break; - s = splbio(); } } } |