Re: [PATCH 2/2] rtems: Fix task restart within interrupt context

2021-05-14 Thread Gedare Bloom
These 2 patches are fine. Although it is an internal interface, you might make the 'ignore' parameter 'ignore_mask' maybe. My initial thought when I see 'ignore' is that variable is ignored/unused, not that it contains the states to ignore. Since the function is internal and has no documentation,

[PATCH 2/2] rtems: Fix task restart within interrupt context

2021-05-14 Thread Sebastian Huber
rtems_task_restart() may be called from within interrupt context. So checking only that the thread to restart is equal to the executing thread is insufficient to determine a self restart. We have to also check that no ISR is in progress. Merge _Thread_Restart_other() and _Thread_Restart_self()