diff --git a/components/soc/include/soc/soc_memory_layout.h b/components/soc/include/soc/soc_memory_layout.h index 068890c8a..a393f0841 100644 --- a/components/soc/include/soc/soc_memory_layout.h +++ b/components/soc/include/soc/soc_memory_layout.h @@ -251,9 +251,26 @@ inline static void * IRAM_ATTR esp_ptr_diram_iram_to_dram(const void *p) { #endif } -inline static bool IRAM_ATTR esp_stack_ptr_is_sane(uint32_t sp) +inline static bool IRAM_ATTR esp_stack_ptr_in_dram(uint32_t sp) { //Check if stack ptr is in between SOC_DRAM_LOW and SOC_DRAM_HIGH, and 16 byte aligned. return !(sp < SOC_DRAM_LOW + 0x10 || sp > SOC_DRAM_HIGH - 0x10 || ((sp & 0xF) != 0)); } +#if CONFIG_SPIRAM_ALLOW_STACK_EXTERNAL_MEMORY +inline static bool IRAM_ATTR esp_stack_ptr_in_extram(uint32_t sp) +{ + //Check if stack ptr is in between SOC_EXTRAM_DATA_LOW and SOC_EXTRAM_DATA_HIGH, and 16 byte aligned. + return !(sp < SOC_EXTRAM_DATA_LOW + 0x10 || sp > SOC_EXTRAM_DATA_HIGH - 0x10 || ((sp & 0xF) != 0)); +} +#endif + +inline static bool IRAM_ATTR esp_stack_ptr_is_sane(uint32_t sp) +{ +#if CONFIG_SPIRAM_ALLOW_STACK_EXTERNAL_MEMORY + return (esp_stack_ptr_in_dram(sp) || esp_stack_ptr_in_extram(sp)); +#else + return esp_stack_ptr_in_dram(sp); +#endif +} + diff --git a/components/spi_flash/cache_utils.c b/components/spi_flash/cache_utils.c index 54a296fc6..495a748be 100644 --- a/components/spi_flash/cache_utils.c +++ b/components/spi_flash/cache_utils.c @@ -113,6 +113,8 @@ void IRAM_ATTR spi_flash_op_block_func(void *arg) void IRAM_ATTR spi_flash_disable_interrupts_caches_and_other_cpu(void) { + assert(esp_ptr_in_dram((const void *)get_sp())); + spi_flash_op_lock(); const uint32_t cpuid = xPortGetCoreID();