bugGNU Octave - Bugs: bug #57222, `version -blas` no longer works...

 
 

bug #57222: `version -blas` no longer works with Debian libopenblas0-pthread libblas.so.3 library

Submitter:  Mike Miller <mtmiller>
Submitted:  Wed 13 Nov 2019 06:39:49 PM UTC
   
 
Category:  Octave Function Severity:  2 - Minor
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 5.1.0 Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 02 Jan 2020 06:16:16 PM UTC, comment #4: 
Mike Miller <mtmiller>
Group Member
Fri 29 Nov 2019 08:02:51 PM UTC, comment #3: 

@Mike: might as well push this before the next release.  It seems fine.

Rik <rik5>
Group administrator
Mon 18 Nov 2019 12:05:25 AM UTC, comment #2: 

The patch looks fine if you want to apply it.

Rik <rik5>
Group administrator
Fri 15 Nov 2019 01:28:21 AM UTC, comment #1: 

Attached a patch that works for me. It allows detection of OpenBLAS with a fallback on openblas_get_num_threads, and reports that the configuration is unknown.

Example, when Octave is linked to the full libopenblas library:


ans = OpenBLAS (config: OpenBLAS 0.3.6 NO_LAPACKE DYNAMIC_ARCH NO_AFFINITY Haswell MAX_THREADS=48)


And now, when Octave is linked to a minimal libblas library built from OpenBLAS sources:


ans = OpenBLAS (config: unknown)



(file #47873)

Mike Miller <mtmiller>
Group Member
Wed 13 Nov 2019 06:39:49 PM UTC, original submission:  

The Debian BLAS implementations have gone through a major reorg. The result is that the libblas.so.3 library file provided by the libopenblas0-pthread package no longer links with libopenblasp-r0.3.7.so. The symbol `openblas_get_config` is no longer included in that library, so even though it is actually OpenBLAS, Octave can't detect that it's OpenBLAS or get the config string.

Here's a list of OpenBLAS specific symbols that are still present in this new library

  • openblas_block_factor
  • openblas_fork_handler
  • openblas_get_num_procs
  • openblas_get_num_threads
  • openblas_goto_num_threads_env
  • openblas_num_threads_env
  • openblas_omp_num_threads_env
  • openblas_read_env
  • openblas_set_num_threads
  • openblas_thread_timeout
  • openblas_verbose
  • openblas_warning


Mike Miller <mtmiller>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #47873:  openblas-minimal.patch added by mtmiller (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-01-02 mtmiller StatusPatch Reviewed Fixed
        Open/ClosedOpen Closed
    2019-11-29 rik5 StatusPatch Submitted Patch Reviewed
    2019-11-15 mtmiller Attached File- Added openblas-minimal.patch, #47873
        StatusNone Patch Submitted
    2019-11-13 mtmiller Carbon-CopyRemoved 80942 -

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code