diff options
| author | Daniele Buono | 2020-05-29 22:51:19 +0200 |
|---|---|---|
| committer | Stefan Hajnoczi | 2020-06-23 16:46:05 +0200 |
| commit | 58ebc2c31337734a8a79b0566b31b19040deb2ea (patch) | |
| tree | 0d085b3db59d811a055a23bea18537aa60da5d14 /include/qemu | |
| parent | minikconf: explicitly set encoding to UTF-8 (diff) | |
| download | qemu-58ebc2c31337734a8a79b0566b31b19040deb2ea.tar.gz qemu-58ebc2c31337734a8a79b0566b31b19040deb2ea.tar.xz qemu-58ebc2c31337734a8a79b0566b31b19040deb2ea.zip | |
coroutine: support SafeStack in ucontext backend
LLVM's SafeStack instrumentation does not yet support programs that make
use of the APIs in ucontext.h
With the current implementation of coroutine-ucontext, the resulting
binary is incorrect, with different coroutines sharing the same unsafe
stack and producing undefined behavior at runtime.
This fix allocates an additional unsafe stack area for each coroutine,
and sets the new unsafe stack pointer before calling swapcontext() in
qemu_coroutine_new.
This is the only place where the pointer needs to be manually updated,
since sigsetjmp/siglongjmp are already instrumented by LLVM to properly
support SafeStack.
The additional stack is then freed in qemu_coroutine_delete.
Signed-off-by: Daniele Buono <dbuono@linux.vnet.ibm.com>
Message-id: 20200529205122.714-2-dbuono@linux.vnet.ibm.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'include/qemu')
| -rw-r--r-- | include/qemu/coroutine_int.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/qemu/coroutine_int.h b/include/qemu/coroutine_int.h index bd6b0468e1..1da148552f 100644 --- a/include/qemu/coroutine_int.h +++ b/include/qemu/coroutine_int.h @@ -28,6 +28,11 @@ #include "qemu/queue.h" #include "qemu/coroutine.h" +#ifdef CONFIG_SAFESTACK +/* Pointer to the unsafe stack, defined by the compiler */ +extern __thread void *__safestack_unsafe_stack_ptr; +#endif + #define COROUTINE_STACK_SIZE (1 << 20) typedef enum { |