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

Cortex A57 fixes and Lapack 3.6.0 #696

Merged

Conversation

ashwinyes
Copy link
Contributor

  • lapack-test fixes for Cortex A57
  • Update to lapack 3.50

xianyi added a commit that referenced this pull request Nov 23, 2015
@xianyi xianyi merged commit b4380ac into OpenMathLib:develop Nov 23, 2015
tkelman added a commit to tkelman/OpenBLAS that referenced this pull request Jan 18, 2016
partial revert of OpenMathLib@299cdcd
from OpenMathLib#696, was not explained why that was needed
@martin-frbg
Copy link
Collaborator

Do not see this on my various Linux/x86 systems either, suspect this is actually an ARM (or at most OS_ANDROID) thing, not Linux in general, cf. #597

@xianyi
Copy link
Collaborator

xianyi commented Jan 26, 2016

@martin-frbg , @jeromerobert

Please see #246

I think the problem is on big machine. For getrf, the current threshold GETRF_MEM_ALLOC_THRESHOLD is 80. When the number of cores is 80, the job_t array will be 80k on the stack.

@martin-frbg
Copy link
Collaborator

Hmm. Wonder if it could be made conditional on "high" number of cores then (I see that the code initializing blas_cpu_number comes immediately after the function, so would only need to be shuffled up a few lines it seems) ?

tkelman added a commit to tkelman/OpenBLAS that referenced this pull request Jan 30, 2016
partial revert of OpenMathLib@299cdcd
from OpenMathLib#696, was not explained why that was needed
xianyi added a commit that referenced this pull request Feb 24, 2016
I cannot reproduce SEGFAULT of lapack-test with default stack size
on ARM Linux.
xianyi added a commit that referenced this pull request Feb 24, 2016
I cannot reproduce SEGFAULT of lapack-test with default stack size
on ARM Linux.
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.

6 participants