IAR сошел с ума. Опции

[Jenya7]

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

Код

if (usart1_rx_ready)
{
   usart1_rx_ready = 0;
   PARSER_ParseCommand(usart1_rx_buf);
}

ставлю точку останова на PARSER_ParseCommand(usart1_rx_buf); - вижу пришла красивая команда с '\0' в конце.
захожу внутрь функции

Код

uint32_t PARSER_ParseCommand(char *str)
{
    uint32_t com_found = 0;
    uint32_t len = 0;
    
    len = strlen(str);
    
    char temp_str[len];

strlen вычисляет длину строки - 504588880 - и естественно при алокации - char temp_str[len] - выпадаем в хард фолт.
Ну и кто он (IAR) после этого?

Сообщение отредактировал Jenya7 - Jul 6 2018, 08:18

[Arlleex]

Код

if (usart1_rx_ready)
{
   usart1_rx_ready = 0;
   PARSER_ParseCommand(usart1_rx_buf);
}
uint32_t PARSER_ParseCommand(char *str)
{
    uint32_t com_found = 0;
    uint32_t len = 0;
    len = strlen(str);
}

usart1_rx_ready есть volatile-переменная?
PARSER_ParseCommand() определена как функция, возвращающая значение. Если явно от нее ничего не требуется, значит и пишите (void)PARSER_ParseCommand(...), тем более когда применяются оптимизирующие ключи компиляции.
Сам буфер usart1_rx_buf должен иметь модификатор volatile, поскольку является буфером приема сырых данных и важно дать понять оптимизатору, чтобы не борогозил особо.
Еще в uint32_t PARSER_ParseCommand(char *str) я бы добавил uint32_t PARSER_ParseCommand(volatile char *str) для соответствия типов, тогда уж.
Вот теперь можно начинать включать оптимизацию и смотреть. Я, конечно, не сторонник ставить volatile везде и всюду, но народ требует крови и зрелищ

Сообщение отредактировал Arlleex - Jul 6 2018, 09:30

[Jenya7]

Код

if (usart1_rx_ready)
{
    usart1_rx_ready = 0;
    PARSER_ParseCommand(usart1_rx_buf);
}
uint32_t PARSER_ParseCommand(char *str)
{
     uint32_t com_found = 0;
     uint32_t len = 0;
     len = strlen(str);
}

usart1_rx_ready есть volatile-переменная?
PARSER_ParseCommand() определена как функция, возвращающая значение. Если явно от нее ничего не требуется, значит и пишите (void)PARSER_ParseCommand(...), тем более когда применяются оптимизирующие ключи компиляции.
Сам буфер usart1_rx_buf должен иметь модификатор volatile, поскольку является буфером приема сырых данных и важно дать понять оптимизатору, чтобы не борогозил особо.
Еще в uint32_t PARSER_ParseCommand(char *str) я бы добавил uint32_t PARSER_ParseCommand(volatile char *str) для соответствия типов, тогда уж.
Вот теперь можно начинать включать оптимизацию и смотреть. Я, конечно, не сторонник ставить volatile везде и всюду, но народ требует крови и зрелищ

если я объявляю volatile char usart1_rx_buf[RX1_BUFFERSIZE];
то IAR ругается во многих случаях - и мне приходиться делать приведение к типу

Код

memset((char *)usart1_rx_buf, '\0', RX1_BUFFERSIZE);
PARSER_ParseCommand(volatile char *com_str)
uint32_t len = StrLen((char*)com_str);
strcpy (temp_str, (char*)com_str);

Сообщение отредактировал Jenya7 - Jul 6 2018, 09:39

[VladislavS]

если я объявляю volatile char usart1_rx_buf[RX1_BUFFERSIZE]; то IAR ругается во многих случаях

И заполняете его в прерываниях, ку?

А вы тоже на стек его, чтобы интересней было

[Jenya7]

И заполняете его в прерываниях, ку?

А вы тоже на стек его, чтобы интересней было

usart1_rx_buf принимает правильные значения, адрес передается правильно нигде не затирается.
переделал на volatile - по прежнему uint32_t len = StrLen(com_str); выдает мусор с оптимизацией.

[Arlleex]

usart1_rx_buf принимает правильные значения, адрес передается правильно нигде не затирается.
переделал на volatile - по прежнему uint32_t len = StrLen(com_str); выдает мусор с оптимизацией.

Вы когда оптимизацию включаете - забудьте про адекватность отладчика сразу.
Прочитайте мое сообщение еще раз внимательно. После всех озвученных мной пунктов программа падает в HF?

[Jenya7]

Вы когда оптимизацию включаете - забудьте про адекватность отладчика сразу.
Прочитайте мое сообщение еще раз внимательно. После всех озвученных мной пунктов программа падает в HF?

да.

то есть включил оптимизацию - хрен тебе адекватность - выпал в хард фолт. потом отключил дебагер, сделал рисет.....и там же остался - в жо...в хард фолте

нет что то ахренительно сломалось. глобальные переменные принимают мусорные значения даже при инициализации.

ЧИЗЭС КРАЙСТ! вот я дурак. ошибся и передавал в функцию memcpy(&mot_task, addr, size); неправильный размер. у меня там по ходу убивалось около тысячи переменных в РАМ.

все ж таки жив ИАРка, рулит. надо свой мемкопи написать с проверкой размера.

Сообщение отредактировал Jenya7 - Jul 6 2018, 13:45

[scifi]

ЧИЗЭС КРАЙСТ! вот я дурак. ошибся и передавал в функцию memcpy(&mot_task, addr, size); неправильный размер. у меня там по ходу убивалось около тысячи переменных в РАМ.

Семён Семёныч...