Обработчик прерываний в HI-TECH, Вызов подпрограмм Опции

[Grigorij]

Здравствуйте.

Наконец-таки начал переползать с ассемблера на Си и возникла следующая проблем. Имеется код (приведены только отрывки на которых спотыкается компилятор):

Код

//изменение скорости обмена
void SetBRate(unsigned brate)
{
....
}

//отправка данных
void SendByte(unsigned brate,unsigned char data)
{    
     SetBRate(brate);
....
}

//отправка данных на индикатор
static void _send_byte(unsigned char data)
{
....
    SendByte(br_19200,data);    
....
    SetBRate(br_1200);
}    

//вывод символа на индикатор
void WriteChar(unsigned char chr)
{
     //корректировка кода для русских букв
     if (chr >= 0xC0 && chr <= 0xEF)
         chr -= 0x40;
     else
         if (chr >= 0xF0)
            chr -= 0x10;
    
     _send_byte(chr);
}

//обработчик прерываний от Timer 0    
void INT_TMR0(void)
{    
     static int cnt = 0;
    
      if (++cnt >= 10)
      {
           WriteChar('1');
           cnt = 0;
       }
    
       T0IF = 0;    
}

//обработчик прерываний
void interrupt INTHandler(void)
{
      if (T0IF) INT_TMR0();    //прерывание от Timer 0
      if (RCIF) INT_RxD();    //прерывание от приемника EUSART
}

При комплияции выдаются следующие сообщения об ошибках:

Код

Error   [472]; . non-reentrant function "__send_byte" appears in multiple call graphs: rooted at "_INTHandler" and "_main"

Error   [472]; . non-reentrant function "_SendByte" appears in multiple call graphs: rooted at "_INTHandler" and "_main"

Error   [472]; . non-reentrant function "_SetBRate" appears in multiple call graphs: rooted at "_INTHandler" and "_main"

В документации на HI-TECH написано, что:

(472) non-reentrant function "*" appears in multiple call graphs: rooted at "*" and "*" (Linker)
This function can be called from both main-line code and interrupt code. Use the reentrant keyword, if this compiler supports it, or recode to avoid using local variables or parameters, or duplicate the function, e.g.:

Однако из описания ошибки я так и не понял, как ее исправить. Подскажить, на какие грабли я наступил?

P.S. Используется MPLAB 8.00 + HI-TECH 9.60 LITE (который идет вместе с MPLAB 8.00). Контроллер PIC16F690.

[evc]

...
(472) non-reentrant function "*" appears in multiple call graphs: rooted at "*" and "*" (Linker)
This function can be called from both main-line code and interrupt code. Use the reentrant keyword, if this compiler supports it, or recode to avoid using local variables or parameters, or duplicate the function, e.g.:

Однако из описания ошибки я так и не понял, как ее исправить. Подскажить, на какие грабли я наступил?

P.S. Используется MPLAB 8.00 + HI-TECH 9.60 LITE (который идет вместе с MPLAB 8.00). Контроллер PIC16F690.

Компилер вам говорит: "Это НЕ функция которая позволяет многократный вход в нее, а она вызываемая в нескольких мест "*" и "*". Эту функцию возможно вызывать из обеих "main" и "interrupt". Пользуйте ключевое слово "reentrant", если этот компилер его поддерживает, или перепишите ее, чтобы она не использовала локальные переменные или продублируйте ее."

Дело в том что вы пользуете эту функцию и в основной программе и в обработчике прерывании. Задумайтесь что случится с данными, если прерывание наступит во время исполнении именно этой функции и обработчик прерывания попытается тоже вызвать ее!
В прерывании лучше внешние функции вообще не применять.

[DL36]

Компилер вам говорит: "Это НЕ функция которая позволяет многократный вход в нее, а она вызываемая в нескольких мест "*" и "*". Эту функцию возможно вызывать из обеих "main" и "interrupt". Пользуйте ключевое слово "reentrant", если этот компилер его поддерживает, или перепишите ее, чтобы она не использовала локальные переменные или продублируйте ее."

Ключевое слово #pragma interrupt_level 0

Дело в том что вы пользуете эту функцию и в основной программе и в обработчике прерывании. Задумайтесь что случится с данными, если прерывание наступит во время исполнении именно этой функции и обработчик прерывания попытается тоже вызвать ее!
В прерывании лучше внешние функции вообще не применять.

[evc]

Ключевое слово #pragma interrupt_level 0

это слово указывает линкеру, что потребитель гарантирует и береть на себя ответственость, что эта функция не будет вызвана обработчиком прерывании и основной программы одновременно, что бы тот (линкер) не выдавал сообщении об ошибке. В некоторых компиляторах (поддерживающие рекурсию, например) есть возможность объявить функцию как "reentrant", т.е. она может быть вызвана еще раз не завершив свою работу.

