Hi Fabio,<br>
<br>
On Wed, Aug 8, 2018 at 1:26 PM, Fabio Niephaus <notifications@github.com><br>
wrote:<br>
<br>
> Hi Eliot,<br>
><br>
> Is there actual proof of issues caused by rtprio not raised correctly or<br>
> is this more a precaution/speculation? smalltalkCI, for instance, builds<br>
> hundreds of projects every week and we didn't see any problems when we<br>
> switched from itimer to threaded VMs.<br>
><br>
<br>
Absolutely.  The symptom is that when there is no idle time then the clock<br>
does not advance and delays don't fire.  So for example,e, if you spawn a<br>
process at userBackgroundPriority that simply loops, delays will not work.<br>
Tis is because the VM thread is now always union and hence the heartbeat<br>
thread, being at the same priority as the VM thread, never gets a chance to<br>
run, and so the heartbeat gets disabled.<br>
<br>
If your concern is that the error message itself may be in error, I assure<br>
you that it is real, and it is reporting that on the system it is not<br>
allowed to spawn threads at higher priority without taking special steps.<br>
And indeed this is described in the source code and the error message.  It<br>
really is there for good reason, not just because I'm anal about it ;-)<br>
<br>
<br>
> The easiest fix here, obviously, would be to hide the note all-together<br>
> and maybe add it to some sort of README.txt that ships alongside the<br>
> binaries. Maybe it could be that simple?<br>
><br>
> —<br>
> You are receiving this because you commented.<br>
> Reply to this email directly, view it on GitHub<br>
> <https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/275#issuecomment-411540745>,<br>
> or mute the thread<br>
> <https://github.com/notifications/unsubscribe-auth/APHa0ENm8XYsI9kG-HZaS5oB8KD1aZHGks5uO0l7gaJpZM4V0Q3j><br>
> .<br>
><br>
_,,,^..^,,,_<br>
best, Eliot<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you commented.<br />Reply to this email directly, <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/275#issuecomment-411569935">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyWySZRyy1uJsNnTeBDTbop1iWzg4Nks5uO2JmgaJpZM4V0Q3j">mute the thread</a>.<img src="https://github.com/notifications/beacon/AhLyW5n0SZj8bzrC7x5-1FojD1Z-LFP6ks5uO2JmgaJpZM4V0Q3j.gif" height="1" width="1" alt="" /></p>
<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://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@eliotmiranda in #275: Hi Fabio,\n\nOn Wed, Aug 8, 2018 at 1:26 PM, Fabio Niephaus \u003cnotifications@github.com\u003e\nwrote:\n\n\u003e Hi Eliot,\n\u003e\n\u003e Is there actual proof of issues caused by rtprio not raised correctly or\n\u003e is this more a precaution/speculation? smalltalkCI, for instance, builds\n\u003e hundreds of projects every week and we didn't see any problems when we\n\u003e switched from itimer to threaded VMs.\n\u003e\n\nAbsolutely.  The symptom is that when there is no idle time then the clock\ndoes not advance and delays don't fire.  So for example,e, if you spawn a\nprocess at userBackgroundPriority that simply loops, delays will not work.\nTis is because the VM thread is now always union and hence the heartbeat\nthread, being at the same priority as the VM thread, never gets a chance to\nrun, and so the heartbeat gets disabled.\n\nIf your concern is that the error message itself may be in error, I assure\nyou that it is real, and it is reporting that on the system it is not\nallowed to spawn threads at higher priority without taking special steps.\nAnd indeed this is described in the source code and the error message.  It\nreally is there for good reason, not just because I'm anal about it ;-)\n\n\n\u003e The easiest fix here, obviously, would be to hide the note all-together\n\u003e and maybe add it to some sort of README.txt that ships alongside the\n\u003e binaries. Maybe it could be that simple?\n\u003e\n\u003e —\n\u003e You are receiving this because you commented.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/issues/275#issuecomment-411540745\u003e,\n\u003e or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/APHa0ENm8XYsI9kG-HZaS5oB8KD1aZHGks5uO0l7gaJpZM4V0Q3j\u003e\n\u003e .\n\u003e\n_,,,^..^,,,_\nbest, Eliot\n"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/275#issuecomment-411569935"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/275#issuecomment-411569935",
"url": "https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/275#issuecomment-411569935",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [OpenSmalltalk/opensmalltalk-vm] Provide control over VM messages (#275)",
"sections": [
{
"text": "",
"activityTitle": "**Eliot Miranda**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@eliotmiranda",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"OpenSmalltalk/opensmalltalk-vm\",\n\"issueId\": 275,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"OpenSmalltalk/opensmalltalk-vm\",\n\"issueId\": 275\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/275#issuecomment-411569935"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 366022115\n}"
}
],
"themeColor": "26292E"
}
]</script>