[OpenSmalltalk/opensmalltalk-vm] Readonly primitives are missing in ARM VM (#216)

 

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Readonly primitives are missing in ARM VM (#216)"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216"}}}</script>

 

@timfel @eliotmiranda Is there any specific reason not to have read-only primitive on ARM ? Where they not working ?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@clementbera in #216: @timfel @eliotmiranda Is there any specific reason not to have read-only primitive on ARM ? Where they not working ?"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-373515051"}}}</script>

 

build.linux32ARMv[67]/pharo.cog.spur/build*/mvm is lacking -DIMMUTABILITY. c.f. -DIMMUTABILITY=1 in build.win*/pharo.cog.spur/Makefile build.macos*pharo.cog.spur/Makefile.

I wish someone would volunteer to rewrite the linux build system to copy the much nicer scheme used in Mac OS and Windows.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@eliotmiranda in #216: build.linux32ARMv[67]/pharo.cog.spur/build*/mvm is lacking -DIMMUTABILITY. c.f. -DIMMUTABILITY=1 in build.win*/pharo.cog.spur/Makefile build.macos*pharo.cog.spur/Makefile.\r\n\r\nI wish someone would volunteer to rewrite the linux build system to copy the much nicer scheme used in Mac OS and Windows."}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-373530980"}}}</script>

 

@timfel @eliotmiranda Is there any specific reason not to have read-only primitive on ARM ? Where they not working ?

I can't remember, but it should be fairly easy to try this out on our CI if you enable them on a branch.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@fniephaus in #216: \u003e @timfel @eliotmiranda Is there any specific reason not to have read-only primitive on ARM ? Where they not working ?\r\n\r\nI can't remember, but it should be fairly easy to try this out on our CI if you enable them on a branch."}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-373637814"}}}</script>

 

So you're saying it's just a compilation setting but read-only objects work on ARM.

Ok I'll enable them then.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@clementbera in #216: So you're saying it's just a compilation setting but read-only objects work on ARM.\r\n\r\nOk I'll enable them then."}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-373653453"}}}</script>

 

@clementbera I didn't say they work on ARM, they may work. But please go ahead and try that out :)

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@fniephaus in #216: @clementbera I didn't say they work on ARM, they may work. But please go ahead and try that out :)"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-373666950"}}}</script>

 

Closed #216.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Closed #216."}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#event-1526905295"}}}</script>

 

So that's solved and integrated.
commit 2a67895

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@clementbera in #216: So that's solved and integrated.\r\ncommit 2a67895d1c4b51cbfba77469c0c696b8b16cf104"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-373933926"}}}</script>

 

Cool, thanks!

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@fniephaus in #216: Cool, thanks!"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-373934623"}}}</script>

 

 
On Mon, Mar 19, 2018 at 3:02 PM, OpenSmalltalk-Bot <[hidden email]
> wrote:

> Where to get VM binary to test?
> Last binaries on bintray are from 12 days ago.
> And I checked latest Pharo VM for ARM. It returns false for "Smalltalk vm
> supportsWriteBarrier"


Hi Denis,

Latest here: http://files.pharo.org/vm/pharo-spur32/linux/armv6/
should work.

Seems bintray has been disabled for two weeks (invalid certificate to
upload artifacts)



>
>
> 2018-03-17 17:46 GMT+01:00 Fabio Niephaus <[hidden email]>:
>
> >
> >
> > Cool, thanks!
> >
> > —
> > You are receiving this because you are subscribed to this thread.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/OpenSmalltalk/opensmalltalk-
> vm/issues/216#issuecomment-373934623>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/
> AhLyW0Mxb7XIun8HVIyzxqbn6SlmYohzks5tfT35gaJpZM4SFWOT>
> > .
>
> >
> >
>
> —
> You are receiving this because you modified the open/close state.
> Reply to this email directly, view it on GitHub
> <https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-374222193>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AC0GgvDTsP1ni3FW_IRxZpeaenNuw00iks5tf7qIgaJpZM4SFWOT>
> .
>



