Module Name: src Committed By: ozaki-r Date: Wed Feb 7 08:50:13 UTC 2018
Modified Files: src/sys/ufs/chfs: ebh.c Log Message: Remove unnecessary assertions KASSERT(!rw_lock_held()) just before rw_destroy() is useless because rw_destroy does more strict check and provides better information on failure. To generate a diff of this commit: cvs rdiff -u -r1.6 -r1.7 src/sys/ufs/chfs/ebh.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/ufs/chfs/ebh.c diff -u src/sys/ufs/chfs/ebh.c:1.6 src/sys/ufs/chfs/ebh.c:1.7 --- src/sys/ufs/chfs/ebh.c:1.6 Sat Feb 7 04:21:11 2015 +++ src/sys/ufs/chfs/ebh.c Wed Feb 7 08:50:13 2018 @@ -1,4 +1,4 @@ -/* $NetBSD: ebh.c,v 1.6 2015/02/07 04:21:11 christos Exp $ */ +/* $NetBSD: ebh.c,v 1.7 2018/02/07 08:50:13 ozaki-r Exp $ */ /*- * Copyright (c) 2010 Department of Software Engineering, @@ -656,7 +656,6 @@ leb_read_unlock(struct chfs_ebh *ebh, in if (le->users == 0) { le = RB_REMOVE(ltree_rbtree, &ebh->ltree, le); if (le) { - KASSERT(!rw_lock_held(&le->mutex)); rw_destroy(&le->mutex); kmem_free(le, sizeof(struct chfs_ltree_entry)); @@ -713,7 +712,6 @@ leb_write_unlock(struct chfs_ebh *ebh, i if (le->users == 0) { RB_REMOVE(ltree_rbtree, &ebh->ltree, le); - KASSERT(!rw_lock_held(&le->mutex)); rw_destroy(&le->mutex); kmem_free(le, sizeof(struct chfs_ltree_entry));