diff options
| author | Anton Rang <rang@acm.org> | 2022-11-09 20:13:01 +0000 |
|---|---|---|
| committer | Eric van Gyzen <vangyzen@FreeBSD.org> | 2022-11-09 20:28:03 +0000 |
| commit | cfbf1da0deede6b82ac197471b6cd439706a2dea (patch) | |
| tree | f481135a6f880d2ec524359ba5b8d48ba2e290ac | |
| parent | 460ed6106cf0854caff62e4eeba8ffcd00ab0690 (diff) | |
| download | src-cfbf1da0deede6b82ac197471b6cd439706a2dea.tar.gz src-cfbf1da0deede6b82ac197471b6cd439706a2dea.zip | |
vm_page_unswappable: remove wrong assertion
markj says:
...the assertion is incorrect and should simply be removed.
It has been racy since we removed the use of the page hash
lock to synchronize wiring of pages.
PR: 267621
Reviewed by: markj, Anton Rang <rang@acm.org>
MFC after: 1 week
Sponsored by: Dell Inc.
Differential Revision: https://reviews.freebsd.org/D37320
| -rw-r--r-- | sys/vm/vm_page.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c index 8895b0f1da5a..4f97b84ee205 100644 --- a/sys/vm/vm_page.c +++ b/sys/vm/vm_page.c @@ -4200,7 +4200,8 @@ void vm_page_unswappable(vm_page_t m) { - KASSERT(!vm_page_wired(m) && (m->oflags & VPO_UNMANAGED) == 0, + VM_OBJECT_ASSERT_LOCKED(m->object); + KASSERT((m->oflags & VPO_UNMANAGED) == 0, ("page %p already unswappable", m)); vm_page_dequeue(m); |