#15045: ATLAS: multiple definition of `ATL_SetAtomicCount'
-------------------------------------------------+-------------------------
Reporter: jdemeyer | Owner:
Type: defect | Status:
Priority: blocker | needs_work
Component: packages: standard | Milestone: sage-5.12
Keywords: | Resolution:
Authors: Volker Braun | Merged in:
Report Upstream: Reported upstream. No | Reviewers:
feedback yet. | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
-------------------------------------------------+-------------------------
Comment (by nbruin):
Replying to [comment:23 nbruin]:
> `ATLAS/tune/threads/tune_count.c:242`
> {{{
> if (tmut < tldec*1.02)
> {
> printf("\nNO REAL ADVANTAGE TO ASSEMBLY, FORCING USE OF
MUTEX\n");
> ATL_assert(!system("make iForceUseMutex"));
> }
> }}}
So would it be a plan to
- check that this check is the culprit of (at least some) failure by
patching it to be `tmut < tldec*1000` and verifying that on relevant
hardware ALTAS now reliably fails to build
- patch atlas to NOT run `make iForceUseMutex` at this point and ship
with that. Perhaps patch it as:
{{{
printf("\nASSEMBLY/MUTEX ratio is %d, but we'll stick with assembly
anyway\n",tldec/tmut);
}}}
so that our logfiles at least tells us what the performance penalty is
(well, there's none because ATLAS doesn't build dynamic libs without).
--
Ticket URL: <http://trac.sagemath.org/ticket/15045#comment:27>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
