Флаги прерывания. Опции

[Jenya7]

Сейчас делаю так. Есть несколько переменных.

Код

volatile uint8_t int1, int2, int3.

В прерывании делаю int1 = 1 и в main

Код

If (int1)
{
    int1 = 0;
     // и так далее
}

Подумал сделать так

Код

#define TIMER_COMP0    BIT0
#define TIMER_COMP1    BIT1
#define TIMER_UF        BIT2
#define ADC_SINCOS        BIT3
#define ADC_MAGREAD    BIT4

volatile uint32_t intFlags;

и потом в прерывании intFlags |= TIMER_COMP0; и в main

Код

if (intFlags & TIMER_COMP0)
{
    intFlags &= ~ TIMER_COMP0;
}

какой вариант лучше? второй по моему на одну операцию больше.

[aaarrr]

ИМХО, пока память не жмет, вариант 1 предпочтительнее.

[Jenya7]

ИМХО, пока память не жмет, вариант 1 предпочтительнее.

вроде не жмет. всего 4 кило, заполнено чуть больше 1 кило.

[ViKo]

Сделать структуру с битовыми полями, и добираться к ним через bit-banding. Не настаиваю, только предлагаю рассмотреть вариант.

[Jenya7]

Сделать структуру с битовыми полями, и добираться к ним через bit-banding. Не настаиваю, только предлагаю рассмотреть вариант.

а это разве не то же что и вариант 2?

[ViKo]

а это разве не то же что и вариант 2?

То же, только выглядит красивее.
Будете писать if (intFlags.TimerComp0) intFlags.TimerComp0 = 0;

[Jenya7]

То же, только выглядит красивее.
Будете писать if (intFlags.TimerComp0) intFlags.TimerComp0 = 0;

так мне не красивее мне быстрее надо.

[SSerge]

В прерывании можете делать что угодно, а вот операция сброса флага в основной программе должна быть атомарной (такой, что прерывание не может вклинится в середину этого действия и всё испортить).

int1 = 0; атомарна, а вот
intFlags &= ~ TIMER_COMP0;
или intFlags.TimerComp0 = 0;
таковыми не являются.
Но можно применять bit-banding, этот механизм позволяет атомарно устанавливать и сбрасывать битовые флаги.

[ViKo]

так мне не красивее мне быстрее надо.

А вы сделайте тестовый проект с разными вариантами, и посмотрите в листинг.
Если от этой почти невесомой проблемы зависит производительность проекта...

[Jenya7]

В прерывании можете делать что угодно, а вот операция сброса флага в основной программе должна быть атомарной (такой, что прерывание не может вклинится в середину этого действия и всё испортить).

int1 = 0; атомарна, а вот
intFlags &= ~ TIMER_COMP0;
или intFlags.TimerComp0 = 0;
таковыми не являются.
Но можно применять bit-banding, этот механизм позволяет атомарно устанавливать и сбрасывать битовые флаги.

мда...это весомый аргумент.

а что после прерывания исполнение не продолжиться со следующей инструкции?

Сообщение отредактировал Jenya7 - Mar 4 2015, 11:10

[SSerge]

мда...это весомый аргумент.

а что после прерывания исполнение не продолжиться со следующей инструкции?

Оно продолжится, но ..

intFlags &= ~ TIMER_COMP0;
транслируется в три команды:
1. выборка intFlags из памяти в регистр,
2. операция and с константой, результат остаётся в регистре,
3. запись из регистра в память нового значения intFlags.

Если после 1. но перед 3. в середину вклинится прерывание, которое установит в intFlags ещё какой-то бит, то этот бит будет потерян, поскольку после выхода из обработчика выполнится команда 3., и она запишет в intFlags старое значение этого бита, которое было до прерывания.

[Jenya7]

Оно продолжится, но ..

intFlags &= ~ TIMER_COMP0;
транслируется в три команды:
1. выборка intFlags из памяти в регистр,
2. операция and с константой, результат остаётся в регистре,
3. запись из регистра в память нового значения intFlags.

Если после 1. но перед 3. в середину вклинится прерывание, которое установит в intFlags ещё какой-то бит, то этот бит будет потерян, поскольку после выхода из обработчика выполнится команда 3., и она запишет в intFlags старое значение этого бита, которое было до прерывания.

понял. похоже лучше пожертвовать куском рама. он большой с него не убудет.

таки да - второй способ небезопасный. спасибо.

Сообщение отредактировал Jenya7 - Mar 4 2015, 14:09