Skip to content

Conversation

@Enjection
Copy link
Owner

Changelog entry

...

Changelog category

  • New feature
  • Experimental feature
  • Improvement
  • Performance improvement
  • Bugfix
  • Backward incompatible change
  • Documentation (changelog entry is not required)
  • Not for changelog (changelog entry is not required)

Additional information

...

Enjection pushed a commit that referenced this pull request Nov 22, 2024
Enjection added a commit that referenced this pull request Feb 20, 2025
Enjection pushed a commit that referenced this pull request Jun 17, 2025
commit_hash:b48f0169c21a41cdbc428f6cd9240e6b26963b6b
Enjection pushed a commit that referenced this pull request Jun 17, 2025
… macros, batch #2, v.2

commit_hash:9dd428cb39c1b588a1f06613f1a1a9140ac31591
Enjection pushed a commit that referenced this pull request Aug 27, 2025
Co-authored-by: Ivan Blinkov <[email protected]>
Co-authored-by: Andrey Fomichev <[email protected]>
Enjection pushed a commit that referenced this pull request Sep 8, 2025
Enjection pushed a commit that referenced this pull request Sep 16, 2025
Enjection pushed a commit that referenced this pull request Sep 30, 2025
```
WARNING: ThreadSanitizer: data race (pid=123179)
  Read of size 8 at 0x0000065aad98 by thread T14:
    #0 NSystemInfo::CachedNumberOfCpus() util/system/info.cpp:241:10 (taxi-uservices-userver-core-tests+0x4155bba)
    #1 engine::(anonymous namespace)::TaskProcessorThreadStartedHook() userver/core/src/engine/task/task_processor.cpp:100:15 (taxi-uservices-userver-core-tests+0x5306bc8)
    #2 engine::TaskProcessor::PrepareWorkerThread(unsigned long) userver/core/src/engine/task/task_processor.cpp:347:5 (taxi-uservices-userver-core-tests+0x5306bc8)
```

The change on a hot path is equivalent to the previously compiler generated x86 code (just a `load` instruction). On other platforms this could actually fix a data race.
commit_hash:acb1fe1b8f3075cc27b18aab4f1bd0b88292c270
Enjection pushed a commit that referenced this pull request Oct 6, 2025
Enjection pushed a commit that referenced this pull request Oct 21, 2025
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.

2 participants