Skip to content

Serialize accesses to parallelized syrk functions from multiple calle…#5619

Merged
martin-frbg merged 1 commit intoOpenMathLib:developfrom
lujiaweics:fix/serialize_parallelized_syrk_function_callers
Jan 20, 2026
Merged

Serialize accesses to parallelized syrk functions from multiple calle…#5619
martin-frbg merged 1 commit intoOpenMathLib:developfrom
lujiaweics:fix/serialize_parallelized_syrk_function_callers

Conversation

@lujiaweics
Copy link
Contributor

…rs, like it was already done for GEMM in level3_thread.c and GEMM3M in level3_gemm3m_thread.c

…rs, like it was already done for GEMM in level3_thread.c and GEMM3M in level3_gemm3m_thread.c
@martin-frbg
Copy link
Collaborator

Thanks, not sure why that got missed. I worry that something similar might be needed for getrf/potrf as well

@martin-frbg martin-frbg merged commit b6aff47 into OpenMathLib:develop Jan 20, 2026
128 of 133 checks passed
@lujiaweics
Copy link
Contributor Author

Thanks for pointing this out! I'll prioritize checking getrf and potrf soon and follow up with any necessary changes.

@lujiaweics
Copy link
Contributor Author

@martin-frbg Hi Martin, I've checked the code and agree that getrf/potrf are missing the concurrency-limiting code as well. Could you confirm if I should create a new PR to fix this bug? Thanks!

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