{"action":"created","issue":{"url":"https://api.github.com/repos/dyninst/dyninst/issues/214","repository_url":"https://api.github.com/repos/dyninst/dyninst","labels_url":"https://api.github.com/repos/dyninst/dyninst/issues/214/labels{/name}","comments_url":"https://api.github.com/repos/dyninst/dyninst/issues/214/comments","events_url":"https://api.github.com/repos/dyninst/dyninst/issues/214/events","html_url":"https://github.com/dyninst/dyninst/pull/214","id":187469119,"number":214,"title":"proccontrol: Synchronize additional threads found during attach","user":{"login":"cuviper","id":36186,"avatar_url":"https://avatars.githubusercontent.com/u/36186?v=3","gravatar_id":"","url":"https://api.github.com/users/cuviper","html_url":"https://github.com/cuviper","followers_url":"https://api.github.com/users/cuviper/followers","following_url":"https://api.github.com/users/cuviper/following{/other_user}","gists_url":"https://api.github.com/users/cuviper/gists{/gist_id}","starred_url":"https://api.github.com/users/cuviper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cuviper/subscriptions","organizations_url":"https://api.github.com/users/cuviper/orgs","repos_url":"https://api.github.com/users/cuviper/repos","events_url":"https://api.github.com/users/cuviper/events{/privacy}","received_events_url":"https://api.github.com/users/cuviper/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-11-05T01:35:43Z","updated_at":"2016-11-05T01:39:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dyninst/dyninst/pulls/214","html_url":"https://github.com/dyninst/dyninst/pull/214","diff_url":"https://github.com/dyninst/dyninst/pull/214.diff","patch_url":"https://github.com/dyninst/dyninst/pull/214.patch"},"body":"When additional threads are found during the attach process, we should\r\nsynchronize to their stopping point, and check for new threads again,\r\nuntil no new threads are found.  This keeps a more consistent state if\r\nthreads are racing to start while we're attaching."},"comment":{"url":"https://api.github.com/repos/dyninst/dyninst/issues/comments/258583042","html_url":"https://github.com/dyninst/dyninst/pull/214#issuecomment-258583042","issue_url":"https://api.github.com/repos/dyninst/dyninst/issues/214","id":258583042,"user":{"login":"cuviper","id":36186,"avatar_url":"https://avatars.githubusercontent.com/u/36186?v=3","gravatar_id":"","url":"https://api.github.com/users/cuviper","html_url":"https://github.com/cuviper","followers_url":"https://api.github.com/users/cuviper/followers","following_url":"https://api.github.com/users/cuviper/following{/other_user}","gists_url":"https://api.github.com/users/cuviper/gists{/gist_id}","starred_url":"https://api.github.com/users/cuviper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cuviper/subscriptions","organizations_url":"https://api.github.com/users/cuviper/orgs","repos_url":"https://api.github.com/users/cuviper/repos","events_url":"https://api.github.com/users/cuviper/events{/privacy}","received_events_url":"https://api.github.com/users/cuviper/received_events","type":"User","site_admin":false},"created_at":"2016-11-05T01:39:39Z","updated_at":"2016-11-05T01:39:39Z","body":"This has so far appeared to solve the remaining problems of #208.  I'll do more testing next week.\r\n\r\ncc @mplegendre You mentioned this point of extra `attachThreads` was originally for BG/Q.  I have no idea if my modification to sync up here will still be safe for that environment - please review.  This was basically your suggested fix though, but I couldn't reuse `waitfor_startup()` as the main process's `getState()` already got marked as running."}}
{"actor":{"display_login":"cuviper"}}