diff options
author | kib <kib@FreeBSD.org> | 2010-12-29 22:26:49 +0000 |
---|---|---|
committer | kib <kib@FreeBSD.org> | 2010-12-29 22:26:49 +0000 |
commit | f112887cab1c7c8017f13e95138e3c4d33992d78 (patch) | |
tree | 85e3ca190990ebd95923f075fd1424022c5dee91 /sys/kern/vfs_bio.c | |
parent | 9af03ea4fc729f0a666186052c3a2a6bedc431ca (diff) | |
download | FreeBSD-src-f112887cab1c7c8017f13e95138e3c4d33992d78.zip FreeBSD-src-f112887cab1c7c8017f13e95138e3c4d33992d78.tar.gz |
Remove OBJ_CLEANING flag. The vfs_setdirty_locked_object() is the only
consumer of the flag, and it used the flag because OBJ_MIGHTBEDIRTY
was cleared early in vm_object_page_clean, before the cleaning pass
was done. This is no longer true after r216799.
Moreover, since OBJ_CLEANING is a flag, and not the counter, it could
be reset too prematurely when parallel vm_object_page_clean() are
performed.
Reviewed by: alc (as a part of the bigger patch)
MFC after: 1 month (after r216799 is merged)
Diffstat (limited to 'sys/kern/vfs_bio.c')
-rw-r--r-- | sys/kern/vfs_bio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/kern/vfs_bio.c b/sys/kern/vfs_bio.c index 0147bff..b1e278d 100644 --- a/sys/kern/vfs_bio.c +++ b/sys/kern/vfs_bio.c @@ -2523,7 +2523,7 @@ vfs_setdirty_locked_object(struct buf *bp) * We qualify the scan for modified pages on whether the * object has been flushed yet. */ - if (object->flags & (OBJ_MIGHTBEDIRTY|OBJ_CLEANING)) { + if ((object->flags & OBJ_MIGHTBEDIRTY) != 0) { vm_offset_t boffset; vm_offset_t eoffset; |