On Oct 28, 2007, at 1:29 AM, mabshoff wrote: > For 2.8.10.alpha1+Craig's fix and python compiled using "--without- > pymalloc" I get with the default cleanup level 1: > > ==6569== LEAK SUMMARY: > ==6569== definitely lost: 181,030 bytes in 2,926 blocks. > ==6569== possibly lost: 266,925 bytes in 747 blocks. > ==6569== still reachable: 29,184,388 bytes in 179,707 blocks. > ==6569== suppressed: 0 bytes in 0 blocks. > > With cleanup level 3 in general and cleanup level 1 just for > integer.pyx I get > > ==23818== LEAK SUMMARY: > ==23818== definitely lost: 181,117 bytes in 2,927 blocks. > ==23818== possibly lost: 266,838 bytes in 746 blocks. > ==23818== still reachable: 29,157,453 bytes in 179,383 blocks. > ==23818== suppressed: 0 bytes in 0 blocks. > > So my conclusion is that you are on the right way with the cleanup > code.
What is our goal (i.e. what is a clean python run?) It's a bit better but it seems like we've barely made a dent in it--are the dictionaries we were looking at decrefing earlier still around? (They could be holding onto a lot of stuff.) > Hopefully once we can compile integer.pyx with cleanup level 3 > and not crash Try the below patch--this will prevent the double-free from the global dummy integer. There might be a better fix, as this will make it so that this integer object is never freed. > the amount of still reachable memory should decrease > tremendously because integer.pyx should just about be referenced in > every extension we have. It doesn't matter how many things reference integer.pyx, only how many things are referenced by integer.pyx (which isn't too much, but may include such things as element.pyx). - Robert --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to sage-devel@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://sage.scipy.org/sage/ and http://modular.math.washington.edu/sage/ -~----------~----~----~----~------~----~------~--~---
no-collect-integer.patch
Description: Binary data