[Grigorij]

это слово указывает линкеру, что потребитель гарантирует и береть на себя ответственость, что эта функция не будет вызвана обработчиком прерывании и основной программы одновременно, что бы тот (линкер) не выдавал сообщении об ошибке. В некоторых компиляторах (поддерживающие рекурсию, например) есть возможность объявить функцию как "reentrant", т.е. она может быть вызвана еще раз не завершив свою работу.

К сожалению ключевое слово #pragma interrupt_level 0 не помогло. Если я объявляю ф-цию следующим образом:

Код

#pragma interrupt_level 0
void SendByte(unsigned brate,unsigned char data)
{    
     SetBRate(brate);
     TXREG = data;
     while (!TRMT);
}

То получаю от компилятор сообщение:

Код

Error   [473]; . function "_SendByte" is not called from specified interrupt_level

Проблему пока решил вынесением проверки флагов прерывания в основную программу, убрав обработчик прерываний (потом подумаю как его еще можно использовать).

Код

void main(void)
{        
    Init();
    
    while (1)
    {
         if (T0IF) INT_TMR0();
         if (RCIF) INT_RxD();
    }
}

Решение, наверное, не слишком красивое, но пока сойдет. Как вариант, думаю продублировать некоторые ф-ции, которые используются как в обработчике прерываний, так и в основной программе, т.е. сделать что-то на подобии:

Код

void intr_SetBRate(unsigned brate)
{
....
}

void intr_SendByte(unsigned brate,unsigned char data)
{    
     intr_SetBRate(brate);
     TXREG = data;
     while (!TRMT);
}

Но сдерживат тот факт, что произойдет увеличит объем кода, а это не есть хорошо. Тоже каксается и использования макросов.

Большое спасибо всем за помощь.

[DL36]

К сожалению ключевое слово #pragma interrupt_level 0 не помогло. Если я объявляю ф-цию следующим образом:

Надо добавить такую же строчку перед обработчиком прерывания. Хотя в Вашем случае видимо оптимальнее сделать две одинаковые функции с разными именами.

Надо учиться использовать прерывания, это основа без нее никак.

[Grigorij]

Надо добавить такую же строчку перед обработчиком прерывания. Хотя в Вашем случае видимо оптимальнее сделать две одинаковые функции с разными именами.

Спасибо, действительно помогло добавление #pragma interrupt_level 0 перед обработчиком прерываний. Хотя я думаю в обработчик прерываний вынести обработку только наиболее значимых прерываний, а все остальные обработать так, как я писал выше (т.е. в основном цикле программы). Думаю должно получиться неплохо.

Хотя вариант с использованием двух одинаковых функции с разными именами я пока еще не отбросил.

Надо учиться использовать прерывания, это основа без нее никак.

Знаю, что основа. Поэтому и задал вопрос, чтобы понять как пользоваться прерываниями в HI-TECH, т.к. в asm-е у меня вопросов с ними вроде не возникало.

[DL36]

Спасибо, действительно помогло добавление #pragma interrupt_level 0 перед обработчиком прерываний. Хотя я думаю в обработчик прерываний вынести обработку только наиболее значимых прерываний, а все остальные обработать так, как я писал выше (т.е. в основном цикле программы). Думаю должно получиться неплохо.

Хотя вариант с использованием двух одинаковых функции с разными именами я пока еще не отбросил.
Знаю, что основа. Поэтому и задал вопрос, чтобы понять как пользоваться прерываниями в HI-TECH, т.к. в asm-е у меня вопросов с ними вроде не возникало.

Старайтесь в прерывании производить только самые необходимые действия. Например принять байт по уарт, положить его в буфер, поднять флаг что надо что то сделать и уйти из прерывания, дабы не потерять другие вызовы.

Попробуйте описать тут концептуально свою задачу, думаю народ поможет, поскольку у Вас где то ошибка в концепции построения программы.

[Галстук]

...у Вас где то ошибка в концепции построения программы.

а может концепция верная.

В PICC-18 User Guide, 4.11.4 Interrupt Levels рассматривается ситуация использования одной и той же функции из interrupt и из main-line code. Для этого случая специально предназначена #pragma interrupt_level 0, возможно еще значение 1. Компилятор после этого ругаться перестает. Считается, что пользователь для всех функций, которым приписан один уровень (для чего надо написать эту волшебную фразу перед каждой из них), обеспечит правильную работу - т.е. они не будут вызваны одновременно из interrupt и main-line. Напрмер, перед вызовом функции из main-line можно запретить прерывания (и разрешить прерывания сразу после возврата из нее).

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