freenode/#clasp - IRC Chatlog
Search
18:59:54
drmeister
So ./waf tests with the MPS version - if it passes all of the tests then finalizers are working?
19:01:43
drmeister
In MPS we get finalization messages from the MPS and I look up the object in a weak-key hash table.
19:04:12
drmeister
I need to have the testing code compiled if I execute the code in "sys:regression-tests;finalizers.lisp"
19:10:00
drmeister
kpoeck: When you mentioned "the size of a weak hash table should be a multiple of 16?" What was that in reference to?
19:10:21
drmeister
Because I'm still seeing a problem with weak hash tables and it might tie into their size.
19:10:42
drmeister
I'm getting this problem... ../../include/clasp/mps/code/buffer.c:575: MPS ASSERTION FAILED: SizeIsAligned(size, BufferPool(buffer)->alignment)
19:11:34
drmeister
Yeah - but there is additional stuff in the object that might mess up the alignment.
19:12:49
drmeister
https://github.com/clasp-developers/clasp/blob/master/include/clasp/gctools/gcalloc.h#L302
19:34:12
drmeister
I'm putting in this now in every case: (dotimes (i 10) (gctools:garbage-collect))
19:35:29
drmeister
I don't know if there are full or incremental with either GC. I just know what's in (gctools:garbage-collect)
23:38:47
drmeister
::notify kpoeck CONS cell finalizers should work - but they don't. Everything is being setup properly. The only thing I can think of is that the new 16-byte alignment for CONS cells is shifting the CONS cell address by 8 bytes and the finalizer code doesn't like that.
0:20:53
drmeister
::notify kpoeck MPS finalizers appear to work fine. Boehm has a problem with CONS finalizers and it may be an issue with their memory alignment and how they are allocated. This might be fixed when I switch back to 8-byte alignment for CONS cells. (Yes I'm thinking of doing that to get back one word/CONS cell).