--
Clément Béra
https://clementbera.github.io/
https://clementbera.wordpress.com/

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@clementbera in #216: On Mon, Mar 19, 2018 at 3:02 PM, OpenSmalltalk-Bot \u003cnotifications@github.com\n\u003e wrote:\n\n\u003e Where to get VM binary to test?\n\u003e Last binaries on bintray are from 12 days ago.\n\u003e And I checked latest Pharo VM for ARM. It returns false for \"Smalltalk vm\n\u003e supportsWriteBarrier\"\n\n\nHi Denis,\n\nLatest here: http://files.pharo.org/vm/pharo-spur32/linux/armv6/\nshould work.\n\nSeems bintray has been disabled for two weeks (invalid certificate to\nupload artifacts)\n\n\n\n\u003e\n\u003e\n\u003e 2018-03-17 17:46 GMT+01:00 Fabio Niephaus \u003cnotifications@github.com\u003e:\n\u003e\n\u003e \u003e\n\u003e \u003e\n\u003e \u003e Cool, thanks!\n\u003e \u003e\n\u003e \u003e —\n\u003e \u003e You are receiving this because you are subscribed to this thread.\n\u003e \u003e Reply to this email directly, view it on GitHub\n\u003e \u003e \u003chttps://github.com/OpenSmalltalk/opensmalltalk-\n\u003e vm/issues/216#issuecomment-373934623\u003e,\n\u003e \u003e or mute the thread\n\u003e \u003e \u003chttps://github.com/notifications/unsubscribe-auth/\n\u003e AhLyW0Mxb7XIun8HVIyzxqbn6SlmYohzks5tfT35gaJpZM4SFWOT\u003e\n\u003e \u003e .\n\u003e\n\u003e \u003e\n\u003e \u003e\n\u003e\n\u003e —\n\u003e You are receiving this because you modified the open/close state.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-374222193\u003e,\n\u003e or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/AC0GgvDTsP1ni3FW_IRxZpeaenNuw00iks5tf7qIgaJpZM4SFWOT\u003e\n\u003e .\n\u003e\n\n\n\n-- \nClément Béra\nhttps://clementbera.github.io/\nhttps://clementbera.wordpress.com/\n"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-374236092"}}}</script>

 

Hi Clement.
It does not support it. I checked with latest:

CoInterpreter VMMaker.oscog-eem.2347 uuid: 062614a7-e3da-4b30-997a-9568911b9ff5 Mar 16 2018
StackToRegisterMappingCogit VMMaker.oscog-eem.2347 uuid: 062614a7-e3da-4b30-997a-9568911b9ff5 Mar 16 2018
VM: 201803151936 https://github.com/OpenSmalltalk/opensmalltalk-vm.git $ Date: Thu Mar 15 20:36:43 2018 +0100 $ Plugins: 201803151936 https://github.com/OpenSmalltalk/opensmalltalk-vm.git $

Looks like it was build with version from 15 March.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@dionisiydk in #216: Hi Clement.\r\nIt does not support it. I checked with latest:\r\n\r\nCoInterpreter VMMaker.oscog-eem.2347 uuid: 062614a7-e3da-4b30-997a-9568911b9ff5 Mar 16 2018\r\nStackToRegisterMappingCogit VMMaker.oscog-eem.2347 uuid: 062614a7-e3da-4b30-997a-9568911b9ff5 Mar 16 2018\r\nVM: 201803151936 https://github.com/OpenSmalltalk/opensmalltalk-vm.git $ Date: Thu Mar 15 20:36:43 2018 +0100 $ Plugins: 201803151936 https://github.com/OpenSmalltalk/opensmalltalk-vm.git $\r\n\r\nLooks like it was build with version from 15 March. "}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-374239968"}}}</script>

 

Seems bintray has been disabled for two weeks (invalid certificate to
upload artifacts)

8d716d4 uses a different Bintray user for deployments. Unfortunately, I cannot tell if this fixes the problem because the Cog branch is currently broken...

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@fniephaus in #216: \u003e Seems bintray has been disabled for two weeks (invalid certificate to\r\nupload artifacts)\r\n\r\nhttps://github.com/OpenSmalltalk/opensmalltalk-vm/commit/8d716d4ed7cb28c25491692f763e71b277b9007c uses a different Bintray user for deployments. Unfortunately, I cannot tell if this fixes the problem because the `Cog` branch is currently broken..."}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/216#issuecomment-374246150"}}}</script>