Electronic – STM32F103 Systick Interrupt stopped working when debugging in ram – (Coocox IDE)

You can, if your processor allows it (it should, since I believe all of that series are Cortex M4 based), set the Systick Interrupt to a higher priority than the other interrupts and then find out if you can make your compiler/processor not turn off interrupts globally while processing an interrupt.

That is, if you want to have delays in your interrupts.

Hint: You shouldn't want that.

Interrupts are meant to run efficiently and quickly to handle important events that are critical. If something needs a delay beyond a couple of ASM("__volatile__ NOP\n NOP\n NOP\n")'s, it's probably not something you want in your interrupt.

If you have a "heavy task" that gets triggered by an interrupt it's much neater to write:

uint32_t taskFlags;  // semafore register, can also be any other data type that fits your flags (1ul --> uint32_t or larger!)
#define HANDLE_MY_INTERRUPT_STUFF    (1ul<<0) // Define a flag for interrupt things.

void main(void) {
    while(1) {
        if(taskFlags & HANDLE_MY_INTERRUPT_STUFF) { // Check the flag
            // Do your interrupt stuff
            // Allowed to have many delays and other weird stuff
            
            taskFlags &= ~HANDLE_MY_INTERRUPT_STUFF; // Clear the flag.
        }
    }
}

InterruptHandler(void) {
    taskFlags |= HANDLE_MY_INTERRUPT_STUFF;  // set the flag
}

And now suddenly all that heavy processing gets done in the main loop, where it can be interrupted and all and won't harm any other time-critical things. But only triggered by the interrupt source, through the flags.

Although through the flags you will of course risk a slight variable delay in execution, it's always a balancing act.

Late addition:

Between processor types the delay will usually be between 4 and 40 clock cycles + average other interrupt code execution, so at a couple MHz using a 10ms delay will usually already marginalise that delay and its variation.)

This is the correct answer. Thanks to Everyone

/* Project: USART(Interrupt RX)
 * Author : Devjeet Mandal   
 * Date   : 15/2/2018       11:48
 * 
 */

/* Includes ------------------------------------------------------------------*/
#include "stm32f10x.h"
#include <string.h>

/* Private define ------------------------------------------------------------*/
#define SYSCLK   72000000
#define BAUD            115200

#define LED_PORT_EN()           ( RCC->APB2ENR |= RCC_APB2ENR_IOPDEN )
#define LED_PORT                    GPIOD

#define LED_MODE_BIT1           8
#define LED_MODE_BIT2           9
#define LED_CNF_BIT1            10
#define LED_CNF_BIT2            11

/* Private macro -------------------------------------------------------------*/
#define CNF_SET_PORTD(BIT1,BIT2)            (   LED_PORT->CRL &= ~((1<<BIT1) | (1<<BIT2)) )                 //General purpose output push-pull
#define MODE_SET_PORTD(BIT1,BIT2)           ( LED_PORT->CRL |=  (1<<BIT1) | (1<<BIT2) )                     //Output mode, max speed 50 MHz.

#define SET_GPIO_BIT_PORTD(BIT)             ( LED_PORT->BSRR =  (1 << BIT) )                                            //For setting the Bit   
#define RESET_GPIO_BIT_PORTD(BIT)           ( LED_PORT->BSRR =  ( (1 << BIT) << 16 )    )                       //For Resseting Bit

/* Private function prototypes -----------------------------------------------*/
 void Delay(int ms);
 void Led_Init(void);
 void Blink_Led(int ms);
 void Transmit_Char(char data);
 void Transmit_String(char *str);
 void USART_Init(void);
 void NVIC_Init(void);
 void USART1_IRQHandler(void);
 void Clear_Buffer(char *str);

 /* Private variables ---------------------------------------------------------*/
 __IO uint16_t datapos;
 char rx_data[255];


 int main(void)
 {
     Led_Init();
     USART_Init();
     NVIC_Init();
     datapos = 0;
     while(1)
     {
         if(strstr(rx_data,"led on\r"))
         {
                RESET_GPIO_BIT_PORTD(2);                        //Make Led High
                Transmit_String("\nLED is ON\r\n");
              datapos = 0;
              Clear_Buffer(rx_data);
         }
         else if(strstr(rx_data,"led off\r"))
         {
                SET_GPIO_BIT_PORTD(2);                      //Make Led Low
                Transmit_String("\nLED is OFF\r\n");
              datapos = 0;
              Clear_Buffer(rx_data);
         }
         Delay(1000);
     }
 }

 /** @breif: Clear Buffer
     * @param: Buffer to be cleared
     * @retVal: None
   */
 void Clear_Buffer(char *str)
 {
     while (*str != '\0')
     {
         *str = 0;
         str++;
     }
 }


 /** @breif: USART1 IRQ
     * @param: None
     * @retVal: None
   */
 void USART1_IRQHandler(void)
 {
     if(USART1->SR & USART_SR_RXNE)
     {
            rx_data[datapos] = USART1->DR;
            datapos++;
            if (datapos > 255 )
            {
                datapos = 0;
                Clear_Buffer(rx_data);
            }
   }  
 }

 /** @breif: Initialize setting for NVIC
     * @param: None
     * @retVal: None
   */
 void NVIC_Init()
 {
     NVIC_EnableIRQ(USART1_IRQn);
     NVIC_SetPriority(USART1_IRQn, 0);
 }

 /** @breif: Initialize setting for Usart
     * @param: None
     * @retVal: None
   */

 void USART_Init()
 {
     /*Enable Clock for USART port and USART1 */
     RCC->APB2ENR |=  RCC_APB2ENR_IOPAEN | RCC_APB2ENR_USART1EN;
     /*Rx,Tx setup GPIOA->CRH = 0x000004B0*/
     GPIOA->CRH |= GPIO_CRH_MODE9;          // Tx Mode- 11: Output mode, max speed 50 MHz.
     GPIOA->CRH |= GPIO_CRH_CNF9_1;         // Tx CNF - 10: Alternate function output Push-pull
     GPIOA->CRH &= ~(GPIO_CRH_CNF9_0);

     GPIOA->CRH &= ~(GPIO_CRH_MODE10);  // Rx Mode- 00: Input mode (reset state)
     GPIOA->CRH |= GPIO_CRH_CNF10_0;    // Rx CNF - 01: Floating input (reset state)*/
     /* Baud rate set 72MHz/115200 */
     USART1->BRR = SYSCLK/BAUD;
     /*Enable Rx, Enable Tx, Enable USART*/
     USART1->CR1 = USART_CR1_RE | USART_CR1_TE | USART_CR1_UE;
     USART1->CR1 |= USART_CR1_RXNEIE;  //enable rx interrupt
 }


  /** @breif: Transmit a String
     * @param: the string to be transmitted
     * @retVal: None
   */
 void Transmit_String(char *str)
 {
     while(*str != '\0')
     {
         Transmit_Char(*str);
         str++;
     }
 }

 /** @breif: Transmit a character
     * @param: the character to be transmitted
     * @retVal: None
   */

void Transmit_Char(char data)
{
    while (!(USART1->SR & USART_SR_TXE));
    USART1->DR = data;
}


/** @breif: For wait and doing nothing i.e for delay
     * @param: delaya time
     * @retVal: None
   */


    void Delay(int ms)
    {
        int i,j; 
        for (i = 0; i < ms; ++i) {
            for (j = 0; j < 1000; ++j);
            }
    }


 /** @breif: Initalize GPIO For Led
     * @param: None
     * @retVal: None
   */


    void Led_Init()
    {
        LED_PORT_EN();                                                              //Enable RCC for Led Port
        CNF_SET_PORTD(LED_CNF_BIT1,LED_CNF_BIT2);           //SET CNF       General purpose output push-pull
        MODE_SET_PORTD(LED_MODE_BIT1,LED_MODE_BIT2);    //SET MODE  Output mode, max speed 50 MHz. 
    }


 /** @breif: Blink Led Placed in PORT D Pin 2
     * @param: Delay for each state(ON/OFF)
     * @retVal: None
   */


    void Blink_Led(int ms)
    {
        RESET_GPIO_BIT_PORTD(2);                        //Make Led High
        Delay(ms);                                                  //wait
        SET_GPIO_BIT_PORTD(2);                          //Make Led Low
        Delay(ms);                                                  //wait
    }