[pve-devel] [PATCH] ceph : use jemalloc for build

[Fabian Grünbichler]

On Fri, Apr 28, 2017 at 02:32:07PM +0200, Alexandre DERUMIER wrote:
> >>this flag is only for rocksdb, and only for the windows build? 
> 
> I have tested it, and it's working fine. (using perf command, I was seeing jemalloc)

should have been more clear here - it DOES work, but not because of that
flag (the Cmake build output even explicitly states that it will ignore
it ;)). the Cmake build scripts use tcmalloc or jemalloc based on
whether you have tcmalloc or jemalloc headers installed (in that order),
and because of your added Build-Conflicts tcmalloc headers cannot be
installed.

ALLOCATOR explicitly sets the desired allocator, bypassing the
autodetection.

> 
> >>I propose "-DALLOCATOR=jemalloc" instead - what do you think? 
> 
> I think it should work, I will test it this weekend.
> 

thanks for confirming that it works as expected.

I did some quick fio benchmark on Friday, and did not see any real
performance improvements (but the memory usage grows by ~50% for the OSD
processes!). could you share your benchmarks/test setup?