Skip to content

Conversation

@snnn
Copy link
Member

@snnn snnn commented Dec 12, 2018

Shutdown threadpool at exit

@snnn snnn requested a review from a team as a code owner December 12, 2018 23:59
@RyanUnderhill
Copy link
Contributor

This is just for Linux?

@snnn
Copy link
Member Author

snnn commented Dec 13, 2018

This is just for Linux?

Yes. Windows doesn't need this. It is using the default thread pool which can't be shutdown. And this is the reason why your are seeing memory leaking, which I think it's just a false alarm.

souptc pushed a commit that referenced this pull request Dec 17, 2018
LRN cpu kernel impl.

Related work items: #161
@snnn snnn force-pushed the snnn/shutdown_thread_pool branch from 02b985f to 48b9805 Compare December 21, 2018 01:00
@snnn snnn closed this Jan 15, 2019
@snnn snnn deleted the snnn/shutdown_thread_pool branch January 15, 2019 20:46
TedThemistokleous pushed a commit to TedThemistokleous/onnxruntime that referenced this pull request Jul 18, 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.

3 participants