Ticket #9151 (closed enhancement: duplicate)

Opened 3 years ago

Last modified 6 weeks ago

build multithreaded version of ATLAS

Reported by: kilian Owned by: GeorgSWeber
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: build Keywords: ATLAS BLAS LAPACK multi-threading pthreads
Cc: jdemeyer Work issues:
Report Upstream: N/A Reviewers: Jean-Pierre Flori
Authors: Merged in:
Dependencies: Stopgaps:

Description

Hi,

I noted that multi-threading in ATLAS is switched off by default (switch -t 0). I was wondering if it wouldn't be better to make a multi-threaded build the default, since it results in big performance increases on most modern computers.

I attached a patch that enables threading. It works fine for me on a multi-core Linux machine. I tried to make it work for single-core machines (for which atlas might turn of threading automatically) by testing for the presence of the threaded version of the BLAS library before running the make command that builds the shared atlas library. However I haven't tested it on a single-core machine.

Kilian

Attachments

atlas_pthread.patch Download (1.0 KB) - added by kilian 3 years ago.
enables multi-threaded build in atlas-3.8.3.p12.spkg

Change History

Changed 3 years ago by kilian

enables multi-threaded build in atlas-3.8.3.p12.spkg

comment:1 Changed 3 years ago by kilian

Hi,

It would be great if this patch would make it into the next release. I tested it on Linux 32bit and 64bit, single and multi core, Intel and AMD. On the 8 core machine, it increased the speed of e.g. a matrix dot product about 4-fold, and it didn't break the single-threaded build on the one-core machine.

Kilian

comment:2 Changed 7 weeks ago by jpflori

  • Cc jdemeyer added
  • Reviewers set to Jean-Pierre Flori
  • Status changed from new to needs_review
  • Milestone changed from sage-feature to sage-duplicate/invalid/wontfix

Superseded by #10508.

comment:3 Changed 6 weeks ago by tscrim

  • Status changed from needs_review to positive_review

comment:4 Changed 6 weeks ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to duplicate
Note: See TracTickets for help on using tickets.