mirror of
https://gitlab.rtems.org/rtems/rtos/rtems.git
synced 2026-09-22 13:15:54 +08:00
score: Simplify CORE mutex seize
Rename _CORE_mutex_Seize_no_protocol_slow() in _CORE_mutex_Seize_slow(). Remove previous _CORE_mutex_Seize_slow() since the protocol handling is now done in the thread queue operations.
This commit is contained in:
@@ -93,14 +93,6 @@ RTEMS_INLINE_ROUTINE bool _CORE_mutex_Is_locked(
|
||||
}
|
||||
|
||||
Status_Control _CORE_mutex_Seize_slow(
|
||||
CORE_mutex_Control *the_mutex,
|
||||
Thread_Control *executing,
|
||||
Thread_Control *owner,
|
||||
bool wait,
|
||||
Thread_queue_Context *queue_context
|
||||
);
|
||||
|
||||
Status_Control _CORE_mutex_Seize_no_protocol_slow(
|
||||
CORE_mutex_Control *the_mutex,
|
||||
const Thread_queue_Operations *operations,
|
||||
Thread_Control *executing,
|
||||
@@ -171,8 +163,8 @@ RTEMS_INLINE_ROUTINE Status_Control _CORE_recursive_mutex_Seize(
|
||||
|
||||
return _CORE_mutex_Seize_slow(
|
||||
&the_mutex->Mutex,
|
||||
CORE_MUTEX_TQ_PRIORITY_INHERIT_OPERATIONS,
|
||||
executing,
|
||||
owner,
|
||||
wait,
|
||||
queue_context
|
||||
);
|
||||
@@ -251,7 +243,7 @@ RTEMS_INLINE_ROUTINE Status_Control _CORE_recursive_mutex_Seize_no_protocol(
|
||||
return status;
|
||||
}
|
||||
|
||||
return _CORE_mutex_Seize_no_protocol_slow(
|
||||
return _CORE_mutex_Seize_slow(
|
||||
&the_mutex->Mutex,
|
||||
operations,
|
||||
executing,
|
||||
@@ -444,7 +436,7 @@ RTEMS_INLINE_ROUTINE Status_Control _CORE_ceiling_mutex_Seize(
|
||||
return status;
|
||||
}
|
||||
|
||||
return _CORE_mutex_Seize_no_protocol_slow(
|
||||
return _CORE_mutex_Seize_slow(
|
||||
&the_mutex->Recursive.Mutex,
|
||||
CORE_MUTEX_TQ_OPERATIONS,
|
||||
executing,
|
||||
|
||||
@@ -24,65 +24,6 @@
|
||||
#include <rtems/score/watchdog.h>
|
||||
|
||||
Status_Control _CORE_mutex_Seize_slow(
|
||||
CORE_mutex_Control *the_mutex,
|
||||
Thread_Control *executing,
|
||||
Thread_Control *owner,
|
||||
bool wait,
|
||||
Thread_queue_Context *queue_context
|
||||
)
|
||||
{
|
||||
if ( !wait ) {
|
||||
_CORE_mutex_Release( the_mutex, queue_context );
|
||||
return STATUS_UNAVAILABLE;
|
||||
}
|
||||
|
||||
#if !defined(RTEMS_SMP)
|
||||
/*
|
||||
* We must disable thread dispatching here since we enable the interrupts for
|
||||
* priority inheritance mutexes.
|
||||
*/
|
||||
_Thread_Dispatch_disable();
|
||||
|
||||
/*
|
||||
* To enable interrupts here works only since exactly one executing thread
|
||||
* exists and only threads are allowed to seize and surrender mutexes with
|
||||
* the priority inheritance protocol. On SMP configurations more than one
|
||||
* executing thread may exist, so here we must not release the lock, since
|
||||
* otherwise the current owner may be no longer the owner of the mutex
|
||||
* once we released the lock.
|
||||
*/
|
||||
_CORE_mutex_Release( the_mutex, queue_context );
|
||||
#endif
|
||||
|
||||
#if defined(RTEMS_SMP)
|
||||
_Thread_queue_Context_set_expected_level( queue_context, 1 );
|
||||
#else
|
||||
_ISR_lock_ISR_disable( &queue_context->Lock_context.Lock_context );
|
||||
_CORE_mutex_Acquire_critical( the_mutex, queue_context );
|
||||
_Thread_queue_Context_set_expected_level( queue_context, 2 );
|
||||
#endif
|
||||
|
||||
_Thread_queue_Context_set_deadlock_callout(
|
||||
queue_context,
|
||||
_Thread_queue_Deadlock_status
|
||||
);
|
||||
|
||||
_Thread_queue_Enqueue_critical(
|
||||
&the_mutex->Wait_queue.Queue,
|
||||
CORE_MUTEX_TQ_PRIORITY_INHERIT_OPERATIONS,
|
||||
executing,
|
||||
STATES_WAITING_FOR_MUTEX,
|
||||
queue_context
|
||||
);
|
||||
|
||||
#if !defined(RTEMS_SMP)
|
||||
_Thread_Dispatch_enable( _Per_CPU_Get() );
|
||||
#endif
|
||||
|
||||
return _Thread_Wait_get_status( executing );
|
||||
}
|
||||
|
||||
Status_Control _CORE_mutex_Seize_no_protocol_slow(
|
||||
CORE_mutex_Control *the_mutex,
|
||||
const Thread_queue_Operations *operations,
|
||||
Thread_Control *executing,
|
||||
|
||||
Reference in New Issue
Block a user