Opened 9 years ago

Closed 8 years ago

#14335 closed enhancement (fixed)

update M4RI to version 20130416

Reported by: malb Owned by: tbd
Priority: minor Milestone: sage-5.10
Component: packages: standard Keywords:
Cc: Merged in: sage-5.10.beta1
Authors: Martin Albrecht Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by malb)

20130416 is mainly maintenance release of M4RI improving performance for small matrices.

Release notes are here: https://bitbucket.org/malb/m4ri/wiki/M4RI-20130416

Install http://boxen.math.washington.edu/home/malb/spkgs/libm4ri-20130416.spkg

Change History (8)

comment:1 Changed 9 years ago by malb

  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 9 years ago by malb

FWIW, I just built Sage 5.8 from scratch with the SPKG linked above on bsd.math.washington.edu.

comment:3 Changed 9 years ago by Snark

I successfully compiled the spkg with sage 5.8 on my arm box.

comment:4 Changed 9 years ago by Snark

I looked at libm4rie's compilation and found there where many warnings ; I checked libm4ri's build log, and found that many, many files trigger the following warnings:

In file included from ./m4ri/mzd.h:45:0,
                 from m4ri/brilliantrussian.h:41,
                 from m4ri/brilliantrussian.c:28:
./m4ri/misc.h:577:0: warning: "__M4RI_CPU_L3_CACHE" redefined [enabled by default]
In file included from ./m4ri/mzd.h:35:0,
                 from m4ri/brilliantrussian.h:41,
                 from m4ri/brilliantrussian.c:28:
./m4ri/m4ri_config.h:11:0: note: this is the location of the previous definition
In file included from ./m4ri/mzd.h:45:0,
                 from m4ri/brilliantrussian.h:41,
                 from m4ri/brilliantrussian.c:28:
./m4ri/misc.h:586:0: warning: "__M4RI_CPU_L2_CACHE" redefined [enabled by default]
In file included from ./m4ri/mzd.h:35:0,
                 from m4ri/brilliantrussian.h:41,
                 from m4ri/brilliantrussian.c:28:
./m4ri/m4ri_config.h:10:0: note: this is the location of the previous definitionIn file included from ./m4ri/mzd.h:45:0,
                 from m4ri/brilliantrussian.h:41,
                 from m4ri/brilliantrussian.c:28:
./m4ri/misc.h:595:0: warning: "__M4RI_CPU_L1_CACHE" redefined [enabled by default]
In file included from ./m4ri/mzd.h:35:0,
                 from m4ri/brilliantrussian.h:41,
                 from m4ri/brilliantrussian.c:28:
./m4ri/m4ri_config.h:9:0: note: this is the location of the previous definition

It doesn't harm but obfuscates the logs and perhaps means there's some header file missing guards.

comment:5 Changed 9 years ago by malb

Those warnings are expected. It means that L1-L3 were not detected successfully and are replaced by default values. So it's not critical. It might make sense to investigate how to detect L1/L2 on your kind of boxes (for future releases) and we should also print a less scary warning once.

comment:6 Changed 9 years ago by vbraun

  • Authors set to Martin Albrecht
  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

Looks good to me.

comment:7 Changed 9 years ago by jdemeyer

  • Milestone changed from sage-5.9 to sage-5.10

comment:8 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.10.beta1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.