STM32 Systick Variable zurücksetzen

[Jan K. (jan_k)]

Hi Leute, ich habe im Prinzip ein simples Problem. Ich möchte eine 
einfache Idle Zeit Berechnung machen. Dafür setze ich im SysTick des 
STM32f103TB alle 10ms eine counter Variable zurück. Diese ist natürlich 
volatile.
Nun passiert aber manchmal, dass die Variable schlicht nicht zu 0 
gesetzt wird, dann steht etwa das doppelte in "last_idle_counts".

Die memory Barriere war lediglich ein Versuch, der nix brachte.

Woran könnte das liegen?

main.c:

#include "stm32f10x.h"

static volatile uint32_t idle_counts = 0;
static const uint32_t idle_counts_max = 1; // max counts per systick if no task and interrupt is running. Reference value


int main ( void )
{

  // Init Clock
  // Init_STM();
  SystemInit();
  ClockInit();
  SystemCoreClockUpdate();

  SysTick_Config(SystemCoreClock/100); // each 10 ms
  
  while( 1 )
  {
    ++idle_counts;
  }

}


void ClockInit(void)
{
  /* at this point HSI or HSE is the clock source, but no PLL */
  RCC->CR |= RCC_CR_HSION;
  while ((RCC->CR & RCC_CR_HSIRDY) != RCC_CR_HSIRDY);   /* Wait until HSI ready */

  // disable pll, otherwise we cannot set the multiplier 
  RCC->CR &= ~RCC_CR_PLLON;   
  while ((RCC->CR & RCC_CR_PLLRDY) != 0); // wait until off 
  
  // 8 MHz RC, 4 MHz * 16 = 64 MHz SYSCL = AHB = APB2, APB1 = HCLK/2 = 32 MHz, ADC = APB2/6 = 10.67 MHz; ADC 14 MHz max!
  RCC->CFGR = 0x00000000;
  RCC->CFGR |= RCC_CFGR_PLLSRC_HSI_Div2 | RCC_CFGR_PLLMULL16 | RCC_CFGR_PPRE1_DIV2 | RCC_CFGR_SW_PLL | RCC_CFGR_ADCPRE_DIV6;
  while((RCC->CFGR & RCC_CFGR_SWS) == RCC_CFGR_SWS_PLL); // wait
  
  // enable PLL again
  RCC->CR |= RCC_CR_PLLON;
  while ((RCC->CR & RCC_CR_PLLRDY) == RCC_CR_PLLRDY); // wait until rdy
  
  
  // prefetch enabled, 2 wait states
  FLASH->ACR |= FLASH_ACR_PRFTBE | FLASH_ACR_LATENCY_2;
 
  /* Check flash latency */
  while ((FLASH->ACR & FLASH_ACR_LATENCY) != FLASH_ACR_LATENCY_2);
}
 
void SysTick_Handler(void)  
{                              
  uint32_t last_idle_counts;

  static uint32_t idle_counts_mean;
  
  last_idle_counts = idle_counts;
  idle_counts = 0;
  __DMB();

  idle_counts_mean = (last_idle_counts + idle_counts_mean) / 2;
}

Vielen Dank!

[(prx) A. K. (prx)]

Jan K. schrieb:
>     ++idle_counts;

Das läuft auf 3 separate und zwischendrin unterbrechbare Befehle raus:
   Rx = idle_counts
   Rx = Rx + 1
   idle_counts = Rx
Jetzt klar?

[(prx) A. K. (prx)]

Abhilfe, ohne die Interrupts abzuschalten:

++idle_counts;

ersetzen durch

uint32_t t;
do {
  t = __LDREXW(&idle_count) + 1;
} while (__STREXW(t, &idle_count);

[Jan K. (jan_k)]

Meine Fresse.

Reicht für heute. Habe gerade noch überlegt, ob ich da ne semaphore bzw 
ldrex und strex oder so benutzen muss und habe für mich klar für nein 
entschieden, weil er ja in der ISR nur "gelesen" wird... Alter Verwalter

Danke ;)

Edit: da warst du schneller :(

[(prx) A. K. (prx)]

Jan K. schrieb:
> weil er ja in der ISR nur "gelesen" wird

Für was hältst du das?
 idle_counts = 0;

[Jan K. (jan_k)]

Daher die Anführungszeichen. Sollte meinen Gedankenfehler darstellen. 
Habe das resetten als Schreibzugriff schlicht ignoriert.