<div dir="ltr"><div dir="ltr">Hi David,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 21, 2020 at 12:24 PM David T. Lewis <<a href="mailto:lewis@mail.msen.com">lewis@mail.msen.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">About a week's advance notice on the list(s) seems pretty reasonable,<br>
possibly with an <ACK> from the CI folks to make sure. Switching VMs<br>
might be an inconvenience, but this is trunk after all and that's what<br>
it's for.<br>
<br>
I'm assuming that "broken" means the the user would see some failing<br>
tests due to the need for updated VM support, but the image would<br>
still run and be somewhat functional on the previous VM.<br>
<br>
This might also be an opportune time to add primitiveMultipleBytecodeSetsActive<br>
if you are so inclined. It requires trunk users to update their VM before<br>
the image format number can be updated, so it would be convenient if<br>
it could come along with the primitiveAt[Put] changes.<br></blockquote><div><br></div><div>We still have to discuss this.  I'm sorry, been too busy to do so.  I still don't understand the intent here.  The VM already provides a status bit in parameter 65 which says whether the VM supports multiple bytecode sets.  If the intent is to mark an image when multiple bytecode sets have been used then the primitive doesn't appear to me to do what's required.  I'll try and answer in the context of the thread soon, because I agree it's important.  Ideally there would be a pair of flags set somewhere that said if the current image had ever executed a method in either bytecode set.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
Dave<br>
<br>
<br>
On Tue, Jul 21, 2020 at 11:45:27AM +0200, Jakob Reschke wrote:<br>
> Hi Eliot,<br>
> <br>
> I only ever upgrade my VM if something breaks or if I am asked to. So<br>
> if you'd post a message with something like "Please upgrade your VM"<br>
> in the subject line to the list once you push the changes to Trunk,<br>
> that would be great for me!<br>
> <br>
> Concerning the time span: in an ideal world I would say "as soon as<br>
> the CI is green", but I have my doubts that this will be enough for<br>
> Squeak. Assuming that you use the new VM immediately and frequently<br>
> (or anyone else does), I think a day of work up to a week after the VM<br>
> changes is fine. Or as soon as you are confident about it, however you<br>
> measure that. ;-)<br>
> <br>
> Kind regards,<br>
> Jakob<br>
> <br>
> Am Di., 21. Juli 2020 um 02:55 Uhr schrieb Eliot Miranda<br>
> <<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>>:<br>
> ><br>
> > Hi All,<br>
> >     I've just found and fixed a VM bug in an as-yet-unpublished pair of primitives that replace FloatArrayPlugin>>primitiveAt[Put] and Float64ArrayPlugin>>primitiveAt[Put].  The new replacement, which allows two methods (at:[put:]) in FloatArray to relace four methods (Float32Array>>at:[put:] & Float64Array>>at:[put:]) are some 5 to 10 times faster than the plugin methods.  The issue is when to publish the corrections to trunk.<br>
> ><br>
> > Since the existing VM is broken I don't want to simply push to trunk and have people inconvenienced by a sudden emergence of failures in the FoatArrayTests.  However, I do want to push the corrections soon, because they're a substantial improvement.  So the question is how long should I wait?<br>
> ><br>
> > Is it OK if I push the fixes to FloatArray and subclasses in a week?  Do people using trunk keep an eye on the CI builds and upgrade, or would they appreciate a heads up?  If so, as soon as the AppVeyor builds succeed for CogVM source as per VMMaker.oscog-eem.2778, I'll let you know and ask that you upgrade your VM.<br>
> > _,,,^..^,,,_<br>
> > best, Eliot<br>
> ><br>
> <br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div></div>