Reduce default logger usage by skottmckay · Pull Request #23030 · microsoft/onnxruntime

@skottmckay

…inimizing the usage of the default logger is important for these scenarios.

Wire through the session logger to as many places as possible. The EP logger can also be used once the session is created (can't be used during EP construction/kernel registration but can be used in GetCapability and Compile).

github-advanced-security[bot]

yuslepukhin

yuslepukhin

@skottmckay

@skottmckay

yuslepukhin

@skottmckay skottmckay deleted the skottmckay/ReduceDefaultLoggerUsage_PR branch

December 10, 2024 01:54

ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request

Dec 11, 2024
### Description
<!-- Describe your changes. -->
We have use cases where multiple sessions are created concurrently.
Minimizing the usage of the default logger is important for these
scenarios.

Wire through the session logger to as many places as possible. The EP
logger can also be used once the session is created (can't be used
during EP construction/kernel registration but can be used in
GetCapability and Compile).

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Improve logging when there are concurrent sessions.

ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request

Dec 11, 2024
### Description
<!-- Describe your changes. -->
We have use cases where multiple sessions are created concurrently.
Minimizing the usage of the default logger is important for these
scenarios.

Wire through the session logger to as many places as possible. The EP
logger can also be used once the session is created (can't be used
during EP construction/kernel registration but can be used in
GetCapability and Compile).

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Improve logging when there are concurrent sessions.

ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request

Dec 11, 2024
### Description
<!-- Describe your changes. -->
We have use cases where multiple sessions are created concurrently.
Minimizing the usage of the default logger is important for these
scenarios.

Wire through the session logger to as many places as possible. The EP
logger can also be used once the session is created (can't be used
during EP construction/kernel registration but can be used in
GetCapability and Compile).

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Improve logging when there are concurrent sessions.

tarekziade pushed a commit to tarekziade/onnxruntime that referenced this pull request

Jan 10, 2025
### Description
<!-- Describe your changes. -->
We have use cases where multiple sessions are created concurrently.
Minimizing the usage of the default logger is important for these
scenarios.

Wire through the session logger to as many places as possible. The EP
logger can also be used once the session is created (can't be used
during EP construction/kernel registration but can be used in
GetCapability and Compile).

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Improve logging when there are concurrent sessions.

tianleiwu pushed a commit that referenced this pull request

May 12, 2025
### Description
Fix the build error in VSINPU EP which caused by  #23030


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

Signed-off-by: Kee <xuke537@hotmail.com>

quic-ankus pushed a commit to CodeLinaro/onnxruntime that referenced this pull request

Nov 25, 2025
### Description
Fix the build error in VSINPU EP which caused by  microsoft#23030


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

Signed-off-by: Kee <xuke537@hotmail.com>