[Tarantool-patches] [PATCH luajit 0/2] Fix lua_yield from the C hook.
Igor Munkin
imun at tarantool.org
Tue Jul 4 20:10:40 MSK 2023
Sergey,
I've checked the patchset into all long-term branches in
tarantool/luajit and bumped a new version in master, release/2.11 and
release/2.10.
On 22.06.23, Sergey Kaplun wrote:
> This patchset fixes the behaviour of the `lua_yield()` invocation inside
> C hooks. The first patch fixes the behaviour for non-GC64 mode, but the
> GC64 mode is still failing with a core dump. The second patch fixes
> behaviour for the GC64 mode as well.
>
> This patchset is based on the our C tests to be introduced, but their
> patch set has 2 LGTMs, so I just checkout this branch from the branch
> with C tests.
>
> PR: https://github.com/tarantool/tarantool/pull/8804
> Branch: https://github.com/tarantool/luajit/tree/skaplun/gh-noticket-yield-c-hook
> Related issue: https://github.com/tarantool/tarantool/issues/8516
> Mail list: https://www.freelists.org/post/luajit/BUG-Unable-to-yield-in-a-debug-hook-in-latest-21-beta
>
> Mike Pall (2):
> Fix lua_yield() from C hook.
> Another fix for lua_yield() from C hook.
>
> src/lj_api.c | 5 +-
> src/lj_ccallback.c | 2 +-
> src/lj_err.c | 2 +-
> src/lj_frame.h | 2 +-
> src/lj_meta.c | 2 +-
> .../fix-yield-c-hook-script.lua | 19 +++++++
> .../tarantool-c-tests/fix-yield-c-hook.test.c | 49 +++++++++++++++++++
> 7 files changed, 75 insertions(+), 6 deletions(-)
> create mode 100644 test/tarantool-c-tests/fix-yield-c-hook-script.lua
> create mode 100644 test/tarantool-c-tests/fix-yield-c-hook.test.c
>
> --
> 2.34.1
>
--
Best regards,
IM
More information about the Tarantool-patches
mailing list