volatile не помагает Опции

[vROMAv]

Есть некоторый код, проблема в том что имеется переменная которая после выхода из прерывания восстанавливает своё значение,
т.е. независимо от volatile компилятор по ходу разместил
переменную event_flag в регистре??!!!
При чём данная ситуация возникает не часто, иногда установленное значение
в прерывании остаётся и после выхода из него((


/////////////////////////////////
// AVR Studio 4.16 Build 628 //
// WinAVR-20090313 //
// Atmega8535 //
/////////////////////////////////

............................
volatile uint8_t event_flags;
#define START 0x01
............................
#define PACK_NOW 0x20

volatile uint8_t macro_rx = 0;

ISR (USART_RX_vect)
{
.............
.............
event_flags |= PACK_NOW;
macro_rx = 1;

//тут PACK_NOW устанавливается, но после выхода из прерывания,
//в event_flags восстанавливается предыдущее значение

}

void main_loop()
{
............
............
if(macro_rx)
{
if(!(event_flags & PACK_NOW))
{
//!!!!!! ОШИБКА !!!!!!
//т.е. macro_rx так и осталась в установленном в прерывании значении
//а event_flags нет!!!
}
}
............
............
}

int main()
{
while (true)
{
main_loop();
}
}

Помогите разобраться в ситуации...

[sonycman]

Переменная сама по себе никак не может принимать произвольные значения.
Где то идёт запись в эту переменную.

Приведите весь код, работающий с данными volatile переменными.

[vROMAv]

А где написано что event_flags принимает произвольные значения?
Это не весь код, в начале программы выполняется event_flags |= START;
Затем пошагово выполняю код в avr studio. Иногда натыкаюсь на вышеописанную ситуацию: В прерывании event_flags устанавливается (значение изменяется, видно в окне Watch), а на следующем шаге - выхожу из прерывания и вижу что event_flags снова равна START

Весь код проблематично привести...

[GetSmart]

Весь код проблематично привести...

Поиском найдите где эта переменная ещё меняется. И подумайте в каком из этих мест она может затираться.

Очень маловероятно чтобы эта переменная была в регистре. Листинг скажет точно.

[vROMAv]

Я же описал выше: Хожу по коду пошагово по F11,
Попадаю в прерывание USARTa здесь пошагово дохожу до строчек

event_flags |= PACK_NOW;
macro_rx = 1;

Смотрю в окно watch - переменная event_flags равна установленному вначале программы значению (0x01), затем выполняю 2 вышеприведенные строчки и вижу как event_flags стало равно 0x21, а macro_rx стало равным 1, затем снова делаю шаг по F11, и выхожу из прерывания и попадаю в main_loop, В ОКНЕ watch снова event_flags равен 0x01. Т.е. никаких команд после выхода из прерывания не было, а значение поменялось. Это могу объяснить только тем что event_flag была в регистре.

Сообщение отредактировал vROMAv - Feb 10 2011, 14:16

[GetSmart]

Ходите по асм-коду. Там всё предельно ясно - в регистре или нет.

[Oldring]

т.е. независимо от volatile компилятор по ходу разместил
переменную event_flag в регистре??!!!

Включите генерацию компилятором ассемблерного кода и посмотрите, где лежит переменная. Глюки компиляторов тоже встречаются, но реже, чем ошибки программистов.

[_Артём_]

В AVRStudio добавте переменную в Watch и увидите где она расположена.

[vROMAv]

В файле mega32.map переменной event_flags нет вообще это может чтонибудь значить?

[ar__systems]

Дался вам этот волатайл. При чем тут вообще это? Обычная статическая переменная. Ей без разницы, где она лежит, в регистре или в памяти. Если компилятор ее в регистр положил, наверное этот регистр никто больше не используют. 99% процентов что ваша программа кривая.

Сообщение отредактировал ar__systems - Feb 10 2011, 16:07

[firstvald]

А есть опции оптимизатора? Если есть возможность спуститься на несколклько шагов в оптимизации и посмотреть как будет работать?

Ну второе , что можно посмотреть : по мапу посмотреть какие переменные размещаются до этой переменной, если там есть массив - надо насторожиться и попробовать ему размер увеличить - может он проезжается .

[demiurg_spb]

// WinAVR-20090313 //

Есть и посвежее - рекомендую...

volatile uint8_t event_flags;

Сделайте 8 байтовых переменных вместо битовых флагов - это самое простое.
Или работайте с битами атомарно (при запрещённых прерываниях, что может быть более накладно).

Дался вам этот волатайл. При чем тут вообще это?

При том... Читайте стандарт там всё подробно описано.

[XVR]

А у вас в main_loop'е случайно нет локальной переменной event_flags?

[vROMAv]

Всем спасибо за помощь. Действительно в одном из участков кода проморгал атомарность((

ЗЫ: Старался всегда учитывать эту возможную бяку, но никогда ещё на неё не попадал. А ведь хорошо что попал, впредь буду более внимательно относится к битовым операциям.

[_Pasha]

А ведь хорошо что попал, впредь буду более внимательно относится к битовым операциям.

Присмотритесь повнимательнее в пространство SFR - GPR0 итд - меняются легко флаги, это если не хочется в регистре переменную держать.