From 8e3cf978c448e2bca1b5bc6d40512717172f2b15 Mon Sep 17 00:00:00 2001 From: Sachin Parekh Date: Wed, 26 Feb 2020 19:08:26 +0530 Subject: [PATCH] Xtensa_ESP32: Change _iram_end to _iram_text_end xtensa_loadstore_handler.S uses _iram_end to prevent modification of IRAM code. With the LoadStore exception handler in place, IRAM can also be used for .bss and .data section. Hence the sanity check should be based upon _iram_text_end and not _iram_end --- .../ThirdParty/GCC/Xtensa_ESP32/xtensa_loadstore_handler.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/portable/ThirdParty/GCC/Xtensa_ESP32/xtensa_loadstore_handler.S b/portable/ThirdParty/GCC/Xtensa_ESP32/xtensa_loadstore_handler.S index f4312df79..32d7a8a16 100644 --- a/portable/ThirdParty/GCC/Xtensa_ESP32/xtensa_loadstore_handler.S +++ b/portable/ThirdParty/GCC/Xtensa_ESP32/xtensa_loadstore_handler.S @@ -86,7 +86,7 @@ LoadStoreErrorHandler: /* Check whether the address lies in the valid range */ rsr a3, excvaddr - movi a4, _iram_end // End of code section of IRAM + movi a4, _iram_text_end // End of code section of IRAM bge a3, a4, 1f movi a4, SOC_CACHE_APP_LOW // Check if in APP cache region blt a3, a4, .LS_wrong_opcode @@ -280,7 +280,7 @@ AlignmentErrorHandler: /* Check whether the address lies in the valid range */ rsr a3, excvaddr - movi a4, _iram_end // End of code section of IRAM + movi a4, _iram_text_end // End of code section of IRAM bge a3, a4, 1f movi a4, SOC_CACHE_APP_LOW // Check if in APP cache region blt a3, a4, .LS_wrong_opcode