freertos: Make the tick/idle hooks dual core compatible

The way these hooks are implemented at the moment, once you register a tick or idle task hook, it is run by both cores, leading to intermittent crashes that are difficult to explain (and debug). One solution is to register the hook with the core that is currently running, which is what this patch attempts to do.

Merges https://github.com/espressif/esp-idf/pull/781
This commit is contained in:
Daniel Campora 2017-07-08 20:47:55 +02:00 committed by Angus Gratton
parent 110c71d3f1
commit 1c046f30a6

View file

@ -16,6 +16,7 @@
#include <stdint.h> #include <stdint.h>
#include <stdlib.h> #include <stdlib.h>
#include <stdbool.h> #include <stdbool.h>
#include "freertos/FreeRTOS.h"
#include "esp_attr.h" #include "esp_attr.h"
#include "esp_freertos_hooks.h" #include "esp_freertos_hooks.h"
@ -23,15 +24,16 @@
//an idle or tick hook. //an idle or tick hook.
#define MAX_HOOKS 8 #define MAX_HOOKS 8
static esp_freertos_idle_cb_t idle_cb[MAX_HOOKS]={0}; static esp_freertos_idle_cb_t idle_cb[portNUM_PROCESSORS][MAX_HOOKS]={0};
static esp_freertos_tick_cb_t tick_cb[MAX_HOOKS]={0}; static esp_freertos_tick_cb_t tick_cb[portNUM_PROCESSORS][MAX_HOOKS]={0};
void IRAM_ATTR esp_vApplicationTickHook() void IRAM_ATTR esp_vApplicationTickHook()
{ {
int n; int n;
int core = xPortGetCoreID();
for (n=0; n<MAX_HOOKS; n++) { for (n=0; n<MAX_HOOKS; n++) {
if (tick_cb[n]!=NULL) { if (tick_cb[core][n]!=NULL) {
tick_cb[n](); tick_cb[core][n]();
} }
} }
} }
@ -41,9 +43,10 @@ void esp_vApplicationIdleHook()
bool doWait=true; bool doWait=true;
bool r; bool r;
int n; int n;
int core = xPortGetCoreID();
for (n=0; n<MAX_HOOKS; n++) { for (n=0; n<MAX_HOOKS; n++) {
if (idle_cb[n]!=NULL) { if (idle_cb[core][n]!=NULL) {
r=idle_cb[n](); r=idle_cb[core][n]();
if (!r) doWait=false; if (!r) doWait=false;
} }
} }
@ -57,9 +60,10 @@ void esp_vApplicationIdleHook()
esp_err_t esp_register_freertos_idle_hook(esp_freertos_idle_cb_t new_idle_cb) esp_err_t esp_register_freertos_idle_hook(esp_freertos_idle_cb_t new_idle_cb)
{ {
int n; int n;
int core = xPortGetCoreID();
for (n=0; n<MAX_HOOKS; n++) { for (n=0; n<MAX_HOOKS; n++) {
if (idle_cb[n]==NULL) { if (idle_cb[core][n]==NULL) {
idle_cb[n]=new_idle_cb; idle_cb[core][n]=new_idle_cb;
return ESP_OK; return ESP_OK;
} }
} }
@ -69,9 +73,10 @@ esp_err_t esp_register_freertos_idle_hook(esp_freertos_idle_cb_t new_idle_cb)
esp_err_t esp_register_freertos_tick_hook(esp_freertos_tick_cb_t new_tick_cb) esp_err_t esp_register_freertos_tick_hook(esp_freertos_tick_cb_t new_tick_cb)
{ {
int n; int n;
int core = xPortGetCoreID();
for (n=0; n<MAX_HOOKS; n++) { for (n=0; n<MAX_HOOKS; n++) {
if (tick_cb[n]==NULL) { if (tick_cb[core][n]==NULL) {
tick_cb[n]=new_tick_cb; tick_cb[core][n]=new_tick_cb;
return ESP_OK; return ESP_OK;
} }
} }
@ -81,16 +86,18 @@ esp_err_t esp_register_freertos_tick_hook(esp_freertos_tick_cb_t new_tick_cb)
void esp_deregister_freertos_idle_hook(esp_freertos_idle_cb_t old_idle_cb) void esp_deregister_freertos_idle_hook(esp_freertos_idle_cb_t old_idle_cb)
{ {
int n; int n;
int core = xPortGetCoreID();
for (n=0; n<MAX_HOOKS; n++) { for (n=0; n<MAX_HOOKS; n++) {
if (idle_cb[n]==old_idle_cb) idle_cb[n]=NULL; if (idle_cb[core][n]==old_idle_cb) idle_cb[core][n]=NULL;
} }
} }
void esp_deregister_freertos_tick_hook(esp_freertos_tick_cb_t old_tick_cb) void esp_deregister_freertos_tick_hook(esp_freertos_tick_cb_t old_tick_cb)
{ {
int n; int n;
int core = xPortGetCoreID();
for (n=0; n<MAX_HOOKS; n++) { for (n=0; n<MAX_HOOKS; n++) {
if (tick_cb[n]==old_tick_cb) tick_cb[n]=NULL; if (tick_cb[core][n]==old_tick_cb) tick_cb[core][n]=NULL;
} }
} }