[patch] Wrong assertion in kern_umtx.c
Eric van Gyzen
eric at vangyzen.net
Thu Nov 13 17:39:50 UTC 2014
There is a [practically] tautological assertion in kern_umtx.c. I have
not even compile-tested the following patch. I'll test it when I have
time. I'd be grateful if someone beats me to it.
Eric
diff --git a/sys/kern/kern_umtx.c b/sys/kern/kern_umtx.c
index 33fdf71..c6b42c0 100644
--- a/sys/kern/kern_umtx.c
+++ b/sys/kern/kern_umtx.c
@@ -169,7 +169,7 @@ struct umtxq_chain {
};
#define UMTXQ_LOCKED_ASSERT(uc)
mtx_assert(&(uc)->uc_lock, MA_OWNED)
-#define UMTXQ_BUSY_ASSERT(uc) KASSERT(&(uc)->uc_busy, ("umtx
chain is not busy"))
+#define UMTXQ_BUSY_ASSERT(uc) KASSERT((uc)->uc_busy, ("umtx
chain is not busy"))
/*
* Don't propagate time-sharing priority, there is a security reason,
More information about the freebsd-current
mailing list