From: Douglas K. <do...@go...> - 2016-10-28 22:26:51
|
this may not be new. I can repro it after my changes but with the feature compiled out. The one thing I'm doing that you're surely (I'm guessing) not is to inject additional gc calls into the test. Clearly we have a long-standing *extremely* subtle bug that only pertains to parallel load. In light of that I don't think this is a release blocker. On Tue, Oct 25, 2016 at 7:18 AM, Christophe Rhodes <cs...@ca...> wrote: > Douglas Katzman <do...@go...> writes: > > > But if anybody sees that test fail, please speak up, and I will change > > the compact instance feature to be disabled by default until I can dig > > into it further. We of course still use the feature at Google, because > > concurrent loading of code is a non-use-case. > > For what it's worth, I just ran that test for 24 hours in a loop and saw > no failures here. (With #+immobile-space as well, in case that makes a > difference in the light of your recent commit). > > Cheers, > > Christophe > |