tarantool / luajit / 13966928732

93%

tarantool/master: 93%

Build Type

push

github

Committed by Buristan

Commit Message

ffi/gc: restore back rehashing of finalizers table

This is a follow-up to the commits
2115828bc ("FFI: Drop finalizer table
rehash after GC cycle.") and bfcbaa70e
("Drop unused function wrapper."). After them, the rehashing of the cdata
finalizer table at the end of the GC cycle is dropped. Without
reshashing of this table, the table increases the estimated amount of
memory for the GC. Hence, with the bigger `estimate`, the threshold
before starting the GC cycle is increased too. This allows allocating
more cdata objects and increasing the size of the finalizer table again.
This increases the memory estimate again and so on. As a result, we have
unlimited memory growth without rehashing of the table for the
cdata-intensive workloads.

This patch reverts back the code changes (but not the test) of the
aforementioned commits. Also, it fixes the possible crash after
rehashing of the cdata finalizers table by adding the protected call to
the GC steps on the trace.

Coverage Stats

5705 of 6040 branches covered (94.45%)

Branch coverage included in aggregate %.

20 of 20 new or added lines in 2 files covered. (100.0%)

5 existing lines in 3 files now uncovered.

21766 of 23480 relevant lines covered (92.7%)

3835879.96 hits per line