diff options
author | Szabolcs Nagy <szabolcs.nagy@arm.com> | 2020-04-01 10:31:41 +0100 |
---|---|---|
committer | Szabolcs Nagy <szabolcs.nagy@arm.com> | 2020-06-12 14:34:39 +0100 |
commit | 72d607dcade45c73c3f979d5f8ec1064c15a98b5 (patch) | |
tree | c6e5d3710bab6b8966771ed9899b27759da9b1fb /stdio-common/bug18.c | |
parent | 26245e2725ad815bbfcd18f58515de4a88cc03e0 (diff) | |
download | glibc-72d607dcade45c73c3f979d5f8ec1064c15a98b5.tar.gz glibc-72d607dcade45c73c3f979d5f8ec1064c15a98b5.tar.xz glibc-72d607dcade45c73c3f979d5f8ec1064c15a98b5.zip |
aarch64: fix swapcontext for BTI
setcontext returns to the specified context via an indirect jump, so there should be a BTI j. In case of getcontext (and all other returns_twice functions) the compiler adds BTI j at the call site, but swapcontext is a normal c call that is currently not handled specially by the compiler. So we change swapcontext such that the saved context returns to a local address that has BTI j and then swapcontext returns to the caller via a normal RET. For this we save the original return address in the slot for x1 of the context because x1 need not be preserved by swapcontext but it is restored when the context saved by swapcontext is resumed. The alternative fix (which is done on x86) would make swapcontext special in the compiler so BTI j is emitted at call sites, on x86 there is an indirect_return attribute for this, on AArch64 we would have to use returns_twice. It was decided against because such fix may need user code updates: the attribute has to be added when swapcontext is called via a function pointer and it breaks always_inline functions with swapcontext. Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Diffstat (limited to 'stdio-common/bug18.c')
0 files changed, 0 insertions, 0 deletions