串口问题求助，请兄弟们支援

klanet

我的主机芯片是at91rm9200，现在发现了一个问题。
我的主机通过串口采集另外一个模块的数据，发现数据采集出错。时常有数据丢失。
后来修改了协议的代码。
协议开始读数据是通过一个buffer来读取的，就是read一个buffer，buffer大小为1024。有数据就返回。这样是会丢数据的。
后来把buffer改为1，也就是变为一个一个数据的读取，数据的正常了。

请问这是为啥？我总不能规定以后所有的采集协议读取都是一个一个字节读取吧。

dreamice

回复 1# klanet


    这个估计跟你的中断处理程序有关，你一个一个字节的读取，可能是串行阻塞型的，而开了软件buffer，有可能就是中断处理程序直接非阻塞的往里面塞数据，中断处理的优先级很高，一旦buffer满了就丢弃数据，具体还要仔细结合代码分析，方便的话可以贴一下代码。

yikaikai

读写有没有加锁？

klanet

回复 2# dreamice


    版主要的是那部分的代码？
驱动的还是协议层的？
协议层的

1.     // 找头字符：
   
2.     int i=0;
   
3.     sRecStr[0] = 0;
   
4.     do
   
5.     {
   
6.         ReadFile( hComm, (char*)sRecStr, 1, &lRead, NULL );
   
7.         i++;
   
8.     }while( lRead==1 && sRecStr[0]!=bySOI && i<nMaxDllBufNum-1 );
   
9. 
   
10.     // 读后续数据串
    
11.     if( lRead && sRecStr[0]==bySOI )
    
12.     {
    
13.         // 读取到数据段长度字符,以便决定本包总数据量.
    
14.         ReadFile( hComm, (char*)sRecStr+1, 12, &lRead, NULL );
    
15.         if( lRead==12 )
    
16.         {
    
17.             int nLen=0;
    
18.             sscanf( (char *)sRecStr+9,"%04x", &nLen );
    
19.             nLen = nLen & 0x0FFF;           // 去掉校验位
    
20.             if( nLen+18 < nMaxDllBufNum-1 ) // 缓冲区溢出判断
    
21.             {
    
22.                 ReadFile( hComm, (char*)sRecStr+13, nLen+5, &lRead, NULL );
    
23.             }
    
24.             lRead += 12;
    
25.         }
    
26.         lRead++;
    
27.     }
    
28. 
    
29.     sRecStr[lRead] = 0;
    
30. 
    
31.     // 跟踪调试的接收信息处理：回应信息记录。

复制代码

上面的是buffer读取，改为一个一个字节读取就不贴出来了，都差不多。

驱动代码在 drivers/serial/atmel_serial.c
下面是中断发的代码

1. static void atmel_tx_chars(struct uart_port *port)
   
2. {
   
3.         struct circ_buf *xmit = &port->info->xmit;
   
4. 
   
5.         if (port->x_char) {
   
6.                 UART_PUT_CHAR(port, port->x_char);
   
7.                 port->icount.tx++;
   
8.                 port->x_char = 0;
   
9.                 return;
   
10.         }
    
11.         if (uart_circ_empty(xmit) || uart_tx_stopped(port)) {
    
12.                 atmel_stop_tx(port);
    
13.                 return;
    
14.         }
    
15. 
    
16.         while (UART_GET_CSR(port) & ATMEL_US_TXRDY) {
    
17.                 UART_PUT_CHAR(port, xmit->buf[xmit->tail]);
    
18.                 xmit->tail = (xmit->tail + 1) & (UART_XMIT_SIZE - 1);
    
19.                 port->icount.tx++;
    
20.                 if (uart_circ_empty(xmit))
    
21.                         break;
    
22.         }
    
23. 
    
24.         if (uart_circ_chars_pending(xmit) < WAKEUP_CHARS)
    
25.                 uart_write_wakeup(port);
    
26. 
    
27.         if (uart_circ_empty(xmit))
    
28.                 atmel_stop_tx(port);
    
29. }
    

复制代码

下面是中断收

1. static void atmel_rx_chars(struct uart_port *port)
   
2. {
   
3.         struct atmel_uart_port *atmel_port = (struct atmel_uart_port *) port;
   
4.         struct tty_struct *tty = port->info->tty;
   
5.         unsigned int status, ch, flg;
   
6. 
   
7.         status = UART_GET_CSR(port);
   
8.         while (status & ATMEL_US_RXRDY) {
   
9.                 ch = UART_GET_CHAR(port);
   
10. 
    
11.                 port->icount.rx++;
    
12. 
    
13.                 flg = TTY_NORMAL;
    
14. 
    
15.                 /*
    
16.                  * note that the error handling code is
    
17.                  * out of the main execution path
    
18.                  */
    
19.                 if (unlikely(status & (ATMEL_US_PARE | ATMEL_US_FRAME
    
20.                                        | ATMEL_US_OVRE | ATMEL_US_RXBRK)
    
21.                              || atmel_port->break_active)) {
    
22.                         UART_PUT_CR(port, ATMEL_US_RSTSTA);        /* clear error */
    
23.                         if (status & ATMEL_US_RXBRK
    
24.                             && !atmel_port->break_active) {
    
25.                                 status &= ~(ATMEL_US_PARE | ATMEL_US_FRAME);        /* ignore side-effect */
    
26.                                 port->icount.brk++;
    
27.                                 atmel_port->break_active = 1;
    
28.                                 UART_PUT_IER(port, ATMEL_US_RXBRK);
    
29.                                 if (uart_handle_break(port))
    
30.                                         goto ignore_char;
    
31.                         } else {
    
32.                                 /*
    
33.                                  * This is either the end-of-break
    
34.                                  * condition or we've received at
    
35.                                  * least one character without RXBRK
    
36.                                  * being set. In both cases, the next
    
37.                                  * RXBRK will indicate start-of-break.
    
38.                                  */
    
39.                                 UART_PUT_IDR(port, ATMEL_US_RXBRK);
    
40.                                 status &= ~ATMEL_US_RXBRK;
    
41.                                 atmel_port->break_active = 0;
    
42.                         }
    
43.                         if (status & ATMEL_US_PARE)
    
44.                                 port->icount.parity++;
    
45.                         if (status & ATMEL_US_FRAME)
    
46.                                 port->icount.frame++;
    
47.                         if (status & ATMEL_US_OVRE)
    
48.                                 port->icount.overrun++;
    
49. 
    
50.                         status &= port->read_status_mask;
    
51. 
    
52.                         if (status & ATMEL_US_RXBRK)
    
53.                                 flg = TTY_BREAK;
    
54.                         else if (status & ATMEL_US_PARE)
    
55.                                 flg = TTY_PARITY;
    
56.                         else if (status & ATMEL_US_FRAME)
    
57.                                 flg = TTY_FRAME;
    
58.                 }
    
59. 
    
60.                 if (uart_handle_sysrq_char(port, ch))
    
61.                         goto ignore_char;
    
62. 
    
63.                 uart_insert_char(port, status, ATMEL_US_OVRE, ch, flg);
    
64. 
    
65.         ignore_char:
    
66.                 status = UART_GET_CSR(port);
    
67.         }
    
68. 
    
69.         tty_flip_buffer_push(tty);
    
70. }
    

复制代码

klanet

回复 3# yikaikai


    没有

lelee007

buffer的管理肯定有问题，把代码贴上来呗

ricks_wu

为一个陌生人无偿看这么长长的代码
这是一种怎样的神经病精神