RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

rtems: Simplify _Signal_Action_handler()

Browse Source 
Use the invariant that an ASR is only processed if ASR processing is
enabled.
This commit is contained in:
Sebastian Huber
2021-03-02 10:55:10 +01:00
parent 320cb42b18
commit 9a13f3796a
1 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
cpukit/rtems/src/signalsend.c
View File

@@ -41,7 +41,6 @@ static void _Signal_Action_handler(
bool normal_is_preemptible; bool normal_is_preemptible;
uint32_t normal_cpu_time_budget; uint32_t normal_cpu_time_budget;
Thread_CPU_budget_algorithms normal_budget_algorithm; Thread_CPU_budget_algorithms normal_budget_algorithm;
bool normal_asr_is_enabled;
uint32_t normal_isr_level; uint32_t normal_isr_level;
uint32_t before_call_isr_level; uint32_t before_call_isr_level;
bool after_call_is_preemptible; bool after_call_is_preemptible;
@@ -68,8 +67,8 @@ static void _Signal_Action_handler(
/* Save normal mode */ /* Save normal mode */
_Assert( asr->is_enabled );
normal_is_preemptible = executing->is_preemptible; normal_is_preemptible = executing->is_preemptible;
normal_asr_is_enabled = asr->is_enabled;
normal_cpu_time_budget = executing->cpu_time_budget; normal_cpu_time_budget = executing->cpu_time_budget;
normal_budget_algorithm = executing->budget_algorithm; normal_budget_algorithm = executing->budget_algorithm;
@@ -133,13 +132,9 @@ static void _Signal_Action_handler(
*/ */
after_call_asr_is_enabled = asr->is_enabled; after_call_asr_is_enabled = asr->is_enabled;
asr->is_enabled = normal_asr_is_enabled; asr->is_enabled = true;
if ( if ( !after_call_asr_is_enabled && asr->signals_pending != 0 ) {
normal_asr_is_enabled &&
!after_call_asr_is_enabled &&
asr->signals_pending != 0
) {
_Thread_Append_post_switch_action( executing, action ); _Thread_Append_post_switch_action( executing, action );
} }
} }