feedc0de/esp-idf
1
0
Fork 0
forked from espressif/esp-idf
Code Pull Requests Activity

Merge branch 'bugfix/memprot_wrong_fi_check' into 'master'

Browse Source 
System/Memprot: Fixed voltage glitching detection logic

See merge request espressif/esp-idf!15407
This commit is contained in:
Martin Vychodil
2021-10-04 15:38:07 +00:00
parent 865fddde15 5344de34c3
commit d0b311227a
2 changed files with 72 additions and 71 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
components/esp_hw_support/port/esp32c3/memprot.c
View File

@@ -19,7 +19,7 @@
#include "riscv/interrupt.h" #include "riscv/interrupt.h"
#include "esp32c3/rom/ets_sys.h" #include "esp32c3/rom/ets_sys.h"
#include "esp_log.h" #include "esp_log.h"
#include "esp_fault.h"
#include "soc/cpu.h" #include "soc/cpu.h"
extern int _iram_text_end; extern int _iram_text_end;
@@ -526,12 +526,11 @@ void esp_memprot_set_prot_int(bool invoke_panic_handler, bool lock_feature, void
esp_memprot_set_monitor_en(MEMPROT_DRAM0_SRAM, false); esp_memprot_set_monitor_en(MEMPROT_DRAM0_SRAM, false);
} }
// do not enable if being debugged //if being debugged check we are not glitched and dont enable Memprot
if (esp_cpu_in_ocd_debug_mode()) { if (esp_cpu_in_ocd_debug_mode()) {
return; ESP_FAULT_ASSERT(esp_cpu_in_ocd_debug_mode());
} } else {
//panic handling
if (invoke_panic_handler) { if (invoke_panic_handler) {
if (use_iram0) { if (use_iram0) {
esp_memprot_set_intr_matrix(MEMPROT_IRAM0_SRAM); esp_memprot_set_intr_matrix(MEMPROT_IRAM0_SRAM);
@@ -546,8 +545,10 @@ void esp_memprot_set_prot_int(bool invoke_panic_handler, bool lock_feature, void
esp_memprot_set_split_line(MEMPROT_IRAM0_LINE_1_SPLITLINE, line_addr); esp_memprot_set_split_line(MEMPROT_IRAM0_LINE_1_SPLITLINE, line_addr);
esp_memprot_set_split_line(MEMPROT_IRAM0_LINE_0_SPLITLINE, line_addr); esp_memprot_set_split_line(MEMPROT_IRAM0_LINE_0_SPLITLINE, line_addr);
esp_memprot_set_split_line(MEMPROT_IRAM0_DRAM0_SPLITLINE, line_addr); esp_memprot_set_split_line(MEMPROT_IRAM0_DRAM0_SPLITLINE, line_addr);
esp_memprot_set_split_line(MEMPROT_DRAM0_DMA_LINE_0_SPLITLINE, (void *)(MAP_IRAM_TO_DRAM((uint32_t)line_addr))); esp_memprot_set_split_line(MEMPROT_DRAM0_DMA_LINE_0_SPLITLINE,
esp_memprot_set_split_line(MEMPROT_DRAM0_DMA_LINE_1_SPLITLINE, (void *)(MAP_IRAM_TO_DRAM((uint32_t)line_addr))); (void *) (MAP_IRAM_TO_DRAM((uint32_t) line_addr)));
esp_memprot_set_split_line(MEMPROT_DRAM0_DMA_LINE_1_SPLITLINE,
(void *) (MAP_IRAM_TO_DRAM((uint32_t) line_addr)));
//set permissions //set permissions
if (required_mem_prot & MEMPROT_IRAM0_SRAM) { if (required_mem_prot & MEMPROT_IRAM0_SRAM) {
@@ -557,13 +558,13 @@ void esp_memprot_set_prot_int(bool invoke_panic_handler, bool lock_feature, void
esp_memprot_iram_set_pms_area(MEMPROT_IRAM0_PMS_AREA_3, true, true, false); esp_memprot_iram_set_pms_area(MEMPROT_IRAM0_PMS_AREA_3, true, true, false);
} }
if (required_mem_prot & MEMPROT_DRAM0_SRAM) { if (required_mem_prot & MEMPROT_DRAM0_SRAM) {
esp_memprot_dram_set_pms_area( MEMPROT_DRAM0_PMS_AREA_0, true, false ); esp_memprot_dram_set_pms_area(MEMPROT_DRAM0_PMS_AREA_0, true, false);
esp_memprot_dram_set_pms_area(MEMPROT_DRAM0_PMS_AREA_1, true, true); esp_memprot_dram_set_pms_area(MEMPROT_DRAM0_PMS_AREA_1, true, true);
esp_memprot_dram_set_pms_area(MEMPROT_DRAM0_PMS_AREA_2, true, true); esp_memprot_dram_set_pms_area(MEMPROT_DRAM0_PMS_AREA_2, true, true);
esp_memprot_dram_set_pms_area(MEMPROT_DRAM0_PMS_AREA_3, true, true); esp_memprot_dram_set_pms_area(MEMPROT_DRAM0_PMS_AREA_3, true, true);
} }
//reenable protection //reenable the protection
if (use_iram0) { if (use_iram0) {
esp_memprot_monitor_clear_intr(MEMPROT_IRAM0_SRAM); esp_memprot_monitor_clear_intr(MEMPROT_IRAM0_SRAM);
esp_memprot_set_monitor_en(MEMPROT_IRAM0_SRAM, true); esp_memprot_set_monitor_en(MEMPROT_IRAM0_SRAM, true);
@@ -585,6 +586,7 @@ void esp_memprot_set_prot_int(bool invoke_panic_handler, bool lock_feature, void
esp_memprot_set_monitor_lock(MEMPROT_DRAM0_SRAM); esp_memprot_set_monitor_lock(MEMPROT_DRAM0_SRAM);
} }
} }
}
} }
uint32_t esp_memprot_get_dram_status_reg_1() uint32_t esp_memprot_get_dram_status_reg_1()

9
components/esp_hw_support/port/esp32s2/memprot.c
View File

@@ -802,11 +802,10 @@ esp_err_t esp_memprot_set_prot(bool invoke_panic_handler, bool lock_feature, uin
return ret; return ret;
} }
//connect to intr. matrix if not being debugged //if being debugged check we are not glitched and dont enable Memprot
if (!esp_cpu_in_ocd_debug_mode()) { if (esp_cpu_in_ocd_debug_mode()) {
ESP_FAULT_ASSERT(esp_cpu_in_ocd_debug_mode());
ESP_FAULT_ASSERT(!esp_cpu_in_ocd_debug_mode()); } else {
//initialize for specific buses (any memory type does the job) //initialize for specific buses (any memory type does the job)
if (invoke_panic_handler) { if (invoke_panic_handler) {
if (use_iram0 && (ret = esp_memprot_intr_init(MEMPROT_IRAM0_SRAM)) != ESP_OK) { if (use_iram0 && (ret = esp_memprot_intr_init(MEMPROT_IRAM0_SRAM)) != ESP_OK) {