RTTV1.1.0源码中有关优先级翻转的重要问题

发布于 2013-02-20 00:10:48 浏览：3207 订阅该版

在RTTV1.1.0版本的源码中,我在看到有关互斥量的实现源码发现一个疑问,在获取互斥量的代码中:
ipc.c源文件的第695行代码:
```//...
 /* change the owner thread priority of mutex */
                if (thread->current_priority < mutex->owner->current_priority)//如果当前线程的优先级比互斥锁的拥有者线程的优先级还小的话,则修改互斥锁的拥有者线程优先级为当前线程的优先级,其实这时相当于降低其优先级
                {
                    /* change the owner thread priority */
                    rt_thread_control(mutex->owner,
                                      RT_THREAD_CTRL_CHANGE_PRIORITY,
                                      &thread->current_priority);
                }
//...```
这里if (thread->current_priority < mutex->owner->current_priority)是否应该为if (thread->current_priority > mutex->owner->current_priority)?即将小于号改为大于号? 因为当前线程的优先级比互斥锁优先级拥有者线程的优先级小时,则不需要修改拥有者优先级,而只有在当前线程的优先大于拥有者线程的优先级时,才需要将拥有者线程的优先级提升为当前线程的优先级,而不是降为当前线程的优先级,不知道我的理解是否正确?

另一个问题:还是在ipc.c源文件中的729行,即还是rt_mutex_take函数中的末尾部分代码:
```                if (thread->error != RT_EOK)
                {
                    /* return error */
                    return thread->error;
                }
                else
                {
                    /* the mutex is taken successfully. */
                    /* disable interrupt */
                    temp = rt_hw_interrupt_disable();
                }```
不知道这几行代码放在此处是什么含义?谁能帮忙解释下?

6 个回答

fengzi 2013-02-20

这家伙很懒，什么也没写！

Arthur Bu 2013-02-20

这家伙很懒，什么也没写！

prife 2013-02-20

这家伙很懒，什么也没写！

Arthur Bu 2013-02-20

这家伙很懒，什么也没写！

其实我想问的是

```
if (thread->error != RT_EOK)
                {
                    /* return error */
                    return thread->error;
                }
```

这几行代码的含义,我的理解是,由于当前线程在rt_mutex_take的开始处已经执行:

```
/* reset thread error */
    thread->error = RT_EOK;
```

即凡是执行rt_mutex_take函数的线程的thread->error开始时都会被重置为RT_EOK,假设传入的参数time>0,且当前存在另一线程已经占有了此互斥锁,也就是说此线程将会运行到723行的:

```
 /* enable interrupt */
                rt_hw_interrupt_enable(temp);

/* do schedule */
                rt_schedule();

if (thread->error != RT_EOK)
                {
                    /* return error */
                    return thread->error;
                }
//...
```

此时当前线程挂起后将执行重新调度,假设经过n时间后,此线程再次获得处理机,假设另一线程还是没有释放此互斥锁,当前线程则将执行

```
if (thread->error != RT_EOK)
                {
                    /* return error */
                    return thread->error;
                }

```

就是这里我不是太理解,我搜索了下,这个当前的thread->error并没有被修改呀?那么此处就是怎么依据thread->error来判断当前线程是否已获取互斥锁的拥有权?不知道我这样理解得对不对?