Link jemalloc pool and test with libc explicitly to avoid recursive calls to jemalloc (tmp fix)

[igchor]

This forces the linker to link with libc before linking with jemalloc. Because of that, jemalloc will not override symbols for malloc/free.

Without this patch, we would call malloc from within the arena_extent_alloc (jemalloc hook) that would be intercepted by jemalloc.

This fixes the issue with debug jemalloc when running tests with tracking enabled:

<jemalloc>: Should own 0 locks of rank >= 14: extent_grow(17)
Aborted (core dumped)

[bratpiorka]

Interesting, I thought in this case the linking would fail because of multiple definitions of the same symbols. Anyway - you added a TODO in the code - what would be the final solution?

[vinser52]

I would extend TODO you added with the detailed explanations like you posted for this PR:

This forces the linker to link with libc before linking with jemalloc. Because of that, jemalloc will not override symbols for malloc/free.

Without this patch, we would call malloc from within the arena_extent_alloc (jemalloc hook) that would be intercepted by jemalloc.

This fixes the issue with debug jemalloc when running tests with tracking enabled:

: Should own 0 locks of rank >= 14: extent_grow(17)
Aborted (core dumped)

[ldorau]

@igchor Please add a detailed comment to the code like @vinser52 suggested above

[igchor]

@igchor Please add a detailed comment to the code like @vinser52 suggested above

Done. I've created an issue and added link to that issue in the comments.