svn commit: r348826 - stable/12/sys/vm
Mark Johnston
markj at FreeBSD.org
Sun Jun 9 03:29:29 UTC 2019
Author: markj
Date: Sun Jun 9 03:29:26 2019
New Revision: 348826
URL: https://svnweb.freebsd.org/changeset/base/348826
Log:
MFC r348502:
Add a vm_page_wired() predicate.
Modified:
stable/12/sys/vm/memguard.c
stable/12/sys/vm/swap_pager.c
stable/12/sys/vm/vm_fault.c
stable/12/sys/vm/vm_object.c
stable/12/sys/vm/vm_page.c
stable/12/sys/vm/vm_page.h
stable/12/sys/vm/vm_pageout.c
Directory Properties:
stable/12/ (props changed)
Modified: stable/12/sys/vm/memguard.c
==============================================================================
--- stable/12/sys/vm/memguard.c Sun Jun 9 02:32:52 2019 (r348825)
+++ stable/12/sys/vm/memguard.c Sun Jun 9 03:29:26 2019 (r348826)
@@ -259,7 +259,7 @@ v2sizep(vm_offset_t va)
if (pa == 0)
panic("MemGuard detected double-free of %p", (void *)va);
p = PHYS_TO_VM_PAGE(pa);
- KASSERT(p->wire_count != 0 && p->queue == PQ_NONE,
+ KASSERT(vm_page_wired(p) && p->queue == PQ_NONE,
("MEMGUARD: Expected wired page %p in vtomgfifo!", p));
return (&p->plinks.memguard.p);
}
@@ -274,7 +274,7 @@ v2sizev(vm_offset_t va)
if (pa == 0)
panic("MemGuard detected double-free of %p", (void *)va);
p = PHYS_TO_VM_PAGE(pa);
- KASSERT(p->wire_count != 0 && p->queue == PQ_NONE,
+ KASSERT(vm_page_wired(p) && p->queue == PQ_NONE,
("MEMGUARD: Expected wired page %p in vtomgfifo!", p));
return (&p->plinks.memguard.v);
}
Modified: stable/12/sys/vm/swap_pager.c
==============================================================================
--- stable/12/sys/vm/swap_pager.c Sun Jun 9 02:32:52 2019 (r348825)
+++ stable/12/sys/vm/swap_pager.c Sun Jun 9 03:29:26 2019 (r348826)
@@ -1681,7 +1681,7 @@ swp_pager_force_pagein(vm_object_t object, vm_pindex_t
vm_page_dirty(m);
#ifdef INVARIANTS
vm_page_lock(m);
- if (m->wire_count == 0 && m->queue == PQ_NONE)
+ if (!vm_page_wired(m) && m->queue == PQ_NONE)
panic("page %p is neither wired nor queued", m);
vm_page_unlock(m);
#endif
Modified: stable/12/sys/vm/vm_fault.c
==============================================================================
--- stable/12/sys/vm/vm_fault.c Sun Jun 9 02:32:52 2019 (r348825)
+++ stable/12/sys/vm/vm_fault.c Sun Jun 9 03:29:26 2019 (r348826)
@@ -1004,7 +1004,7 @@ readrest:
*/
if (rv == VM_PAGER_ERROR || rv == VM_PAGER_BAD) {
vm_page_lock(fs.m);
- if (fs.m->wire_count == 0)
+ if (!vm_page_wired(fs.m))
vm_page_free(fs.m);
else
vm_page_xunbusy_maybelocked(fs.m);
@@ -1027,7 +1027,7 @@ readrest:
*/
if (fs.object != fs.first_object) {
vm_page_lock(fs.m);
- if (fs.m->wire_count == 0)
+ if (!vm_page_wired(fs.m))
vm_page_free(fs.m);
else
vm_page_xunbusy_maybelocked(fs.m);
@@ -1794,7 +1794,7 @@ again:
vm_page_wire(dst_m);
vm_page_unlock(dst_m);
} else {
- KASSERT(dst_m->wire_count > 0,
+ KASSERT(vm_page_wired(dst_m),
("dst_m %p is not wired", dst_m));
}
} else {
Modified: stable/12/sys/vm/vm_object.c
==============================================================================
--- stable/12/sys/vm/vm_object.c Sun Jun 9 02:32:52 2019 (r348825)
+++ stable/12/sys/vm/vm_object.c Sun Jun 9 03:29:26 2019 (r348826)
@@ -742,7 +742,7 @@ vm_object_terminate_pages(vm_object_t object)
*/
vm_page_change_lock(p, &mtx);
p->object = NULL;
- if (p->wire_count != 0)
+ if (vm_page_wired(p))
continue;
VM_CNT_INC(v_pfree);
vm_page_free(p);
@@ -1617,7 +1617,7 @@ vm_object_collapse_scan(vm_object_t object, int op)
vm_page_lock(p);
KASSERT(!pmap_page_is_mapped(p),
("freeing mapped page %p", p));
- if (p->wire_count == 0)
+ if (!vm_page_wired(p))
vm_page_free(p);
else
vm_page_remove(p);
@@ -1661,7 +1661,7 @@ vm_object_collapse_scan(vm_object_t object, int op)
vm_page_lock(p);
KASSERT(!pmap_page_is_mapped(p),
("freeing mapped page %p", p));
- if (p->wire_count == 0)
+ if (!vm_page_wired(p))
vm_page_free(p);
else
vm_page_remove(p);
@@ -1966,7 +1966,7 @@ again:
VM_OBJECT_WLOCK(object);
goto again;
}
- if (p->wire_count != 0) {
+ if (vm_page_wired(p)) {
if ((options & OBJPR_NOTMAPPED) == 0 &&
object->ref_count != 0)
pmap_remove_all(p);
Modified: stable/12/sys/vm/vm_page.c
==============================================================================
--- stable/12/sys/vm/vm_page.c Sun Jun 9 02:32:52 2019 (r348825)
+++ stable/12/sys/vm/vm_page.c Sun Jun 9 03:29:26 2019 (r348826)
@@ -2602,7 +2602,7 @@ retry:
error = ENOMEM;
goto unlock;
}
- KASSERT(m_new->wire_count == 0,
+ KASSERT(!vm_page_wired(m_new),
("page %p is wired", m_new));
/*
@@ -3399,7 +3399,7 @@ vm_page_activate(vm_page_t m)
vm_page_assert_locked(m);
- if (m->wire_count > 0 || (m->oflags & VPO_UNMANAGED) != 0)
+ if (vm_page_wired(m) || (m->oflags & VPO_UNMANAGED) != 0)
return;
if (vm_page_queue(m) == PQ_ACTIVE) {
if (m->act_count < ACT_INIT)
@@ -3474,7 +3474,7 @@ vm_page_free_prep(vm_page_t m)
m->valid = 0;
vm_page_undirty(m);
- if (m->wire_count != 0)
+ if (vm_page_wired(m) != 0)
panic("vm_page_free_prep: freeing wired page %p", m);
if (m->hold_count != 0) {
m->flags &= ~PG_ZERO;
@@ -3575,7 +3575,7 @@ vm_page_wire(vm_page_t m)
m));
return;
}
- if (m->wire_count == 0) {
+ if (!vm_page_wired(m)) {
KASSERT((m->oflags & VPO_UNMANAGED) == 0 ||
m->queue == PQ_NONE,
("vm_page_wire: unmanaged page %p is queued", m));
@@ -3653,7 +3653,7 @@ vm_page_unwire_noq(vm_page_t m)
("vm_page_unwire: fictitious page %p's wire count isn't one", m));
return (false);
}
- if (m->wire_count == 0)
+ if (!vm_page_wired(m))
panic("vm_page_unwire: page %p's wire count is zero", m);
m->wire_count--;
if (m->wire_count == 0) {
@@ -3675,7 +3675,7 @@ vm_page_deactivate(vm_page_t m)
vm_page_assert_locked(m);
- if (m->wire_count > 0 || (m->oflags & VPO_UNMANAGED) != 0)
+ if (vm_page_wired(m) || (m->oflags & VPO_UNMANAGED) != 0)
return;
if (!vm_page_inactive(m)) {
@@ -3699,7 +3699,7 @@ vm_page_deactivate_noreuse(vm_page_t m)
vm_page_assert_locked(m);
- if (m->wire_count > 0 || (m->oflags & VPO_UNMANAGED) != 0)
+ if (vm_page_wired(m) || (m->oflags & VPO_UNMANAGED) != 0)
return;
if (!vm_page_inactive(m)) {
@@ -3721,7 +3721,7 @@ vm_page_launder(vm_page_t m)
{
vm_page_assert_locked(m);
- if (m->wire_count > 0 || (m->oflags & VPO_UNMANAGED) != 0)
+ if (vm_page_wired(m) || (m->oflags & VPO_UNMANAGED) != 0)
return;
if (vm_page_in_laundry(m))
@@ -3742,7 +3742,7 @@ vm_page_unswappable(vm_page_t m)
{
vm_page_assert_locked(m);
- KASSERT(m->wire_count == 0 && (m->oflags & VPO_UNMANAGED) == 0,
+ KASSERT(!vm_page_wired(m) && (m->oflags & VPO_UNMANAGED) == 0,
("page %p already unswappable", m));
vm_page_dequeue(m);
Modified: stable/12/sys/vm/vm_page.h
==============================================================================
--- stable/12/sys/vm/vm_page.h Sun Jun 9 02:32:52 2019 (r348825)
+++ stable/12/sys/vm/vm_page.h Sun Jun 9 03:29:26 2019 (r348826)
@@ -820,5 +820,12 @@ vm_page_held(vm_page_t m)
return (m->hold_count > 0 || m->wire_count > 0);
}
+static inline bool
+vm_page_wired(vm_page_t m)
+{
+
+ return (m->wire_count > 0);
+}
+
#endif /* _KERNEL */
#endif /* !_VM_PAGE_ */
Modified: stable/12/sys/vm/vm_pageout.c
==============================================================================
--- stable/12/sys/vm/vm_pageout.c Sun Jun 9 02:32:52 2019 (r348825)
+++ stable/12/sys/vm/vm_pageout.c Sun Jun 9 03:29:26 2019 (r348826)
@@ -753,7 +753,7 @@ recheck:
*/
if (m->hold_count != 0)
continue;
- if (m->wire_count != 0) {
+ if (vm_page_wired(m)) {
vm_page_dequeue_deferred(m);
continue;
}
@@ -1204,7 +1204,7 @@ act_scan:
/*
* Wired pages are dequeued lazily.
*/
- if (m->wire_count != 0) {
+ if (vm_page_wired(m)) {
vm_page_dequeue_deferred(m);
continue;
}
@@ -1431,7 +1431,7 @@ recheck:
addl_page_shortage++;
goto reinsert;
}
- if (m->wire_count != 0) {
+ if (vm_page_wired(m)) {
vm_page_dequeue_deferred(m);
continue;
}
More information about the svn-src-all
mailing list