[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] coroutine-ucontext: Save fake stack for pooled coroutine
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH v2] coroutine-ucontext: Save fake stack for pooled coroutine |
Date: |
Wed, 17 Jan 2024 09:07:36 -0500 |
On Wed, Jan 17, 2024 at 04:06:02PM +0900, Akihiko Odaki wrote:
> Coroutine may be pooled even after COROUTINE_TERMINATE if
> CONFIG_COROUTINE_POOL is enabled and fake stack should be saved in
> such a case to keep AddressSanitizerUseAfterReturn working. Even worse,
> I'm seeing stack corruption without fake stack being saved.
>
> Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> ---
> Changes in v2:
> - Added missing set_current() (Marc-André Lureau)
> - Added G_STATIC_ASSERT(!IS_ENABLED(CONFIG_TSAN)) (Marc-André Lureau)
> - Renamed terminate() to terminate_asan() for clarity and consistency.
> - Changed terminate_asan() to call start_switch_fiber_asan() for
> consistency.
> - Link to v1:
> 20240112-asan-v1-1-e330f0d0032c@daynix.com">https://lore.kernel.org/r/20240112-asan-v1-1-e330f0d0032c@daynix.com
> ---
> util/coroutine-ucontext.c | 35 ++++++++++++++++++++++++++---------
> 1 file changed, 26 insertions(+), 9 deletions(-)
Thanks, applied to my block tree:
https://gitlab.com/stefanha/qemu/commits/block
Stefan
signature.asc
Description: PGP signature