summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Pieuchot <mpi@cvs.openbsd.org>2022-06-30 13:54:38 +0000
committerMartin Pieuchot <mpi@cvs.openbsd.org>2022-06-30 13:54:38 +0000
commit057ab4f8f89025aff5c7997450410c070a33f65a (patch)
tree1667adb39122a866dd45eb0cadf2995a00bd99fb
parent23facc128db37d76dff92ac7506a361f22662983 (diff)
Remove an incorrect KASSERT() introduced in previous.
If uvm_pagermapin() fails that doesn't mean the emergency pages aren't in use. In OOM situation, a single segment can still be held by an asynchronous write so the second call to uvm_pagermapin() can fail.
-rw-r--r--sys/uvm/uvm_swap.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/uvm/uvm_swap.c b/sys/uvm/uvm_swap.c
index 609c54c5034..6516adbed5f 100644
--- a/sys/uvm/uvm_swap.c
+++ b/sys/uvm/uvm_swap.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: uvm_swap.c,v 1.158 2022/06/28 19:39:54 mpi Exp $ */
+/* $OpenBSD: uvm_swap.c,v 1.159 2022/06/30 13:54:37 mpi Exp $ */
/* $NetBSD: uvm_swap.c,v 1.40 2000/11/17 11:39:39 mrg Exp $ */
/*
@@ -1725,7 +1725,6 @@ uvm_swap_io(struct vm_page **pps, int startslot, int npages, int flags)
bouncekva = uvm_pagermapin(tpps, npages, swmapflags);
if (bouncekva == 0) {
- KASSERT(tpps[0] != oompps[0]);
pool_put(&bufpool, bp);
uvm_pagermapout(kva, npages);
uvm_swap_freepages(tpps, npages);