Opened 11 years ago

Closed 8 years ago

#9151 closed enhancement (duplicate)

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 Merged in:
Authors: Reviewers: Jean-Pierre Flori
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

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 (1)

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

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by kilian

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

comment:1 Changed 11 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 8 years ago by jpflori

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

Superseded by #10508.

comment:3 Changed 8 years ago by tscrim

  • Status changed from needs_review to positive_review

comment:4 Changed 8 years ago by jdemeyer

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