From b6b227e6ccec589b807af5b39acfe29bcc192342 Mon Sep 17 00:00:00 2001 From: jingli Date: Tue, 9 Nov 2021 20:21:17 +0800 Subject: [PATCH] fix: app crash when OTA because the OTA task's stack is in rtc fast memory --- components/spi_flash/cache_utils.c | 31 ++++++++++++++++-------------- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/components/spi_flash/cache_utils.c b/components/spi_flash/cache_utils.c index 3c054167d4..6f5d43d9f2 100644 --- a/components/spi_flash/cache_utils.c +++ b/components/spi_flash/cache_utils.c @@ -1,16 +1,8 @@ -// Copyright 2015-2016 Espressif Systems (Shanghai) PTE LTD -// -// Licensed under the Apache License, Version 2.0 (the "License"); -// you may not use this file except in compliance with the License. -// You may obtain a copy of the License at -// -// http://www.apache.org/licenses/LICENSE-2.0 -// -// Unless required by applicable law or agreed to in writing, software -// distributed under the License is distributed on an "AS IS" BASIS, -// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -// See the License for the specific language governing permissions and -// limitations under the License. +/* + * SPDX-FileCopyrightText: 2015-2021 Espressif Systems (Shanghai) CO LTD + * + * SPDX-License-Identifier: Apache-2.0 + */ #include #include @@ -68,6 +60,17 @@ static volatile bool s_flash_op_complete = false; static volatile int s_flash_op_cpu = -1; #endif +static inline bool esp_task_stack_is_sane_cache_disabled(void) +{ + const void *sp = (const void *)get_sp(); + + return esp_ptr_in_dram(sp) +#if CONFIG_ESP32_ALLOW_RTC_FAST_MEM_AS_HEAP || CONFIG_ESP32S2_ALLOW_RTC_FAST_MEM_AS_HEAP + || esp_ptr_in_rtc_dram_fast(sp) +#endif + ; +} + void spi_flash_init_lock(void) { s_flash_op_mutex = xSemaphoreCreateRecursiveMutex(); @@ -115,7 +118,7 @@ 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())); + assert(esp_task_stack_is_sane_cache_disabled()); spi_flash_op_lock();