diff options
author | Dave Voutila <dv@cvs.openbsd.org> | 2024-05-20 17:03:37 +0000 |
---|---|---|
committer | Dave Voutila <dv@cvs.openbsd.org> | 2024-05-20 17:03:37 +0000 |
commit | 83bfd105b5d84fe73b9fa39ce81385cf35daefb8 (patch) | |
tree | 6581487f23070a038af6e4ffaef726a50aedda18 /sys/uvm | |
parent | a4a26d1667c36dbf9f93b1ac0d9023fe500b6a09 (diff) |
Remove always-false condition in amap_free KASSERT.
The if-condition immediately preceding it already checks the value
so the first part of the KASSERT is redundant.
ok mpi@, miod@, jca@
Diffstat (limited to 'sys/uvm')
-rw-r--r-- | sys/uvm/uvm_amap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/uvm/uvm_amap.c b/sys/uvm/uvm_amap.c index 1d1a5f7b2fe..d9296efad06 100644 --- a/sys/uvm/uvm_amap.c +++ b/sys/uvm/uvm_amap.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uvm_amap.c,v 1.94 2024/04/17 13:17:31 mpi Exp $ */ +/* $OpenBSD: uvm_amap.c,v 1.95 2024/05/20 17:03:36 dv Exp $ */ /* $NetBSD: uvm_amap.c,v 1.27 2000/11/25 06:27:59 chs Exp $ */ /* @@ -450,7 +450,7 @@ amap_free(struct vm_amap *amap) KASSERT((amap->am_flags & AMAP_SWAPOFF) == 0); if (amap->am_lock != NULL) { - KASSERT(amap->am_lock == NULL || !rw_write_held(amap->am_lock)); + KASSERT(!rw_write_held(amap->am_lock)); rw_obj_free(amap->am_lock); } |