OpenBLAS icon indicating copy to clipboard operation
OpenBLAS copied to clipboard

[WIP] Add more documentation regarding parallelism

Open TiborGY opened this issue 6 years ago • 7 comments

TiborGY avatar Oct 21 '18 11:10 TiborGY

Aint it better to improve Makefile.rule where the mystery options are set instead?

brada4 avatar Oct 21 '18 13:10 brada4

Aint it better to improve Makefile.rule where the mystery options are set instead?

Maybe, but I am working on a more extensive piece of documentation on threading. Too long to be reasonably included in Makefile.rule.

TiborGY avatar Oct 21 '18 14:10 TiborGY

Are you sure it is necessary

  • there is no complete OMP hierachy support, but will be hopefully
  • new code trigerred by USE_TLS=1 in Makefile.rule will make most MM parallel without extra locking

brada4 avatar Oct 21 '18 17:10 brada4

No offence meant, but I wonder if it would be more appropriate if you developed this piece of documentation in the wiki of your fork first ? (BTW OpenBLAS is continuing the original GotoBLAS work of K.Goto, a statement like "the project aims to develop a BLAS package" makes it look as if it was incomplete or its API unstable)

martin-frbg avatar Oct 21 '18 18:10 martin-frbg

No offence meant, but I wonder if it would be more appropriate if you developed this piece of documentation in the wiki of your fork first ? (BTW OpenBLAS is continuing the original GotoBLAS work of K.Goto, a statement like "the project aims to develop a BLAS package" makes it look as if it was incomplete or its API unstable)

Oh OK, the fact that even the wiki is forked on GitHub was never in my thoughts, good idea. The language used may need adjustment/rephrasing. All suggestions are welcome.

TiborGY avatar Oct 21 '18 18:10 TiborGY

Given the currently open threading issues, I will probably wait with doing this until things have settled down, one way or another.

TiborGY avatar Nov 25 '18 15:11 TiborGY

Check https://github.com/xianyi/OpenBLAS/issues/1882#issuecomment-441563356 with some input for what happens in case where you try to mix OMP implementations.

brada4 avatar Nov 26 '18 17:11 brada4