Vm-dev July 2016

vm-dev@lists.squeakfoundation.org

34 participants
214 discussions

Start a nNew thread

CameraPlugin support for Mac OS?
by Yoshiki Ohshima 08 Jul '16

08 Jul '16

Is it true that there is no CameraPlugin support for Mac OS? -- -- Yoshiki

4 16

VM Maker: CogTools-eem.74.mcz
by commits＠source.squeak.org 08 Jul '16

08 Jul '16

Eliot Miranda uploaded a new version of CogTools to project VM Maker: http://source.squeak.org/VMMaker/CogTools-eem.74.mcz ==================== Summary ==================== Name: CogTools-eem.74 Author: eem Time: 8 July 2016, 1:01:48.875698 pm UUID: 6c802c74-0f7e-43e1-b4e7-1c05f89a2ce9 Ancestors: CogTools-eem.73 Don't use Utilities to obtain timestamp. =============== Diff against CogTools-eem.73 =============== Item was changed: ----- Method: VMProfiler>>putReportPreambleOn: (in category 'reports') ----- putReportPreambleOn: s | expr | + s nextPutAll: (SmalltalkImage current getSystemAttribute: 0); space; nextPutAll: Date today yyyymmdd; space. + Time now print24: true on: s. + s cr. - s nextPutAll: (SmalltalkImage current getSystemAttribute: 0); space; nextPutAll: Utilities changeStampPerSe; cr. (startStats size >= 44 and: [(startStats at: 44) isNumber]) ifTrue: [s nextPutAll: 'eden size: '; nextPutAll: (startStats at: 44) asStringWithCommas. s nextPutAll: ' stack pages: '; print: (startStats at: 42). (startStats size >= 46 and: [(startStats at: 46) isNumber and: [(startStats at: 46) > 0]]) ifTrue: [s nextPutAll: ' code size: '; nextPutAll: (startStats at: 46) asStringWithCommas]. s cr]. s cr. (expr := self trimmedExpressionText) notEmpty ifTrue: [s nextPutAll: expr; cr; cr]. (gcPriorToProfile or: [clearPriorToProfile or: [forkProfile]]) ifTrue: [gcPriorToProfile ifTrue: [s nextPutAll: 'gc prior. ']. clearPriorToProfile ifTrue: [s nextPutAll: 'clear prior. ']. forkProfile ifTrue: [s nextPutAll: 'run in separate process.']. s cr]. elapsedTime > 0 ifTrue: [s print: elapsedTime / 1000.0; nextPutAll: ' seconds; sampling frequency '; print: (total * 1000 / elapsedTime) rounded; nextPutAll: ' hz'; cr]!

1 0

[OpenSmalltalk/opensmalltalk-vm] 224596: Use atoll for SQ_HOST64 to cover win32 LLP64 case
by GitHub 08 Jul '16

08 Jul '16

Branch: refs/heads/Cog Home: https://github.com/OpenSmalltalk/opensmalltalk-vm Commit: 2245967bf5663451534d743f3a38aa5b6f91c7c0 https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/2245967bf566345153… Author: Nicolas Cellier <nicolas.cellier(a)sirehna.com> Date: 2016-07-08 (Fri, 08 Jul 2016) Changed paths: M platforms/win32/vm/sqWin32Main.c Log Message: ----------- Use atoll for SQ_HOST64 to cover win32 LLP64 case Also revert to original atoi in SQ_HOST32 case.

1 0

[OpenSmalltalk/opensmalltalk-vm] 931559: Replace another (UINT) cast by (usqIntptr_t) for L...
by GitHub 08 Jul '16

08 Jul '16

Branch: refs/heads/Cog Home: https://github.com/OpenSmalltalk/opensmalltalk-vm Commit: 931559445c72d9a53541289330ddb5157c94daf5 https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/931559445c72d9a535… Author: Nicolas Cellier <nicolas.cellier(a)sirehna.com> Date: 2016-07-08 (Fri, 08 Jul 2016) Changed paths: M platforms/win32/vm/sqWin32Main.c Log Message: ----------- Replace another (UINT) cast by (usqIntptr_t) for LLP64 compatibility (UINT) would not work on LP64 anyway, it would truncate the pointer.

