rt_list_t结构处理的问题

发布于 2011-04-12 15:41:20 浏览：3661 订阅该版

在 src/kservice.h文件里有几个函数:
    rt_inline void rt_list_init(rt_list_t *l);
    rt_inline void rt_list_insert_after(rt_list_t *l, rt_list_t *n);
    rt_inline void rt_list_insert_before(rt_list_t *l, rt_list_t *n);
    rt_inline void rt_list_remove(rt_list_t *n);
    rt_inline int rt_list_isempty(const rt_list_t *l);

我的几个问题是:
1.函数没有任何参数检测.如:
rt_inline void rt_list_insert_after(rt_list_t *l, rt_list_t *n)
{
	l->next->prev = n;
	n->next = l->next;

l->next = n;
	n->prev = l;
}
  l和l->next都有可能为NULL,那 "l->next->prev = n;"就有问题了,而且没有文档进行使用的注意事项的说明.

2.没有考虑到多任务的同步和互斥.如:两个任务同时在同一链表上增加节点
        task1运行:      rt_list_insert_after(time, node1);
        task2运行:      rt_list_insert_after(time, node2);
这是,在语句"l->next->prev = n;"就有可能造成处理错误;当task1刚运行完"l->next->prev = n"语句,task2就抢占了cpu,并执行了"rt_list_insert_after(time, node2);"函数,这时,链表就会出错了.

----------------------------------------------------------------------------------
以上是我的一点观点,不知道有问题没有,或者上层调用的代码已经做了某些处理,使得上面这些函数的运行环境非常好,不会出现这些问题(这样的话,这些函数是不能对外公开的,因为不能保证使用环境了).  
我的Email:zhu1982lin@yahoo.com.cn
QQ:53827302.
欢迎QQ讨论.