Skip to content

Refine threading control options and move inter op thread pool to session state.#1841

Merged
pranavsharma merged 35 commits into
masterfrom
threading_control
Sep 19, 2019
Merged

Refine threading control options and move inter op thread pool to session state.#1841
pranavsharma merged 35 commits into
masterfrom
threading_control

Conversation

@pranavsharma
Copy link
Copy Markdown
Contributor

@pranavsharma pranavsharma commented Sep 14, 2019

Description: Refine threading control options and move inter op thread pool to session state.
Added thread_utils.h/cc to centralize the decision around the thread pool size under various conditions.

Motivation and Context
Currently the thread pool size of the parallel executor is hardcoded to 32 for some reason. This PR makes the options to configure the thread pool sizes clearer.

Pranav Sharma added 26 commits July 22, 2019 11:09
@pranavsharma pranavsharma requested a review from a team as a code owner September 14, 2019 02:32
Comment thread include/onnxruntime/core/session/onnxruntime_c_api.h
Comment thread csharp/src/Microsoft.ML.OnnxRuntime/SessionOptions.cs
@pranavsharma pranavsharma changed the title Add inter_op_thread_pool_size and rename session_thread_pool_size to intra_op_thread_pool_size. WIP: Add inter_op_thread_pool_size and rename session_thread_pool_size to intra_op_thread_pool_size. Sep 18, 2019
Comment thread include/onnxruntime/core/session/onnxruntime_c_api.h
@pranavsharma pranavsharma changed the title WIP: Add inter_op_thread_pool_size and rename session_thread_pool_size to intra_op_thread_pool_size. Refine threading control options and move inter op thread pool to session state. Sep 18, 2019
Comment thread onnxruntime/core/util/thread_utils.cc
Comment thread onnxruntime/test/perftest/ort_test_session.cc
@pranavsharma pranavsharma merged commit a9ce941 into master Sep 19, 2019
@snnn snnn deleted the threading_control branch September 19, 2019 05:42
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.

3 participants