Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

set number of openblas threads to 1 while precompiling #46792

Merged
merged 1 commit into from
Sep 20, 2022

Conversation

KristofferC
Copy link
Sponsor Member

With the increased number of openblas threads by defaults, there have been reports about various types of crashes when precompilation is done in parallel. Limit the number of openblas threads during precompilation to reduce resource usage slightly.

@KristofferC KristofferC added the backport 1.8 Change should be backported to release-1.8 label Sep 16, 2022
@KristofferC KristofferC mentioned this pull request Sep 16, 2022
28 tasks
base/loading.jl Outdated Show resolved Hide resolved
@KristofferC KristofferC force-pushed the kc/openblas_precompile branch 2 times, most recently from 54efe79 to 4cb4b61 Compare September 19, 2022 08:42
@staticfloat
Copy link
Sponsor Member

Do you think it’s worthwhile to force Julia to one thread as well?

@KristofferC
Copy link
Sponsor Member Author

Do you think it’s worthwhile to force Julia to one thread as well?

I don't actually know. And Julia is by default 1 thread. I'll leave it at this for now to get this in and we can think more about the julia thread case.

@KristofferC KristofferC merged commit b6d2434 into master Sep 20, 2022
@KristofferC KristofferC deleted the kc/openblas_precompile branch September 20, 2022 06:07
KristofferC added a commit that referenced this pull request Sep 20, 2022
@KristofferC KristofferC removed the backport 1.8 Change should be backported to release-1.8 label Sep 30, 2022
PallHaraldsson added a commit to PallHaraldsson/julia that referenced this pull request Sep 30, 2022
This seems like a sensible followup to JuliaLang#46792

It's untested, I just used the github editor... and I don't know the optimal value, this just seemed sensible.
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