一个简单的GUI程序造成X占CPU达80％以上

xwzss

这是我程序的部分代码：

1. 
   
2.               while (1) {
   
3.                                 remaining = sizeof(buffer) - bufpos;
   
4.                                 if ( (len = read(fd, buffer + bufpos, remaining)) > 0) {
   
5.                                                 bufpos += len;
   
6.                                                 pos = 0;                               
   
7.                                                 while ( (bufpos - pos) >= LENGTH) {
   
8.                                                                 cnt = data_parse(buffer + pos, display, win, gc);
   
9.                                                        
   
10.                                                                 pos += cnt;
    
11.                                                 }
    
12.                   
    
13.                                                 if (pos < bufpos) {
    
14.                                                                 memmove(buffer, buffer + pos, bufpos - pos);
    
15.                                                                
    
16.                                                                 bufpos -= pos;
    
17.                                                 } else {
    
18.                                                                 bufpos = 0;
    
19.                                                 }
    
20.                                 }
    
21.                 }
    

复制代码

data_parse部分：

1. 
   
2. int
   
3. data_parse(unsigned char *data, Display *display, Window win, GC gc)
   
4. {
   
5.                 int loop, bad;
   
6.                 int x, y, p;
   
7.                 int new_x, new_y, new_p;
   
8. 
   
9.                 bad = 0;
   
10.   
    
11.                 for (loop = 0; loop < LENGTH; loop++) {
    
12.                                 if ( ((loop == 0) && !(data[loop] & 0x80)) || ((loop != 0) && (data[loop] & 0x80))) {
    
13.                                                 bad = 1;
    
14. 
    
15.                                                 if ( (loop != 0) && (data[loop] & 0x80))
    
16.                                                                 return loop;
    
17.                                 }
    
18.                 }
    
19. 
    
20.                 if (bad)
    
21.                                 return LENGTH;
    
22. 
    
23.                 if ( !(data[0] & 1)) {
    
24.                                 old_x = -1;
    
25.                                 old_y = -1;
    
26.                                 return LENGTH;
    
27.                 }
    
28. 
    
29.                 x = (data[1] << 7) + data[2];
    
30.                 y = (data[3] << 7) + data[4];
    
31.                 p = (data[5] << 7) + data[6];
    
32. 
    
33.                 if (x > 6144 || y > 8192 || p > 1024)
    
34.                                 return LENGTH;
    
35. 
    
36.                 new_x = x_table[x];
    
37.                 new_y = y_table[y];
    
38.                 new_p = p_table[p];
    
39. 
    
40.                 if (old_x < 0 || old_y < 0) {
    
41.                                 old_x = new_x;
    
42.                                 old_y = new_y;
    
43.                 }
    
44. 
    
45.                 XSetLineAttributes(display, gc, ebook_p, LineSolid, CapRound, JoinRound);
    
46.                 XDrawLine(display, win, gc, old_x, old_y, new_x, new_y);
    
47.                 XFlush(display);
    
48.                 old_x = new_x;
    
49.                 old_y = new_y;
    
50. 
    
51.                 return LENGTH;
    
52. }
    

复制代码

这是个手写板的部分代码，这个程序从串口读数据，然后分析，得出X，Y坐标，再在屏幕上划出来，不过这个程序使X占CPU达到80％以上，有什么办法降低这个占用率吗？

cyberHunK

由于刚接触linux环境下程序设计，系统的优化及调用机制还不明确，但根据已往的经验，在你的程序中应该出现了“盲等”这种情况！检查一下你代码中出现循环的部分，做断点跟踪观察！对一些事件检查类型的循环，要做成触发的机制，把不必要的循环判断都剔出掉！

yjfkpyu

在while(1)的循环中加入usleep(n)；  不然你的cpu一直在跑你的死循环当然会达近100%了，可以根据你需要的实时程度来设置n的值，n是微妙

pernlee

sleep(1) 就可以了

cqtzj

关注,

yjfkpyu

sleep(1)会太长了，这段程序还是需要一定的实时性的

ttt678

用select函数试试

zhongfangqing

while (1) {
                                remaining = sizeof(buffer) - bufpos;
                                if ( (len = read(fd, buffer + bufpos, remaining)) > 0) {
                                                bufpos += len;
                                                pos = 0;                                
                                                while ( (bufpos - pos) >= LENGTH) {
                                                                cnt = data_parse(buffer + pos, display, win, gc);
                                                        
                                                                pos += cnt;
                                                }
                  
                                                if (pos < bufpos) {
                                                                memmove(buffer, buffer + pos, bufpos - pos);
                                                               
                                                                bufpos -= pos;
                                                } else {
                                                                bufpos = 0;
                                                }
                                }
                                usleep(5000);//这里加一句试试
                }

linux_wuliqiang

问题应该在 if ( (len = read(fd, buffer + bufpos, remaining)) > 0) ，
用gdb 调试一下，看能发现什么不。。。