Variable in Interrupt wird nicht gesetzt

OP #5034040
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.
1
uint8_t global_flag_1ms;
2

3
ISR(TIMER0_COMPA_vect)
4
{
5
  // Set global flag that 1ms passed
6
  global_flag_1ms = 1;
7
  // OUTPUT_LED ++;
8
};
9

10
int main(void)
11
{
12
  // initialize all needed parts
13
  init();
14
  // Activate interrupts global 
15
  sei();
16

17

18
    while (1) 
19
    {
20

21
    // Start the routine just if the timer0 interrupt occurred and 1ms past
22
    if(global_flag_1ms == 1)
23
    {
24
      global_flag_1ms = 0;
25
      OUTPUT_LED ++;
26
    }
27
    
28
    }
29
  // Deactivate interrupts global
30
  cli();
31
}

Danke für konstruktive Beiträge.

MfG
Gast #5034047
...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
OP #5034055
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?
Gast #5034731
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. ;-)
Gast #5034776
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.

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 9 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.