Re: [Tarantool-patches] [PATCH v4 0/2] Implement LuaJIT platform metrics

[Kirill Yukhin <kyukhin@tarantool.org>]

Hello,

On 05 окт 09:30, Sergey Kaplun wrote:
> The series consists of 2 patches. The first one adds corresponding
> counters to LuaJIT internal structures. The second provides C and Lua
> API using this counters to collect metrics.
> 
> Branch: https://github.com/tarantool/luajit/tree/skaplun/gh-5187-luajit-metrics
> Issue: https://github.com/tarantool/tarantool/issues/5187
> 
> Changes in v2:
> - Fixed naming and comments
> - Fixed padding in struct GCState
> - Dropped unnecessary initialisations inside lua_newstate()
> - Avoided flushing any of metrics after each call of luaM_metrics()
> 
> Changes in v3:
> - Cleaned up mess in Makefile.dep
> - Fixed naming and comments
> - Fixed padding in struct GCState for 64-bit architectures
> - Fixed counting amount of JIT traces
> - Fixed objects counting at trace recording
> - Added counting of colors
> - Added C and Lua tests
> 
> Changes in v4:
> - Removed counting of colors
> - Changed global_State structure correspondingly for 32-bit arm build
> 
> Sergey Kaplun (2):
>   core: introduce various platform metrics
>   misc: add C and Lua API for platform metrics

I've checked your patchset into tarantool branch of luajit repo and
bumped a new version in 1.10, 2.4, 2.5 and master.

--
Regards, Kirill Yukhin