xQueueSendBackFromISR fails after many hours

belmondo
2013-02-01
2013-09-16
  • belmondo

    belmondo - 2013-02-01

    I am having an issue with xQueueSendBackFromISR. I use it inside a UART interrupt routine to send a message to another task to state that a line is ready to be read. It works for hours and suddenly for any reason, the xQueueSendBackFromISR fails. It does not return pdPASS. The queue is not full. I even reset it when the problem occur but it does not seem to help. The xQueueSendBackFromISR never seem to respond back and always fails thereafter.

    Does anyone know what the issue could be ?

     
  • belmondo

    belmondo - 2013-02-01

    Here is the Interrupt Code

    void USART1_IRQHandler(void)
    {
        uint8_t ch;
        portBASE_TYPE xHigherPriorityTaskWoken = pdFALSE;
        int num;
        char line[80];
        unsigned int cmd;
        static int count = 0;
        ErrorStatus err;
        // Receive interrupt
        if (USART_GetITStatus(USART1, USART_IT_RXNE) != RESET)
        {
              ch=(uint8_t)USART_ReceiveData(USART1);
              err = BufferPut(&U1Rx, ch);
             if ( err == ERROR ) {
                cmd = 5;
                xQueueSendToBackFromISR(uartQueue,&cmd,&xHigherPriorityTaskWoken);
             }
             else if (ch == '\n') {
                trace_set(7);
                count = 0;
                cmd = 1;
                num = uxQueueMessagesWaitingFromISR(uartQueue);
                if ( xQueueSendToBackFromISR(uartQueue,&cmd,&xHigherPriorityTaskWoken) != pdPASS ) {
                    sprintf(line,"Uart Queue Send Error: %d in the queue.................",num);
                    xprintln(line);
                }
                trace_reset(7);
             }
             else if ((count == 0) && (ch == '>') ){
                cmd = 2;
                xQueueSendToBackFromISR(uartQueue,&cmd,&xHigherPriorityTaskWoken);
             }
             else {
                if (count >= 2000) {
                    cmd = 6;
                    xQueueSendToBackFromISR(uartQueue,&cmd,&xHigherPriorityTaskWoken);
                    count = 0;
                } else {
                    count++;
                }
             }
        }
        // Other Error Flags
        if (USART_GetFlagStatus(USART1, USART_FLAG_ERRORS) != RESET) {
            USART_ReceiveData(USART1); // Clear Error
            xprintln("Uart Error.................");
        }
        portEND_SWITCHING_ISR(xHigherPriorityTaskWoken);
    }
    
     
  • MEdwards

    MEdwards - 2013-02-01

    When it stops working can you put a break point in the code and step into the function to see what it is doing?
    Is your interrupt stack large enough for the line array and to call sprintf() which often uses a lot of stack space?
    Can xprintln be called from an interrupt?

     
  • Richard Damon

    Richard Damon - 2013-02-02

    A couple of issues that might cause this sort of error.

    1) Stack corruption, this ISR uses a LOT of stack for an ISR, since it allocates an 80 byte array. Many ports use the current tasks stack for the ISR, so in this case you need this much extra room in every task's stack. If one doesn't have enough, then when you get "lucky" enough to interrupt while that task is active, problems arise.

    2) Make very sure that you have interrupt priorities set properly. Most ports only allow ISRs of certain priorities to call FreeRTOS APIs, and an interrupt at a higher level than this can cause corruption that could cause this.

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks