Skip to content

[Issue #1331] Remove jemalloc prefix in pixels-retina#1332

Open
AntiO2 wants to merge 1 commit into
pixelsdb:masterfrom
AntiO2:fix/pixels-retina-jemalloc-no-prefix
Open

[Issue #1331] Remove jemalloc prefix in pixels-retina#1332
AntiO2 wants to merge 1 commit into
pixelsdb:masterfrom
AntiO2:fix/pixels-retina-jemalloc-no-prefix

Conversation

@AntiO2
Copy link
Copy Markdown
Collaborator

@AntiO2 AntiO2 commented May 31, 2026

Summary

  • Remove --with-jemalloc-prefix=je_ from the bundled jemalloc build in cpp/pixels-retina.
  • Update JNI jemalloc memory metric calls from je_mallctl to mallctl so they match the unprefixed jemalloc build.
  • Keep the optional ENABLE_JEMALLOC=OFF build path compiling.

Validation

  • cmake --build cpp/pixels-retina/build --target pixels-retina -j 4: passed.
  • jemalloc configure output showed JEMALLOC_PREFIX empty.
  • nm -D cpp/pixels-retina/build/jemalloc_install/lib/libjemalloc.so | rg ' (malloc|free|calloc|realloc|mallctl)$| je_(malloc|free|mallctl)$': exported unprefixed malloc, free, calloc, realloc, and mallctl.
  • cmake -S cpp/pixels-retina -B /tmp/pixels-retina-no-jemalloc -DENABLE_JEMALLOC=OFF -DFETCHCONTENT_SOURCE_DIR_GOOGLETEST=/home/antio2/projects/pixels/cpp/pixels-retina/build/_deps/googletest-src: passed.
  • cmake --build /tmp/pixels-retina-no-jemalloc --target pixels-retina -j 4: passed.

Closes #1331

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[pixels-retina] Remove jemalloc prefix for LD_PRELOAD allocator interposition

1 participant