#14098: zn_poly-0.9.p9 fails at least one its tests on power7
---------------------------+------------------------------------------------
Reporter: fbissey | Owner: drkirkby
Type: defect | Status: new
Priority: major | Milestone: sage-5.7
Component: porting | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: | Merged in:
Dependencies: | Stopgaps:
---------------------------+------------------------------------------------
Comment (by dmharvey):
I am the author.... thanks Paul for drawing my attention to this.
I haven't looked at this code for years so it's almost as mysterious to me
as to everyone else here!
My guess is that the bug is in the test code rather than in the tuning
code. I suspect that the threshold is allowed to be SIZE_MAX, but that the
line
{{{
n = random_ulong (3 * ZNP_mpn_smp_kara_thresh) + 2;
}}}
should be replaced by e.g.
{{{
if (ZNP_mpn_smp_kara_thresh == SIZE_MAX)
n = random_ulong (100) + 2;
else
n = random_ulong (3 * ZNP_mpn_smp_kara_thresh) + 2;
}}}
It could also be a bug in the tuning code, but that would be much harder
to fix. If I remember correctly what this threshold means, it is very
surprising to me that its optimal value is SIZE_MAX on any real system.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14098#comment:10>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
