tarantool icon indicating copy to clipboard operation
tarantool copied to clipboard

ci: testing ASAN instrumentation of LuaJIT internal allocator

Open mandesero opened this issue 1 year ago • 4 comments

This patch is a pre-patch (refer to [1]) to add ASAN instrumentation to the LuaJIT memory allocator.

LuaJIT has two usage scenarios for ASAN:

  • LuaJIT using sysmalloc
  • LuaJIT using internal memory allocator

This patch introduces jobs that separate the testing of these scenarios into individual jobs. This is necessary because using the internal allocator with ASAN requires -DLUAJIT_ENABLE_GC64=ON (refer to [1]).

Note: this patch is required to test the ASAN instrumentation of LuaJIT internal memory allocator.

[1]: Issue https://github.com/tarantool/tarantool/issues/10231

mandesero avatar Jul 11 '24 12:07 mandesero

Coverage Status

coverage: 87.619% (+0.002%) from 87.617% when pulling 5e1be84dbd48a6890e5b22de2f4dcfdeaf828c3e on mandesero:mandesero/asan-build-with-gc64-on into 7a6319528c0c3342187fa5ff2c3ecde882a5fd11 on tarantool:master.

coveralls avatar Jul 11 '24 12:07 coveralls

Waiting for #10227

mandesero avatar Jul 18 '24 16:07 mandesero

After discussion, it was decided to split the use of ASAN into two scenarios:

  • LuaJIT using sysmalloc: -DLUAJIT_USE_ASAN=ON
  • LuaJIT using the internal memory allocator: -DLUAJIT_USE_ASAN_HARDENING=ON

This patch should add jobs that testing this scenarious.

mandesero avatar Aug 12 '24 13:08 mandesero

Waiting for #10399

mandesero avatar Aug 12 '24 20:08 mandesero