Найден баг в порте под Cortex-M3. Опции

[AHTOXA]

Собственно, сабж. Я в процессе оптимизаций PendSV_Handler() перестарался - соптимизировал доставание из стека регистра LR. А этого делать нельзя, ибо там хранится значение exc_return. И если вдруг os_context_switch_hook() попортит LR, то всё сразу накроется.
Баг нестрашный, в том смысле, что с ним прошивка либо работает нормально, либо совсем не работает. То есть, баг не плавающий. Но лучше исправить.
Для порта GCC я уже залил исправленный вариант в репозиторий.
В порте IAR баг тоже есть, надеюсь Андрей его скоро исправит.

[shreck]

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

[AHTOXA]

Рассказываю.
Когда возникает исключение, часть регистров автоматически запихивается в стек. В том числе адрес следующей инструкции для выполнения после возврата (это к вопросу Сергея про "Как же он узнает, куда ему возвращаться"). Кроме того, в регистр LR заносится специальное значение exc_return, в котором сохраняются флажки, описывающие используемый стек и режим привилегий прерываемого процесса. (В M4F там ещё есть флажок активности FPU). При возврате из исключения из этого exc_return восстанавливается режим работы процессора. (Возврат из исключения происходит при записи в PC значения exc_return. У него все старшие биты единички, и по этой маске процессор определяет, что нужно возвращаться).
Конкретно надо поменять вот что:
1. Добавить строчку POP {LR} после строчки BL os_context_switch_hook
2. Заменить строчку POP {PC} на BX LR


Блин! Написал, и понял, что всё было правильно. Отбой воздушной тревоги

[shreck]

Рассказываю ...

Блин! Написал, и понял, что всё было правильно. Отбой воздушной тревоги

"Не спеши выполнять. Отменят"

В догонку, но не в тему.
Был разговор, что надо бы пользователю дать возможность выбора системного таймера. И вроде бы ты такую возможность добавлял. А сейчас ее нету. Да и мелкая оптимизация с короткими/длинными вызовами куда-то делась.