#11752: ecl.pyx should not touch SIGPWR neither SIGXCPU when initializing ecl
--------------------------+-------------------------------------------------
Reporter: pcpa | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-4.7.2
Component: interfaces | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
--------------------------+-------------------------------------------------
Comment(by nbruin):
Replying to [comment:8 pcpa]:
> Thanks! This indeed appears to correct the problem.
Wonderful. The {{{ecl_set_option(ECL_OPT_SIGNAL_HANDLING_THREAD, 0)}}} is
completely uncontroversial. There is a remote possibility that specifying
this option might be enough to dissuade Boehm GC from using SIGPWR etc.
Given that this fixes something that shouldn't occur in standard Sage
configurations, such a minimal fix would be preferable. If that's enough,
I can give your amended patch a positive review right away.
If it doesn't, I think SIGPWR and SIGXCPU are such exotic signals (and
Sage doesn't do anything special with them anyway) that I think it should
be acceptable to relinquish them to ECLs handlers. You could add them to
{{{cdef extern int SIGINT, SIGBUS, SIGSEGV}}} (line 48 in ecl.pyx) to
refer to them by name.
If they do need to be given to ecl, they probably need treatment in
{{{ecl_sig_on}}} and {{{ecl_sig_off}}} as well. That doesn't make it
thread-safe yet (imagine 2 sage threads, one of which executing ECL code
that triggers a GC. What's the other thread going to do with the SIGPWR?).
Perhaps jdemeyer can give some advice what to do in that case.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11752#comment:9>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
