时间戳

首页 > 解决方案 > 写入安全的 UART 中断缓冲区

c - 写入安全的 UART 中断缓冲区

问题描述

我知道易失性键盘，它不能确保同步安全。

以下代码在中断例程中使用，我大量使用该函数

GetCharUART 在主循环中。

编写这样的代码是否安全且稳定？或者我必须像互斥锁一样进行低级同步，如果这是真的，我将如何保护那个 rbuf ？

 volatile char rbuf[5][UART_BUFSIZE];
vu16 rin[5] = { 0, 0, 0, 0, 0 };
vu16 rout[5] = { 0, 0, 0, 0, 0 };
    void USART_IRQHandler(char Channel, USART_TypeDef *USARTx)
    {
        volatile unsigned int IIR;
     
        IIR = USARTx->SR;
        if (IIR & USART_FLAG_RXNE)
        {                  // read interrupt
          USARTx->SR &= ~USART_FLAG_RXNE;             // clear interrupt
     
          rbuf[Channel][rin[Channel]] = USART_ReceiveData(USARTx);
          rin[Channel]++;
          if(rin[Channel]>=UART_BUFSIZE) rin[Channel]=0;
        }
     
        if (IIR & USART_FLAG_TXE)
        {
          USARTx->SR &= ~USART_FLAG_TXE;              // clear interrupt
        }
    }   
     
     
    int GetCharUART (char Channel)
    {
      int result;
     
      if (rin[Channel]==rout[Channel]) {
            return EMPTY;
        }
     
      result=rbuf[Channel][rout[Channel]];
      rout[Channel]++;
      if(rout[Channel]>=UART_BUFSIZE) 
            rout[Channel]=0;
     
      return result;
    }

修改后的代码：

void USART_IRQHandler(char Channel, USART_TypeDef *USARTx)
{
    volatile unsigned int IIR;

    IIR = USARTx->SR;
    if (IIR & USART_FLAG_RXNE)
    {                  // read interrupt
      USARTx->SR &= ~USART_FLAG_RXNE;             // clear interrupt

      rbuf[Channel][rin[Channel]% UART_BUFSIZE] = USART_ReceiveData(USARTx);
      rin[Channel]++;
    }

    if (IIR & USART_FLAG_TXE)
    {
      USARTx->SR &= ~USART_FLAG_TXE;              // clear interrupt
    }
}
/******************************************************************************/
int GetCharUART (char Channel)
{
  int result;

  if (rin[Channel]==rout[Channel]) {
        return EMPTY;
    }

  result=rbuf[Channel][rout[Channel]% UART_BUFSIZE];
  rout[Channel]++;

  return result;
}

标签： cfirmware

解决方案

您的代码存在功能错误。

在 ISR 中，您不会检查“缓冲区已满”情况。代码只是增加rin[Channel]而不看rout[Channel]. 因此，整个缓冲区的数据可能会丢失。

示例：如果rout[Channel]等于 0 且rin[Channel]等于 UART_BUFSIZE-1，则 ISR 将设置rin[Channel]为零。换句话说，缓冲区现在将显示为空并且数据丢失。

So the first step is to fix the code.

推荐阅读