So nicolas should I harvest the find related bug fixes?
On Apr 20, 2009, at 10:34 PM, Nicolas Cellier wrote: > Wait, I think the problem is not at all caused by Concurrency fixes... > > I strongly suspect introduction of SharedQueue>>#findFirst: > > The test hang randomly in: > > InputEventSensor>>peekKeyboardEvent > "Allows for use of old Sensor protocol to get at the keyboard, > as when running kbdTest or the InterpreterSimulator in Morphic" > > ^eventQueue findFirst: [:buf | self isKbdEvent: buf] > > Old code was: > > peekKeyboardEvent > "Return the next keyboard char event from the receiver or nil if none > available" > ^eventQueue nextOrNilSuchThat: > [:buf | > buf first = EventTypeKeyboard and: > [(buf fourth) = EventKeyChar]] > > But I noticed SharedQueue>>findFirst: and > SharedQueue>>nextOrNilSuchThat: are not EXACTLY the same... > findFirst: FAILS TO INCREMENT readPos... > > > Nicolas > > > 2009/4/20 Stéphane Ducasse <[email protected]>: >> I know that john has a vm with traces but I do not know how to do >> what >> you need. >> Do you think that it can help if I integrate the next batch? >> Nicolas I can also remove from the stream the item so that you can >> take time to look at it. >> you just need to load the version 836 of the scriptLoader (from the >> pharo repository). >> and manually launch the loading of the last update. >> >> ScriptLoader new update10287 >> >> Stef >> >> On Apr 20, 2009, at 10:15 PM, Nicolas Cellier wrote: >> >>> Do you know how to dump Smalltalk process stacks from within the VM >>> debugger (I mean gdb) ? >>> >>> 2009/4/20 Nicolas Cellier <[email protected]>: >>>> But the tests just blocked after 2256 run, 2162 passes, etc... >>>> and no response to ALT+. :( >>>> >>>> I will have a quick look... >>>> >>>> 2009/4/20 Stéphane Ducasse <[email protected]>: >>>>> Ok now I understand why I only got 14 and not 16 tests green. >>>>> >>>>> >>>>> On Apr 20, 2009, at 9:54 PM, Nicolas Cellier wrote: >>>>> >>>>>> Yes, it seems OK >>>>>> The tests are running on my linux versions now... not yet frozen >>>>>> The two additionnal DelayTest from Andreas which were in >>>>>> KernelTests-nice.95.mcz are not yet loaded but will be with >>>>>> KernelTests-nice.96.mcz >>>>>> >>>>>> Nicolas >>>>>> >>>>>> 2009/4/20 Stéphane Ducasse <[email protected]>: >>>>>>> it is not a problem. >>>>>>> I just want to make sure. Some changes are easy to undo some >>>>>>> not. >>>>>>> So I will integrate the second batch. >>>>>>> Did you have a look at the integration I just did. >>>>>>> >>>>>>> Stef >>>>>>> >>>>>>> On Apr 20, 2009, at 9:42 PM, Nicolas Cellier wrote: >>>>>>> >>>>>>>> Yes, >>>>>>>> Kernel-nice.renggli.301.mcz simply remove the upper limit for >>>>>>>> Delay >>>>>>>> because Andreas changes solved the problem. >>>>>>>> KernelTests-nice.96.mcz change the Delay upper bound test >>>>>>>> System-Support-nice.11.mcz fix a problem of sending deprecated >>>>>>>> Process>>#offList (it should have gone with earlier change) >>>>>>>> >>>>>>>> I wanted to decompose integration of Andreas changes carefully >>>>>>>> to be >>>>>>>> sure it can be redone... >>>>>>>> But now, I realize that I published too many messages, that's >>>>>>>> confusing >>>>>>>> >>>>>>>> 2009/4/20 Stéphane Ducasse <[email protected]>: >>>>>>>>> >>>>>>>>> nicolas >>>>>>>>> >>>>>>>>> I integrated in 10287 >>>>>>>>> >>>>>>>>>>> script363 >>>>>>>>>>> Delay startTimerEventLoop. >>>>>>>>>>> (self buildConfigurationMapFor: >>>>>>>>>>> #('Kernel-nice.renggli.300.mcz')) load >>>>>>>>> >>>>>>>>> So I imagine that the following one is another fix that I >>>>>>>>> should >>>>>>>>> integrate. >>>>>>>>> Is is correct >>>>>>>>> >>>>>>>>>> Comment #1 on issue 688 by nicolas.cellier.aka.nice: Kernel >>>>>>>>>> concurrency fixes? 3.10 >>>>>>>>>> http://code.google.com/p/pharo/issues/detail?id=688 >>>>>>>>>> >>>>>>>>>> I published the changes in PharoInbox >>>>>>>>>> I checked that entire changes related to mantis 7321 list can >>>>>>>>>> be >>>>>>>>>> loaded with following instructions: >>>>>>>>>> >>>>>>>>>> Delay startTimerEventLoop. >>>>>>>>>> (ScriptLoader new buildConfigurationMapFor: >>>>>>>>>> #('Kernel-nice.renggli.301.mcz' >>>>>>>>>> 'KernelTests-nice.96.mcz' >>>>>>>>>> 'System-Support-nice.11.mcz')) load. >>>>>>>>>> >>>>>>>>>> -- >>>>>>>>>> You received this message because you are listed in the owner >>>>>>>>>> or CC fields of this issue, or because you starred this >>>>>>>>>> issue. >>>>>>>>>> You may adjust your issue notification preferences at: >>>>>>>>>> http://code.google.com/hosting/settings >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> _______________________________________________ >>>>>>>>> Pharo-project mailing list >>>>>>>>> [email protected] >>>>>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>>>>> >>>>>>>> >>>>>>>> _______________________________________________ >>>>>>>> Pharo-project mailing list >>>>>>>> [email protected] >>>>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>>>> >>>>>>> >>>>>>> >>>>>>> _______________________________________________ >>>>>>> Pharo-project mailing list >>>>>>> [email protected] >>>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo- >>>>>>> project >>>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> [email protected] >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>> >>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> [email protected] >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo- >>>>> project >>>>> >>>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [email protected] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> >> _______________________________________________ >> Pharo-project mailing list >> [email protected] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > _______________________________________________ > Pharo-project mailing list > [email protected] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [email protected] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
