Bug 1144414 - Reenable special LAPACK functions
Summary: Reenable special LAPACK functions
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: openblas
Version: 20
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Susi Lehtola
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-09-19 10:37 UTC by Milan Bouchet-Valat
Modified: 2014-09-20 09:50 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-09-19 19:11:18 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Milan Bouchet-Valat 2014-09-19 10:37:22 UTC
Would it be possible to reenable the optimized LAPACK functions which are bundled in OpenBLAS? You apparently removed them when you found a bug, which has now been fixed since 0.2.7 [1][2].

I'm asking because of Julia user of my RPM package complained that Julia was slower than MATLAB, which was actually that the reference LAPACK was slower than Intel MKL.

Thanks!


1: https://bugzilla.redhat.com/show_bug.cgi?id=739398#c42
2: https://github.com/xianyi/OpenBLAS/issues/191

Comment 1 Susi Lehtola 2014-09-19 19:11:18 UTC
If you had bothered to look

* Tue Jul 23 2013 Susi Lehtola <jussilehtola> - 0.2.7-1
- Update to 0.2.7.
- Use OpenBLAS versions of LAPACK functions, as they seem to be
  working now.

* Mon Jul 08 2013 Susi Lehtola <jussilehtola> - 0.2.5-10
- Due to long standing bug, replace all OpenBLAS LAPACK functions with
  generic ones, so that package can finally be released in stable.

Comment 2 Milan Bouchet-Valat 2014-09-19 21:08:49 UTC
Ah sorry. Actually I had a look at git, but I didn't go so far in the past, as I hadn't noticed this bug fix was so old. And it look like the perfect explanation for my problem (the absence of threading). :-)

I've now realized I needed to link to libopenblasp if I want to enable threading. Would it be possible to get a very short explanation somewhere like /usr/share/doc/openblas/? Four lines would be enough:
- libopenblas.so provides the serial library
- libopenblasp.so provides a threaded library based on pthreads (built with USE_THREAD=1 USE_OPENMP=10)
- libopenblaso.so provides a threaded library based on OpenMP (built with USE_THREAD=1 USE_OPENMP=1)

libopenblas64.so, libopenblasp64.so and libopenblaso64.so offer the ILP64 versions of these libraries (built with INTERFACE64=1)

What I wrote is probably inaccurate, but I wanted to take my share of the work. Thanks!

Comment 3 Susi Lehtola 2014-09-19 22:08:00 UTC
That is already in the package descriptions.

Comment 4 Milan Bouchet-Valat 2014-09-20 09:50:42 UTC
Yeah, but for some reason I was looking for docs in /usr/share/doc, I hadn't even noticed there were several packages. I'd say it doesn't hurt to have information in two places.


Note You need to log in before you can comment on or make changes to this bug.