Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better system detection for ATLAS #14605

Closed
jpflori opened this issue May 17, 2013 · 33 comments
Closed

Better system detection for ATLAS #14605

jpflori opened this issue May 17, 2013 · 33 comments

Comments

@jpflori
Copy link

jpflori commented May 17, 2013

See lots of rants in #10508.

This fixes a bunch of problems with ATLAS detection system.
Adds support for (some) ARM hard float systems.
Adds archdefs for Raspberry Pi and Cygwin64.

Use tarball at:

Depends on #14699

CC: @nexttime

Component: packages: standard

Keywords: atlas, days57

Author: Jean-Pierre Flori

Branch/Commit: 3f1defb

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/14605

@jpflori

This comment has been minimized.

@jpflori
Copy link
Author

jpflori commented May 17, 2013

comment:2

Preliminary spkg, please torture it but nothing is commited or documented, I'll try to do that over the week end but as it may not happen here is some stuff to test.

In particular it should (not tested!) work:

  • for non-threaded libraries anywhere, especially on computers with a single threaded unique core,
  • threaded on debian/sparc,
  • threaded on debian/powerpc,
    whereas it used not to before.
    (I just remembered that "-t 0" is passed to configure in spkg-install so threads will always be disabled unless you change that.)

@vbraun
Copy link
Member

vbraun commented May 18, 2013

Author: Jean-Pierre Flori

@vbraun

This comment has been minimized.

@vbraun
Copy link
Member

vbraun commented May 18, 2013

comment:3

I fixed your url

Disabling threading unconditionally sucks big time on all modern hardware.

@jpflori
Copy link
Author

jpflori commented May 18, 2013

comment:4

Thanks!
I really uploaded that in a hurry.
And it's not intended to always disable threading all the time in the final version.
It is just what I was testing and forgot to comment again the "-t 0" line before uploading...

@jpflori
Copy link
Author

jpflori commented May 18, 2013

comment:5

Should be fixed now.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:6

Everything is commited now.
If p0 of #10508 is integrated upstream and tagged in between, this might need a slight rebase.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:7

Or we could just skip #10508...

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:8

nah, there are patches to the Sage library on #10508 so it's a bad idea to go too fast.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:9

And the new detect.patch did not get included in my latest spkg. Groumpf.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:10

Should be ok now.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:11

This still does not correctly detect cpu speed on debian sparc.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:12

Ok, got it, my hex reading functions are broken.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

comment:13

Should be fixed now.

@jpflori
Copy link
Author

jpflori commented May 21, 2013

Attachment: atlas-3.10.1.p1.diff.gz

Spkg diff, for review only.

@jpflori
Copy link
Author

jpflori commented Jun 6, 2013

Dependencies: #14605

@jpflori
Copy link
Author

jpflori commented Jun 6, 2013

Work Issues: rebase

@jdemeyer
Copy link

Changed dependencies from #14605 to #14699

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@jpflori
Copy link
Author

jpflori commented Nov 7, 2013

comment:18

Should be rebased on top of #14410.
Also note that the most important patch from here was already merged into our spkg by Volker at some point IIRC.

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@jpflori
Copy link
Author

jpflori commented Feb 10, 2014

New commits:

9d2673fBetter system detection for ATLAS.

@jpflori

This comment has been minimized.

@jpflori
Copy link
Author

jpflori commented Feb 10, 2014

Commit: 9d2673f

@jpflori
Copy link
Author

jpflori commented Feb 10, 2014

Changed keywords from atlas threads to atlas

@jpflori
Copy link
Author

jpflori commented Feb 10, 2014

Branch: u/jpflori/ticket/14605

@jpflori
Copy link
Author

jpflori commented Feb 10, 2014

Changed work issues from rebase to none

@jpflori jpflori changed the title Fix thread related ATLAS problems Better system detection for ATLAS Feb 10, 2014
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 20, 2014

Changed commit from 9d2673f to 3f1defb

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 20, 2014

Branch pushed to git repo; I updated commit sha1. New commits:

72e6977Merge remote-tracking branch 'trac/develop' into ticket/14605
3f1defbFix autotools optional spkg checksum spurious change.

@vbraun
Copy link
Member

vbraun commented Apr 7, 2014

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Apr 7, 2014

Changed keywords from atlas to atlas, days57

@vbraun
Copy link
Member

vbraun commented Apr 7, 2014

comment:23

lgtm

@vbraun
Copy link
Member

vbraun commented Apr 8, 2014

Changed branch from u/jpflori/ticket/14605 to 3f1defb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants