Skip to content

Conversation

@sunxiaoxia2022
Copy link
Collaborator

Description

Performance drop when setting parallel to TBB STATIC partitioner through by thread pool.

Fixes # (github issue)

Checklist

General

  • Do all unit and benchdnn tests (make test and make test_benchdnn_*) pass locally for each commit?
  • Have you formatted the code using clang-format?

Performance improvements

  • Have you submitted performance data that demonstrates performance improvements?

New features

  • Have you published an RFC for the new feature?
  • Was the RFC approved?
  • Have you added relevant tests?

Bug fixes

  • Have you included information on how to reproduce the issue (either in a github issue or in this PR)?
  • Have you added relevant regression tests?

RFC PR

  • Does RFC document follow the template?
  • Have you added a link to the rendered document?

@sunxiaoxia2022 sunxiaoxia2022 changed the title Performance drop when setting parallel to TBB STATIC partitioner through by thread pool Fix performance drop when setting parallel to TBB STATIC partitioner through by thread pool Sep 22, 2025
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it does make sense to simply remove this code, rather than keep it commented.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, updated.

@maxnick
Copy link
Collaborator

maxnick commented Sep 23, 2025

Please squash all the commits into one and rename it according to the fork naming convention, for example:
"[FORK][FIX] Fix performance issues caused by using the thread pool"

@sunxiaoxia2022 sunxiaoxia2022 changed the title Fix performance drop when setting parallel to TBB STATIC partitioner through by thread pool [FORK][FIX] Fix performance issues caused by using the thread pool Sep 24, 2025
@sunxiaoxia2022 sunxiaoxia2022 force-pushed the xiaoxia/tbb_thread_pool branch 5 times, most recently from 578e2ce to ff01374 Compare November 6, 2025 06:36
@sunxiaoxia2022 sunxiaoxia2022 force-pushed the xiaoxia/tbb_thread_pool branch from ff01374 to a66743c Compare November 6, 2025 06:43
github-merge-queue bot pushed a commit to openvinotoolkit/openvino that referenced this pull request Nov 6, 2025
…er by thread pool (#30167)

### Details:
- *Setting parallel to TBB STATIC partitioner or TBB AUTO partitioner
through by thread pool*
 - oneDNN PR: [openvinotoolkit/oneDNN#291]

### Tickets:
 - *CVS-165229*

---------

Co-authored-by: Shen, Wanglei <[email protected]>
Co-authored-by: Alina Kladieva <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants