michaelh/zephyr
1
0
Fork 0
Code Releases Activity

ARC: MWDT: drop redundant stack checking

Browse source 
MWDT toolchain has Stackcheck_alloca option enabled by default.
So it adds stack checking in addition to Zephyr's stack checking.

As it is completely redundant let's drop it.

Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
This commit is contained in:
Eugeniy Paltsev 2020-09-28 21:38:44 +03:00 • committed by Carles Cufí
commit 9b0ef4f19a
2 changed files with 3 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

5
cmake/compiler/arcmwdt/target.cmake
View file

@ -36,5 +36,6 @@ foreach(isystem_include_dir ${NOSTDINC})
list(APPEND isystem_include_flags -isystem "\"${isystem_include_dir}\"") list(APPEND isystem_include_flags -isystem "\"${isystem_include_dir}\"")
endforeach() endforeach()
# common compile options, no copyright msg, little-endian, no small data # common compile options, no copyright msg, little-endian, no small data,
list(APPEND TOOLCHAIN_C_FLAGS -Hnocopyr -HL -Hnosdata) # no MWDT stack checking
list(APPEND TOOLCHAIN_C_FLAGS -Hnocopyr -HL -Hnosdata -Hoff=Stackcheck_alloca)

10
include/arch/arc/v2/linker.ld
View file

@ -206,16 +206,6 @@ SECTIONS {
__kernel_ram_end = .; __kernel_ram_end = .;
__kernel_ram_size = __kernel_ram_end - __kernel_ram_start; __kernel_ram_size = __kernel_ram_end - __kernel_ram_start;
#ifdef __MWDT_LINKER_CMD__
/* mwdt requires _fstack, _estack which will be used in _stkchk.
* _stkchk is inserted by mwdt automatically, if _fstack, _estack is not
* set correctly, the brk_s instruction will be called
* here, we use a trick to deceive the compiler.
*/
_fstack = _image_ram_start;
_estack = .;
#endif /* __MWDT_LINKER_CMD__ */
GROUP_END(RAMABLE_REGION) GROUP_END(RAMABLE_REGION)
/* Located in generated directory. This file is populated by the /* Located in generated directory. This file is populated by the