avoid deadlock (rt_hw_interrupt_disable and rt_enter_critical when enable smp)
This commit is contained in:
parent
551ddc170e
commit
0e61949116
|
@ -33,10 +33,6 @@
|
||||||
#include <rtthread.h>
|
#include <rtthread.h>
|
||||||
#include <rthw.h>
|
#include <rthw.h>
|
||||||
|
|
||||||
#ifdef RT_USING_SMP
|
|
||||||
rt_hw_spinlock_t _rt_critical_lock;
|
|
||||||
#endif /*RT_USING_SMP*/
|
|
||||||
|
|
||||||
rt_list_t rt_thread_priority_table[RT_THREAD_PRIORITY_MAX];
|
rt_list_t rt_thread_priority_table[RT_THREAD_PRIORITY_MAX];
|
||||||
rt_uint32_t rt_thread_ready_priority_group;
|
rt_uint32_t rt_thread_ready_priority_group;
|
||||||
#if RT_THREAD_PRIORITY_MAX > 32
|
#if RT_THREAD_PRIORITY_MAX > 32
|
||||||
|
@ -859,12 +855,15 @@ void rt_enter_critical(void)
|
||||||
* enough and does not check here
|
* enough and does not check here
|
||||||
*/
|
*/
|
||||||
|
|
||||||
/* lock scheduler for all cpus */
|
|
||||||
if (current_thread->critical_lock_nest == 0)
|
|
||||||
{
|
{
|
||||||
rt_hw_spin_lock(&_rt_critical_lock);
|
register rt_uint16_t lock_nest = current_thread->cpus_lock_nest;
|
||||||
|
current_thread->cpus_lock_nest++;
|
||||||
|
if (lock_nest == 0)
|
||||||
|
{
|
||||||
|
current_thread->scheduler_lock_nest ++;
|
||||||
|
rt_hw_spin_lock(&_cpus_lock);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/* critical for local cpu */
|
/* critical for local cpu */
|
||||||
current_thread->critical_lock_nest ++;
|
current_thread->critical_lock_nest ++;
|
||||||
|
|
||||||
|
@ -917,9 +916,11 @@ void rt_exit_critical(void)
|
||||||
|
|
||||||
current_thread->critical_lock_nest --;
|
current_thread->critical_lock_nest --;
|
||||||
|
|
||||||
if (current_thread->critical_lock_nest == 0)
|
current_thread->cpus_lock_nest--;
|
||||||
|
if (current_thread->cpus_lock_nest == 0)
|
||||||
{
|
{
|
||||||
rt_hw_spin_unlock(&_rt_critical_lock);
|
current_thread->scheduler_lock_nest --;
|
||||||
|
rt_hw_spin_unlock(&_cpus_lock);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (current_thread->scheduler_lock_nest <= 0)
|
if (current_thread->scheduler_lock_nest <= 0)
|
||||||
|
|
Loading…
Reference in New Issue