4 4

[OpenSmalltalk/opensmalltalk-vm] 1d0fbd: Remove a #warning slip (for debug purpose)
by GitHub 08 Jul '16

08 Jul '16

Branch: refs/heads/Cog Home: https://github.com/OpenSmalltalk/opensmalltalk-vm Commit: 1d0fbd7364ec8afa2dbe068bd3db47fa0759d58c https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/1d0fbd7364ec8afa2d… Author: Nicolas Cellier <nicolas.cellier(a)sirehna.com> Date: 2016-07-08 (Fri, 08 Jul 2016) Changed paths: M platforms/Cross/vm/sqMemoryAccess.h Log Message: ----------- Remove a #warning slip (for debug purpose)

1 0

primitiveDisplayString anomaly
by Eliot Miranda 08 Jul '16

08 Jul '16

Hi All, Hi Tim R, in looking at a different issue I've noticed that string display seems to be falling to use the primitive in many cases. I'm running a trunk image with the REPL (spur reader.image) in the simulator and I'm seeing that the very first use (and subsequent uses) of primitiveDisplayString is failing because the sourceForm's sourceBits is too short. In loadBitBltSourceForm see that sourceWidth 16r66E 1646 sourceheight 16rE 14 sourceDepth 16r10 16 sourcePPW 2 sourcePitch 16rCDC 3292 but sourcePitch * sourceHeight 46088 sourceBitsSize 16r744E 29774 yet we expect sourceBitsSize >= sourcePitch * sourceHeight Could something be wrong with our layout of font forms? Anyway, pointing this out because this could be losing us a lot of performance, and I don't understand the graphics system well enough to feel confident fixing this. Any takers? _,,,^..^,,,_ best, Eliot

2 2

Re: [squeak-dev] Endless recursion: "String new: -1"
by Tobias Pape 08 Jul '16

08 Jul '16

Hi all (cc vm-dev) On 07.07.2016, at 14:28, David T. Lewis <lewis(a)mail.msen.com> wrote: > I think the problem is in the primitive error code checking. The primitive > is failing with #'bad argument' but the fallback code attempts to handle it > as #'insufficient object memory'. It then tries to free some memory, fails > to correct the problem, and raises a "Space is low" notifier. > I noted that when we moved to Spur initially and I tried to fix tests. The AllocationTest failed, and I changed ec == #'insufficient object memory' ifTrue: to (ec == #'insufficient object memory' or: [ec == #'bad argument']) ifTrue: in Behavior>>#basicNew: Maybe that was an error? @Eliot, why does Spur return #'bad argument' instead of #'insufficient object memory' when too much memory is to be allocated? Best regards -Tobias > Dave > > > On Thu, Jul 07, 2016 at 09:23:14AM +0200, Levente Uzonyi wrote: >> Someone seems to have trimmed the versions in the changes file. In Squeak >> 4.4 Behavior >> #basicNew: had the following body: >> >> <primitive: 71> >> self isVariable ifFalse: >> [self error: self printString, ' cannot have variable sized >> instances']. >> (sizeRequested isInteger and: [sizeRequested >= 0]) ifTrue: >> ["arg okay; space must be low." >> OutOfMemory signal. >> ^ self basicNew: sizeRequested "retry if user proceeds"]. >> self primitiveFailed >> >> So, non-integer and negative arguments were primitive failures. >> >> Levente >> >> >> On Wed, 6 Jul 2016, David T. Lewis wrote: >> >>> On Wed, Jul 06, 2016 at 06:43:25AM -0700, marcel.taeumel wrote: >>>> Hi, there! >>>> >>>> Is it okay that there is an endless recursion when evaluating "String >>>> new: -1"? >>> >>> No, it is not okay. It should fail with a primitive failure. >>> >>> Dave >>> >>> >>>> >>>> ... >>>> ByteString class(Behavior)>>handleFailingFailingBasicNew: >>>> ByteString class(Behavior)>>handleFailingBasicNew: >>>> ByteString class(Behavior)>>basicNew: >>>> ByteString class(Behavior)>>handleFailingFailingBasicNew: >>>> ByteString class(Behavior)>>handleFailingBasicNew: >>>> ByteString class(Behavior)>>basicNew: >>>> ByteString class(Behavior)>>handleFailingFailingBasicNew: >>>> ByteString class(Behavior)>>handleFailingBasicNew: >>>> ByteString class(Behavior)>>basicNew: >>>> ... >>>> >>>> I would like to have an error signaled instead. Note that the -1 is just >>>> an >>>> example for a bad computation. The error I get is "Space is low" then. :-) >>>> >>>> >>>> Best, >>>> Marcel >>>> >>>> >>>> >>>> -- >>>> View this message in context: >>>> http://forum.world.st/Endless-recursion-String-new-1-tp4905179.html >>>> Sent from the Squeak - Dev mailing list archive at Nabble.com.

