Changeset 9b39bab in rtems

Timestamp:

02/20/98 20:23:57 (26 years ago)

Author:

Joel Sherrill <joel.sherrill@…>

Branches:

4.10, 4.11, 4.8, 4.9, 5, master

Children:

0799dec

Parents:

ee7f4e34

Message:

Modified to make it possible for an ISR to return a mutex which did
not use priority ceiling or priority inheritance protocols.

Files:

• c/src/exec/score/src/coremutex.c (modified) (1 diff)
• cpukit/score/src/coremutex.c (modified) (1 diff)

c/src/exec/score/src/coremutex.c

@@ -241 +241 @@
   executing = _Thread_Executing;
 
+  /*
+   *  The following code allows a thread (or ISR) other than the thread
+   *  which acquired the mutex to release that mutex.  This is only
+   *  allowed when the mutex in quetion is FIFO or simple Priority
+   *  discipline.  But Priority Ceiling or Priority Inheritance mutexes
+   *  must be released by the thread which acquired them.
+   */ 
+
   if ( !_Objects_Are_ids_equal(
-           _Thread_Executing->Object.id, the_mutex->holder_id ) )
-    return( CORE_MUTEX_STATUS_NOT_OWNER_OF_RESOURCE );
+           _Thread_Executing->Object.id, the_mutex->holder_id ) ) {
+
+    switch ( the_mutex->Attributes.discipline ) {
+      case CORE_MUTEX_DISCIPLINES_FIFO:
+      case CORE_MUTEX_DISCIPLINES_PRIORITY:
+        break;
+      case CORE_MUTEX_DISCIPLINES_PRIORITY_CEILING:
+      case CORE_MUTEX_DISCIPLINES_PRIORITY_INHERIT:
+        return( CORE_MUTEX_STATUS_NOT_OWNER_OF_RESOURCE );
+        break;
+    }
+  }
 
   the_mutex->nest_count--;

cpukit/score/src/coremutex.c

@@ -241 +241 @@
   executing = _Thread_Executing;
 
+  /*
+   *  The following code allows a thread (or ISR) other than the thread
+   *  which acquired the mutex to release that mutex.  This is only
+   *  allowed when the mutex in quetion is FIFO or simple Priority
+   *  discipline.  But Priority Ceiling or Priority Inheritance mutexes
+   *  must be released by the thread which acquired them.
+   */ 
+
   if ( !_Objects_Are_ids_equal(
-           _Thread_Executing->Object.id, the_mutex->holder_id ) )
-    return( CORE_MUTEX_STATUS_NOT_OWNER_OF_RESOURCE );
+           _Thread_Executing->Object.id, the_mutex->holder_id ) ) {
+
+    switch ( the_mutex->Attributes.discipline ) {
+      case CORE_MUTEX_DISCIPLINES_FIFO:
+      case CORE_MUTEX_DISCIPLINES_PRIORITY:
+        break;
+      case CORE_MUTEX_DISCIPLINES_PRIORITY_CEILING:
+      case CORE_MUTEX_DISCIPLINES_PRIORITY_INHERIT:
+        return( CORE_MUTEX_STATUS_NOT_OWNER_OF_RESOURCE );
+        break;
+    }
+  }
 
   the_mutex->nest_count--;