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));

Reply via email to