On Feb 21, 2009, at 7:26 AM, Dag Sverre Seljebotn wrote:

> Robert Bradshaw wrote:
>> On Feb 20, 2009, at 10:56 PM, Michael Abshoff wrote:
>>
>>> Robert Bradshaw wrote:
>>>
>>> Hi Robert,
>>>
>>>> I found and fixed two memory leaks.
>>> Got a patch for me? I could still get that one into 3.3 :)
>>
>> Memory leaks in Cython, not in Sage. (Actually, probably not
>> noticeable ones, one was leaking an exception, and the other was
>> "leaking" the values True and False in most cases.
>>
>>>> This refcounting nanny sure is
>>>> nice. Sage now starts up without any warnings, but upon exiting I
>>>> still get
>>>>
>>>>> Exiting SAGE (CPU time 0m0.07s, Wall time 2m13.31s).
>>>>> Exception exceptions.AssertionError in  ignored
>>>>> Exception exceptions.AssertionError in  ignored
>>>>> Exception exceptions.AssertionError in /Users/robert/sage/sage-
>>>>> cython/
>>>>> local/bin/sage-sage: line 587: 16443 Bus error               sage-
>>>>> ipython "$@" -i
>>>> Next up is running the test suite.
>>> Cool, looking forward to all the fixes coming out of that.
>>
>> I'm going to run it overnight. Given that Sage doesn't exit cleanly
>> (not sure what's up with that), I expect everything to fail, but
>> hopefully it'll still be useful.
>
> Don't work too hard on it yet -- there's some bugs in the refnanny now
> which I should fix, and which may produce these kinds of errors.

I'm running the tests, but probably won't be able to work on it much  
today anyways. So far I have a 7MB log (and it's less than half way  
done).

> One suspect is here:
>
> http://hg.cython.org/cython-devel/rev/4e19b8a61272
>
> There are some more as well which could lead to similar symptoms which
> I'll have to wait with pushing -- ironically, the fix itself seems to
> produce a temp handling refcount bug on the refnanny.c code itself,  
> so I
> must fix that simultaneously.

Cool, let us know when you smash this one.

- Robert


_______________________________________________
Cython-dev mailing list
[email protected]
http://codespeak.net/mailman/listinfo/cython-dev

Reply via email to