Skip to content

Suppress OpenMP leak #21820

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Jun 27, 2025
Merged

Suppress OpenMP leak #21820

merged 2 commits into from
Jun 27, 2025

Conversation

jbajic
Copy link
Contributor

@jbajic jbajic commented Jun 24, 2025 •
edited
Loading

Scope & Purpose

Solves the leak coming from OpenMP:

Direct leak of 2539 byte(s) in 2 object(s) allocated from:
    #0 0x55827d4e7e53 in malloc (/oskar/work/ArangoDB/build/bin/arangod+0x6067e53) (BuildId: eb10a54c2ea075a76457974b7c1306544653cc6d)
    #1 0x55828aab2fdb in ___kmp_allocate_align(unsigned long, unsigned long, char const*, int) /llvm-toolchain-19-19.1.7~++20250114103332+cd708029e0b2/openmp/runtime/src/kmp_alloc.cpp:1947:25

Direct leak of 48 byte(s) in 1 object(s) allocated from:
    #0 0x55827d4e803d in calloc (/oskar/work/ArangoDB/build/bin/arangod+0x606803d) (BuildId: eb10a54c2ea075a76457974b7c1306544653cc6d)
    #1 0x55828aab7005 in __kmp_init_nesting_mode() /llvm-toolchain-19-19.1.7~++20250114103332+cd708029e0b2/openmp/runtime/src/kmp_runtime.cpp:9321:36
    #2 0x55828aab7005 in __kmp_do_serial_initialize() /llvm-toolchain-19-19.1.7~++20250114103332+cd708029e0b2/openmp/runtime/src/kmp_runtime.cpp:7274:3
    #3 0x55828884084f in faiss::knn_L2sqr(float const*, float const*, unsigned long, unsigned long, unsigned long, faiss::HeapArray<faiss::CMax<float, long>>*, float const*, faiss::IDSelector const*) /work/ArangoDB/3rdParty/faiss/faiss/utils/distances.cpp:743
    #4 0x558287d8f5d0 in faiss::IndexFlat::search(long, float const*, long, float*, long*, faiss::SearchParameters const*) const /work/ArangoDB/3rdParty/faiss/faiss/IndexFlat.cpp:43
    #5 0x558287d8baa6 in faiss::Index::assign(long, float const*, long*, long) const /work/ArangoDB/3rdParty/faiss/faiss/Index.cpp:38
    #6 0x558282ba69d4 in operator() /work/ArangoDB/arangod/RocksDBEngine/RocksDBVectorIndex.cpp:576
    #7 0x558282ba69d4 in operator()<(lambda at /work/ArangoDB/arangod/RocksDBEngine/RocksDBVectorIndex.cpp:568:29)> /work/ArangoDB/arangod/RocksDBEngine/RocksDBVectorIndex.cpp:549
  • 💩 Bugfix
  • 🍕 New feature
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification

Checklist

  • Tests
    • Regression tests
    • C++ Unit tests
    • integration tests
    • resilience tests
  • 📖 CHANGELOG entry made
  • 📚 documentation written (release notes, API changes, ...)
  • Backports
    • Backport for 3.12.0: (Please link PR)
    • Backport for 3.11: (Please link PR)
    • Backport for 3.10: (Please link PR)

Related Information

(Please reference tickets / specification / other PRs etc)

  • Docs PR:
  • Enterprise PR:
  • GitHub issue / Jira ticket:
  • Design document:

@jbajic jbajic self-assigned this Jun 24, 2025
@cla-bot cla-bot bot added the cla-signed label Jun 24, 2025
@jbajic jbajic marked this pull request as ready for review June 25, 2025 07:06
@neunhoef neunhoef merged commit 9303784 into devel Jun 27, 2025
7 checks passed
@neunhoef neunhoef deleted the bug-fix/suppress-openmp-leak branch June 27, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dothebart dothebart dothebart approved these changes

Assignees

@jbajic jbajic

Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jbajic @dothebart @neunhoef