Project

General

Profile

Actions

Bug #902

closed

Fix lockuninit

Added by nant about 16 years ago. Updated about 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

lockuninit was acquiring the spinlock embedded in struct lock, thus
incrementing the per-thread spinlock count. However, spin_uninit does
not decrement the count, resulting in a panic when trying to sleep
("lwkt_switch: still holding %d exclusive spinlocks!"). From now on,
require that the caller already holds the struct lock in question so
that lockuninit can rely on it being the sole owner of the lock.

Lots of help from: corecode@, aggelos

Comment from Aggelos: "That's what you get when commiting code without
any in-tree users."

diff --git a/sys/kern/kern_lock.c b/sys/kern/kern_lock.c
index c945ef5..5ae8c30 100644
--- a/sys/kern/kern_lock.c
+++ b/sys/kern/kern_lock.c
@ -518,12 +518,18 @ lockreinit(struct lock *lkp, char *wmesg, int
timo, int flags)
spin_unlock_wr(&lkp->lk_spinlock);
}

/*
* Requires that the caller is the exclusive owner of this lock.
+ */
void
lockuninit(struct lock *l) {
- spin_lock_wr(&l->lk_spinlock);
+ /*
+ * At this point we should have removed all the references to this lock
+ * so there can't be anyone waiting on it.
+ */
KKASSERT;
- l->lk_wmesg = "BUG";
+
spin_uninit(&l->lk_spinlock);
}

Actions

Also available in: Atom PDF