Opened 11 years ago
Closed 11 years ago
#10258 closed enhancement (fixed)
Block interrupts during malloc
Reported by: | jdemeyer | Owned by: | jdemeyer |
---|---|---|---|
Priority: | major | Milestone: | sage-4.7 |
Component: | c_lib | Keywords: | malloc sage_malloc |
Cc: | Merged in: | sage-4.7.alpha1 | |
Authors: | Jeroen Demeyer | Reviewers: | Volker Braun |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Attachments (1)
Change History (10)
comment:1 Changed 11 years ago by
- Description modified (diff)
- Owner changed from tba to jdemeyer
comment:2 Changed 11 years ago by
- Description modified (diff)
- Keywords c_lib removed
- Status changed from new to needs_work
comment:3 Changed 11 years ago by
- Milestone changed from sage-feature to sage-4.6.2
- Status changed from needs_work to needs_review
comment:4 Changed 11 years ago by
comment:5 follow-up: ↓ 6 Changed 11 years ago by
- Reviewers set to Volker Braun
- Status changed from needs_review to positive_review
comment:6 in reply to: ↑ 5 Changed 11 years ago by
Replying to vbraun:
Nice!
There are some examples in the documentation that use malloc, for example
doc/en/numerical_sage/ctypes_examples.rst
. We should convert these tosage_malloc
as well, probably in #10109.
You are right that some documentation uses malloc
. But that seems to be pure C code. So changing malloc
to sage_malloc
would imply linking against -Lcsage
and possibly adding some sig_on()
/sig_off()
machinery and possibly more complications...
Changed 11 years ago by
comment:7 Changed 11 years ago by
Rebased patch
comment:8 Changed 11 years ago by
- Milestone changed from sage-4.6.2 to sage-4.7
comment:9 Changed 11 years ago by
- Merged in set to sage-4.7.alpha1
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Nice!
There are some examples in the documentation that use malloc, for example
doc/en/numerical_sage/ctypes_examples.rst
. We should convert these tosage_malloc
as well, probably in #10109.