<br><br><div class="gmail_quote">On Wed, Feb 27, 2013 at 8:47 AM, Igor Stasenko <span dir="ltr"><<a href="mailto:siguctua@gmail.com" target="_blank">siguctua@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On 26 February 2013 19:20, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>> wrote:<br>
><br>
><br>
><br>
> On Tue, Feb 26, 2013 at 4:13 AM, Igor Stasenko <<a href="mailto:siguctua@gmail.com">siguctua@gmail.com</a>> wrote:<br>
>><br>
>> we should reafctor the code for changing the method's trailer (source<br>
>> pointer) to<br>
>> avoid using #become: at all.<br>
>> this won't deal with VM bug per se, but at least prevent it from triggering.<br>
><br>
><br>
> No, that makes little sense. Instead provide me with a reproducible case and I'll fix it quickly. I fixed the case form last week in one day. Become: should work, but I can't fix things if I can't reproduce the bugs. So please take the time to produce an image that crashes from start-up.<br>
><br>
<br>
</div>well, i didn't said that we should not look for a way how to ditch that issue.<br>
but we have little problem with it: it is very hard to reproduce.<br>
<br>
look from another angle: if things will get stable by not using become, then:<br>
a) we can release new version of pharo earlier<br>
b) it narrows down the field of search for bug<br>
<br>
but sure thing, i am with you, and even more..<br>
for things like that we should find a way to write a deterministic<br>
test case which fails / crashes on old VMs<br>
and works on fixed ones..<br>
because without such tests, it leaves a lot of uncertainty, like: did<br>
your latest fixes to become actually fixed anything?<br>
or they introduced yet another regression?<br></blockquote><div><br></div><div>Here's the way I try to reproduce bugs. Once I have a doit that reproduces the case I prefix it with Smalltalk saveAs: 'crash', copy the new crash.changes file to crash.changes.save, and run the doit.</div>
<div><br></div><div>Then I overwrite the changes file with the copy, cp crash.changes.save crash.changes to get a consistent starting point, and run crash.image.</div><div><br></div><div>For example, yesterday Bert Freudenberg found a VM crash loading a particular update fo the compiler (turns out not to be a VM bug). The updateListFor: method was modified to reject newer updates than the one that caused the crash. Then I executed</div>
<div><br></div><div>Smalltalk saveAs: 'crash'.</div><div>Utilities updateFromServer</div><div><br></div><div>and got a consistently reproducible case.</div><div><br></div><div>Now one can run the debug or assert VMs against the crash to debug. In my experience trying to understand complex VM bugs in cases where one tries to interact with the image using the mouse and keyboard is often fruitless.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
>><br>
>><br>
>> On 25 February 2013 23:26, Esteban Lorenzano <<a href="mailto:estebanlm@gmail.com">estebanlm@gmail.com</a>> wrote:<br>
>> > he is talking about some chrashes that happens fast, when you do become :)<br>
>> ><br>
>> > nothing about "fast become"<br>
>> ><br>
>> > On Feb 25, 2013, at 10:26 PM, Mariano Martinez Peck <<a href="mailto:marianopeck@gmail.com">marianopeck@gmail.com</a>><br>
>> > wrote:<br>
>> ><br>
>> > mmm what do you mean by "fast becomeForward:" ?<br>
>> ><br>
>> > On Mon, Feb 25, 2013 at 1:01 PM, Camillo Bruni <<a href="mailto:camillobruni@gmail.com">camillobruni@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> While setting up the new multi platform VM tests I saw that the tests fail<br>
>> >> extremely quickly<br>
>> >> when there are not PharoV10.sources available.<br>
>> >><br>
>> >> It is still our know #becomeForward: but on CompiledMethods, but I just<br>
>> >> want to share that.<br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> > --<br>
>> > Mariano<br>
>> > <a href="http://marianopeck.wordpress.com" target="_blank">http://marianopeck.wordpress.com</a><br>
>> ><br>
>> ><br>
>><br>
>><br>
>><br>
>> --<br>
>> Best regards,<br>
>> Igor Stasenko.<br>
>><br>
><br>
><br>
><br>
> --<br>
> best,<br>
> Eliot<br>
><br>
<br>
<br>
<br>
--<br>
Best regards,<br>
Igor Stasenko.<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div>