[Tarantool-patches] [PATCH] test: stabilize flaky fiber memory leak detection

Kirill Yukhin kyukhin at tarantool.org
Thu Feb 27 23:22:29 MSK 2020


Hello,

On 29 янв 20:03, Alexander Turenko wrote:
> After #4736 regression fix (in fact it just reverts the new logic in
> small) it is possible again that a fiber's region may hold a memory for
> a while, but release it eventually. When the used memory exceeds 128 KiB
> threshold, fiber_gc() puts 'garbage' slabs back to slab_cache and
> subtracts them from region_used() metric. But until this point those
> slabs are accounted in region_used() and so in fiber.info() metrics.
> 
> This commit fixes flakiness of test cases of the following kind:
> 
>  | fiber.info()[fiber.self().id()].memory.used -- should be zero
>  | <...workload...>
>  | fiber.info()[fiber.self().id()].memory.used -- should be zero
> 
> The problem is that the first `<...>.memory.used` value may be non-zero.
> It depends of previous tests that were executed on this tarantool
> instance. It is resolved by restarting of a tarantool instance before
> such test cases to ensure that there are no 'garbage' slabs in a current
> fiber's region.
> 
> Note: This works only if a test case reserves only one slab at the
> moment: otherwise some memory may be hold after the case (and so a
> memory check after a workload will fail). However it seems that our
> cases are small enough to don't trigger this situation.
> 
> Call of region_free() would be enough, but we have no Lua API for it.
> 
> Fixes #4750.

I've checked your patch into 2.2, 2.3 and master as temporary
workaround.

--
Regards, Kirill Yukhin


More information about the Tarantool-patches mailing list