3 11

[OpenSmalltalk/opensmalltalk-vm] a64e0d: Use same Win32 function prototypes in .h and .c fo...
by GitHub 07 Jul '16

07 Jul '16

Branch: refs/heads/Cog Home: https://github.com/OpenSmalltalk/opensmalltalk-vm Commit: a64e0dbbcc97c76c54decd0502b98bb81d2eca02 https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/a64e0dbbcc97c76c54… Author: Nicolas Cellier <nicolas.cellier(a)sirehna.com> Date: 2016-07-07 (Thu, 07 Jul 2016) Changed paths: M platforms/win32/plugins/SecurityPlugin/sqWin32Security.c M platforms/win32/plugins/SocketPlugin/sqWin32NewNet.c M platforms/win32/vm/sqPlatformSpecific.h Log Message: ----------- Use same Win32 function prototypes in .h and .c for X64 compatibility (sqInt vs int) Note: for most network function a 32bits address and port would do, we don't need 64bits. But changing int -> sqint is less intrusive (no change required in VMMaker)

3 3

Re: Why is the method context size fixed to two magical numbers
by Max Leske 07 Jul '16

07 Jul '16

Thanks Clément, that explains it. > On 07 Jul 2016, at 13:52, vm-dev-request(a)lists.squeakfoundation.org wrote: > > Hi, > > In the interpreter VM, contexts are recycled and you have 2 pools of > contexts, one for each size. Contexts represent 30% of allocations in that > VM. > > In the Stack and Cog VM, the size is used to detect stack page overflow and > to allocate contexts, we could use a precise value instead of this bit > flag. Using a single bit saves memory as you need 1 bit per compiled method > instead of 1 byte (or even 1 word !). > > Cheers > > On Thu, Jul 7, 2016 at 1:02 PM, Max Leske <maxleske(a)gmail.com <mailto:maxleske@gmail.com>> wrote: > >> >> Hi, >> >> I’ve been looking at the code that creates new method contexts (in the >> image and in the VM) and I can’t figure out why it would be beneficial to >> fix the size of the context to (currently) 16 (22) or 56 (62) bytes. >> Clément says in one of his blog entries that this if for performance >> reasons but I don’t see where the performance gain is (the Bluebook doesn’t >> mention this either). At allocation time the VM simply takes the number of >> bytes to allocate from the “large context flag”. Maybe the performance gain >> comes from the idea that the size does not have to be calculated? But then >> I ask: why not simply store the frame size instead of this weird flag? >> >> Cheers, >> Max

2 1

[OpenSmalltalk/opensmalltalk-vm] 33dfd2: Use sqIntptr_t rather than sqint for converting in...
by GitHub 07 Jul '16

07 Jul '16

Branch: refs/heads/Cog Home: https://github.com/OpenSmalltalk/opensmalltalk-vm Commit: 33dfd2b6207f60a98e70036768b3f62d19aaffa5 https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/33dfd2b6207f60a98e… Author: Nicolas Cellier <nicolas.cellier.aka.nice(a)gmail.com> Date: 2016-07-07 (Thu, 07 Jul 2016) Changed paths: M platforms/Cross/vm/sqHeapMap.c M platforms/win32/vm/sqWin32Backtrace.c M platforms/win32/vm/sqWin32Prefs.c M platforms/win32/vm/sqWin32Window.c Log Message: ----------- Use sqIntptr_t rather than sqint for converting int<->pointer This preserves compatibility for legacy interpreter 32 bits image on 64 bits VM.

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

Vm-dev July 2016