之前select死循环的问题查清楚了

发布于 2021-06-02 12:07:28 浏览：1299 订阅该版

问题可以见我之前发的贴子的描述，希望官方在下次发布时可以更改如下：
1.在poll_do里面增加pt->triggered = 0；这样可以避免在不是期待唤醒的时候可以挂起线程；
2.将pipe的机制可以改为read的时候不要唤醒写。原因如下：
应用程序使用pipe的框架如下：
![lw.png](https://oss-club.rt-thread.org/uploads/20210602/423694bbb50c433e1ae5a59150eb39e4.png)

线程1、2、3分别是3个独立的线程，优先级分别是25、26、27（25的优先级最大），其中有4到5对fd，此处只画了引起问题的这两对，通过分析内核pipe及poll处理代码，当pipe2里面为空时，假设此时线程26执行，在调用select（FD2[0]）时，发现pipe2为空，挂起线程26，此时25线程ready，执行25线程，假设此时pipe1不为空，select（FD[0]）成功返回，调用read（FD[0]）,read的时候会唤醒FD[1]所在的线程，此时切换到线程26运行，但是此时poll_table里面的属性triggered被置位，造成了一种假象：线程26认为是FD2[1]唤醒的自己，接着poll，发现pipe2还是空，导致一直死循环。

```c
static int poll_do(struct pollfd *fds, nfds_t nfds, struct rt_poll_table *pt, int msec)
{
    int num;
    int istimeout = 0;
    int n;
    struct pollfd *pf;
    int  ret = 0;

if (msec == 0)
    {
        pt->req._proc = RT_NULL;
        istimeout = 1;
    }

while (1)
    {
        pf = fds;
        num = 0;
        pt->triggered = 0；

for (n = 0; n < nfds; n ++)
        {
            ret = do_pollfd(pf, &pt->req);
            if(ret < 0)
            {
                /*dealwith the device return error -1  */
                pt->req._proc = RT_NULL;
                return ret;
            }
            else if(ret > 0)
            {
                num ++;
                pt->req._proc = RT_NULL;
            }
            pf ++;
        }

pt->req._proc = RT_NULL;

if (num || istimeout)
            break;

if (poll_wait_timeout(pt, msec))
            istimeout = 1;
    }

return num;
}
```