diff options
-rw-r--r-- | os/rt/include/chmtx.h | 4 | ||||
-rw-r--r-- | os/rt/src/chcond.c | 14 | ||||
-rw-r--r-- | readme.txt | 1 |
3 files changed, 9 insertions, 10 deletions
diff --git a/os/rt/include/chmtx.h b/os/rt/include/chmtx.h index 5f7793bdc..44e9ea42c 100644 --- a/os/rt/include/chmtx.h +++ b/os/rt/include/chmtx.h @@ -139,9 +139,9 @@ static inline bool chMtxQueueNotEmptyS(mutex_t *mp) { * @return A pointer to the next mutex in the stack.
* @retval NULL if the stack is empty.
*
- * @sclass
+ * @xclass
*/
-static inline mutex_t *chMtxGetNextMutexS(void) {
+static inline mutex_t *chMtxGetNextMutexX(void) {
return chThdGetSelfX()->mtxlist;
}
diff --git a/os/rt/src/chcond.c b/os/rt/src/chcond.c index 1881f1f63..8ee07a814 100644 --- a/os/rt/src/chcond.c +++ b/os/rt/src/chcond.c @@ -204,15 +204,14 @@ msg_t chCondWait(condition_variable_t *cp) { */
msg_t chCondWaitS(condition_variable_t *cp) {
thread_t *ctp = currp;
- mutex_t *mp;
+ mutex_t *mp = chMtxGetNextMutexX();
msg_t msg;
chDbgCheckClassS();
chDbgCheck(cp != NULL);
- chDbgAssert(ctp->mtxlist != NULL, "not owning a mutex");
+ chDbgAssert(mp != NULL, "not owning a mutex");
- /* Getting "current" mutex and releasing it.*/
- mp = chMtxGetNextMutexS();
+ /* Releasing "current" mutex.*/
chMtxUnlockS(mp);
/* Start waiting on the condition variable, on exit the mutex is taken
@@ -294,15 +293,14 @@ msg_t chCondWaitTimeout(condition_variable_t *cp, sysinterval_t timeout) { * @sclass
*/
msg_t chCondWaitTimeoutS(condition_variable_t *cp, sysinterval_t timeout) {
- mutex_t *mp;
+ mutex_t *mp = chMtxGetNextMutexX();
msg_t msg;
chDbgCheckClassS();
chDbgCheck((cp != NULL) && (timeout != TIME_IMMEDIATE));
- chDbgAssert(currp->mtxlist != NULL, "not owning a mutex");
+ chDbgAssert(mp != NULL, "not owning a mutex");
- /* Getting "current" mutex and releasing it.*/
- mp = chMtxGetNextMutexS();
+ /* Releasing "current" mutex.*/
chMtxUnlockS(mp);
/* Start waiting on the condition variable, on exit the mutex is taken
diff --git a/readme.txt b/readme.txt index 95c3ea8a1..deab2bbd1 100644 --- a/readme.txt +++ b/readme.txt @@ -94,6 +94,7 @@ *****************************************************************************
*** Next ***
+- NEW: Change, chMtxGetNextMutexS() renamed to chMtxGetNextMutexX().
- NEW: RT C++ wrapper reworked, now it is mostly inline code, added some new
wrappers and methods. Added wrappers for more API functions. BaseThreads
are no more descendants of ThreadReference.
|