Fix data race on lazy initialization of function pointers in SimSIMD dispatch by vitlibar · Pull Request #102132 · ClickHouse/ClickHouse · GitHub
Skip to content

Fix data race on lazy initialization of function pointers in SimSIMD dispatch#102132

Merged
alexey-milovidov merged 2 commits into
ClickHouse:masterfrom
vitlibar:fix-data-race-on-lazy-init-in-simsimd
Apr 10, 2026
Merged

Fix data race on lazy initialization of function pointers in SimSIMD dispatch#102132
alexey-milovidov merged 2 commits into
ClickHouse:masterfrom
vitlibar:fix-data-race-on-lazy-init-in-simsimd

Conversation

@vitlibar

@vitlibar vitlibar commented Apr 8, 2026 •
edited by robot-clickhouse
Loading

Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Fix data race on lazy initialization of function pointers in SimSIMD dispatch
PR to ClickHouse/SimSIMD: #26

Fixes #101968

Version info

  • Merged into: 26.4.1.785

@clickhouse-gh

clickhouse-gh Bot commented Apr 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added pr-not-for-changelog This PR should not be mentioned in the changelog submodule changed At least one submodule changed in this PR. labels Apr 8, 2026
@vitlibar vitlibar requested a review from Copilot April 8, 2026 21:25
Copilot AI reviewed Apr 8, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@alexey-milovidov

alexey-milovidov commented Apr 9, 2026

Copy link
Copy Markdown
Member

The MSan stress test failure (MemorySanitizer: use-of-uninitialized-value, STID 4179-5154 or 4148-3044) is a known pre-existing issue unrelated to this PR. Fix: #102158

@vitlibar vitlibar force-pushed the fix-data-race-on-lazy-init-in-simsimd branch from a8d90ca to 97a69a1 Compare April 9, 2026 09:46
@vitlibar vitlibar marked this pull request as ready for review April 9, 2026 09:46
@vitlibar

vitlibar commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

This PR correctly sets the submodule pointer to the HEAD commit of branch ClickHouse/v6.5.15 - thus in fact it enables three changes and not one:

@vitlibar vitlibar added the 🍃 green ci 🌿 Fixing flaky tests in CI label Apr 9, 2026
@vitlibar vitlibar mentioned this pull request Apr 9, 2026
Merged
@alexey-milovidov

alexey-milovidov commented Apr 10, 2026

Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov self-assigned this Apr 10, 2026
@alexey-milovidov alexey-milovidov merged commit 9501061 into ClickHouse:master Apr 10, 2026
163 of 164 checks passed
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label Apr 10, 2026
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 mentioned this pull request Jun 27, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@alexey-milovidov alexey-milovidov

Labels

🍃 green ci 🌿 Fixing flaky tests in CI pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo submodule changed At least one submodule changed in this PR.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ThreadSanitizer: data race (STID: 2410-3957)

4 participants

@vitlibar @alexey-milovidov @robot-ch-test-poll4