Variable in Interrupt wird nicht gesetzt

[TM F. (p_richner)]

Hallo Zusammen

Ich habe folgenden Code. Jedoch wird der OUTPUT_LED nie inkrementiert. 
Setze ich diese Zeile jedoch in den Interrupt(Ist jetzt auskommentiert), 
so wird der Output inkrementiert. Ich sehe das Problem nicht und stehe 
voll auf dem Schlauch.

uint8_t global_flag_1ms;

ISR(TIMER0_COMPA_vect)
{
  // Set global flag that 1ms passed
  global_flag_1ms = 1;
  // OUTPUT_LED ++;
};

int main(void)
{
  // initialize all needed parts
  init();
  // Activate interrupts global 
  sei();


    while (1) 
    {

    // Start the routine just if the timer0 interrupt occurred and 1ms past
    if(global_flag_1ms == 1)
    {
      global_flag_1ms = 0;
      OUTPUT_LED ++;
    }
    
    }
  // Deactivate interrupts global
  cli();
}

Danke für konstruktive Beiträge.

MfG

[Karl M. (Gast)]

Hallo,

dein Stichwort ist die Variable einem neuen Zweck "volatile" zu 
zuordnen.

[Karl M. (Gast)]

Hallo,

und wo ist OUTPUT_LED definiert und was ist das ?

[sigma9 (Gast)]

...ich vermute das liegt am fehlenden "volatile" für deine Variable (es 
sei denn machst das und zeigst es uns nicht).

Falls du das nicht machst, wird sie in der ISR einfach weg optimiert.
Lies die mal die entsprechenden Infos durch (C-lib, ...)

/g

[TM F. (p_richner)]

Karl M. schrieb:
> dein Stichwort ist die Variable einem neuen Zweck "volatile" zu
> zuordnen.

Ok Danke das funktionierte.

So ist dies mehr ein Compilerproblem, dass diese Zeile wegoptimiert 
wird?

Ich benutze andere globale Variablen an mehreren Orten und dies 
funktioniert problemlos.

Heisst das, dass globale Variablen immer volatile als Zusatz haben 
sollten?

[Tom (Gast)]

https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Datenaustausch_mit_Interrupt-Routinen

[Rufus Τ. F. (rufus)]

TM F. schrieb:
> Heisst das, dass globale Variablen immer volatile als Zusatz haben
> sollten?

Nein. Das ist nur nötig, wenn aus zwei quasiparallel laufenden 
Programmteilen auf sie zugegriffen wird. Das gibt es auf 
Microcontrollern bei der Kombination Hauptprogramm/ISR.

[Markus F. (mfro)]

TM F. schrieb:
> So ist dies mehr ein Compilerproblem, dass diese Zeile wegoptimiert
> wird?

Nein, das ist definitiv ein Programmiererproblem ...

[Luxusausstattung (Gast)]

Markus F. schrieb:
> Nein, das ist definitiv ein Programmiererproblem

Copiler/Linker hätten schon die Möglichkeit, die Symbole einer ISR gegen 
die Verwendung in einem anderen Kontext zu prüfen. Es gbit soviele 
Schalter, da wäre dies eine sinnvolles Addon.

Neben Variablen können auch Funktionen problematisch sein. ;-)

[Nop (Gast)]

Luxusausstattung schrieb:

> Neben Variablen können auch Funktionen problematisch sein. ;-)

Weswegen ich bei ISRs, Exceptions usw. am liebsten irgendeine Variante 
von "diese Funktion wird benutzt"-Attribut hinsetze. Einfach weil ich 
dann sicher bin, daß nach einem Compiler-Update da nichts Blödes 
passiert.

[gcc (Gast)]

TM F. schrieb:
> So ist dies mehr ein Compilerproblem, dass diese Zeile wegoptimiert
> wird?

Wenn du bei dir die Optimierung im Compiler angeschaltet hast, musst du 
auch damit rechnen, dass sie aktiv ist.