[Tarantool-patches] [PATCH v5 luajit 0/3] Valgrind testing
Sergey Kaplun
skaplun at tarantool.org
Wed Dec 11 16:21:05 MSK 2024
Changes in v5:
- Backported patch to fix "uninitalised value" warning during vmevent
processing of the `IR_NOP`.
- Added the additional suppression file to avoid false positives in
string comparisons not covered by the original suppression file.
- Remove the `--verbose` flag, since it makes the output too messy.
Changes in v4:
- Replaced the `VALGRIND_OPTIONS` variable with `VALGRIND_OPTS`, which
is provided by Valgrind.
- Enabled running `tarantool-c-tests` under Valgrind:
* Disabled the `tarantool-c-tests/gh-8594-sysprof-ffunc-crash.c_test` due
to timeout.
- Renamed the variable for skipping tests to `LUAJIT_TEST_USE_VALGRIND`
and setted it via `LUA_TEST_ENV_MORE`.
- Added the `--error-exitcode=1` flag to `VALGRIND_OPTS` in CI for stricter
error handling.
Branch: https://github.com/tarantool/luajit/tree/mandesero/lj-3705-turn-off-strcmp-opt-in-debug
Related issues:
* https://github.com/tarantool/tarantool/issues/10709
* https://github.com/tarantool/tarantool/issues/3705
* https://github.com/LuaJIT/LuaJIT/issues/624
Maksim Tiushev (2):
cmake: run tests with Valgrind
ci: add Valgrind testing workflow
Mike Pall (1):
Ensure full init of IR_NOP instructions.
.github/actions/setup-valgrind/README.md | 12 +++
.github/actions/setup-valgrind/action.yml | 12 +++
.github/workflows/valgrind-testing.yaml | 102 ++++++++++++++++++
CMakeLists.txt | 5 +
src/lj_asm.c | 2 +-
src/lj_extra.supp | 19 ++++
src/lj_ir.h | 8 ++
src/lj_opt_dce.c | 5 +-
src/lj_opt_mem.c | 25 +----
test/CMakeLists.txt | 19 ++++
test/tarantool-c-tests/CMakeLists.txt | 11 +-
.../gh-8594-sysprof-ffunc-crash.test.c | 9 ++
test/tarantool-tests/CMakeLists.txt | 6 ++
.../gh-7745-oom-on-trace.test.lua | 1 +
.../lj-1034-tabov-error-frame.test.lua | 1 +
.../lj-512-profiler-hook-finalizers.test.lua | 5 +-
.../lj-726-profile-flush-close.test.lua | 5 +-
.../profilers/gh-5688-tool-cli-flag.test.lua | 2 +
...4-add-proto-trace-sysprof-default.test.lua | 2 +
.../profilers/misclib-sysprof-lapi.test.lua | 2 +
20 files changed, 225 insertions(+), 28 deletions(-)
create mode 100644 .github/actions/setup-valgrind/README.md
create mode 100644 .github/actions/setup-valgrind/action.yml
create mode 100644 .github/workflows/valgrind-testing.yaml
create mode 100644 src/lj_extra.supp
--
2.47.0
More information about the Tarantool-patches
mailing list