Ошибка при отладке в AVR Studio маленького макроса Опции

[_Алекс]

Была задача получить текущий адрес выполнения программы – программный счетчик PC.
Среда разработки IAR
Запомнить его, сделал вот так, результат, при отладке в AVR STUDIO выдает ошибку с переполнение стека после нескольких прогонах программы, интересно что при вызове rcall L1, стек увеличивается на 2 байта, при pop, pop уменьшается на два, вобщем кажется все правильно.

Макрос взятия адреса и функция в макросе, которая по идеи должна сохранять PC, но просто сделана как тестовая.

#define OutSaveContekst\
{\
asm ("rcall L1\n" "L1: pop r16\n" "pop r17\n");\
SaveContext();\
\
}

Функция SaveContext (по идеи сохранение контекста) ни чего полезного пока
void SaveContext(void){
static int t = 0;
t++;
};

Функция тест в которой применен макрос
void test(void){
static char i;
if (i = = 5){
OutSaveContekst;
i = i + 1;
};
i = i + 1;
};

Главная функция
void main( void )
{
while(1){
test();
}
};

[IgorKossak]

prottoss, Вы получили таким образом значение SP, а никак не PC, как требовалось.
Очевидно, что после выталкивания двух байт из стека, их надо затолкать обратно, а копию в r16, r17 вернуть из подпрограммы.
Либо как в Вашем случае привести SP к типу указателя на int и взять по нему значение (с прединкрементом по-моему).
В любом случае эту операцию надо делать атомарно (непрерываемо).

[prottoss]

prottoss, Вы получили таким образом значение SP, а никак не PC, как требовалось.
Очевидно, что после выталкивания двух байт из стека, их надо затолкать обратно, а копию в r16, r17 вернуть из подпрограммы.
Либо как в Вашем случае привести SP к типу указателя на int и взять по нему значение (с прединкрементом по-моему).
В любом случае эту операцию надо делать атомарно (непрерываемо).

Сорри, конечно же надо не было не так - сказал одно, а думал о другом))) Наверное вот так правильнее:

Код

unsigned int OutSaveContekst(void)
{      return *((unsigned int __flash *)SP);
}

Тогда получается в асме:

Код

   19          unsigned int OutSaveContekst(void)
   \                     OutSaveContekst:
     20          {      return *((unsigned int __flash *)SP);
   \   00000000   B7ED               IN      R30, 0x3D
   \   00000002   B7FE               IN      R31, 0x3E
   \   00000004   9105               LPM     R16, Z+
   \   00000006   9114               LPM     R17, Z
   \   00000008   9508               RET
     21          }

prottoss, Вы получили таким образом значение SP, а никак не PC, как требовалось.
Очевидно, что после выталкивания двух байт из стека, их надо затолкать обратно, а копию в r16, r17 вернуть из подпрограммы.
Либо как в Вашем случае привести SP к типу указателя на int и взять по нему значение (с прединкрементом по-моему).
В любом случае эту операцию надо делать атомарно (непрерываемо).

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

Код

unsigned int __flash *OutSaveContekst(void)
   \                     OutSaveContekst:
     20          {      return (unsigned int __flash *)SP;
   \   00000000   B70D               IN      R16, 0x3D
   \   00000002   B71E               IN      R17, 0x3E
   \   00000004   9508               RET
     21          }

Абсолютно тоже самое))) А то что я привел в выше в этом же посте (извиняюсь за калабур) возвратит код команды B7ED IN R30, 0x3D