Re: [Help-smalltalk] GC Crash/Bug due recursion?

[Paolo Bonzini]

On 07/05/2011 08:32 AM, Paolo Bonzini wrote:
> On 07/04/2011 09:42 PM, Gwenael Casaccio wrote:
> > I've made some changes:
> >
> > 1) I keep _gst_mark_an_oop_internal and removed
> > _gst_add_an_oop_to_mark_queue for adding an OOP in the Queue. So like
> > the previous one, it has an OOP has argument and visit it.
> >
> > 2) in _gst_mark_an_oop_internal when visiting one OOP I keep the same
> > behavior with TAIL_MARK_OOP, and TAIL_MARK_OOPRANGE add OOPs in the
> > Queue.
> >
> > 3) _gst_mark_oop_range keeps the same behavior as the old one
> >
> > The patch is tested and produced with master (all tests are green)
>
> Thanks, these are the minimum changes required. I'll tweak it a bit to
> remove the final push/pop and commit it.

Here is what I committed.

Paolo

0001-replace-recursion-with-mark-stack.patch
Description: Text Data