{"action":"created","issue":{"url":"https://api.github.com/repos/openucx/ucx/issues/871","repository_url":"https://api.github.com/repos/openucx/ucx","labels_url":"https://api.github.com/repos/openucx/ucx/issues/871/labels{/name}","comments_url":"https://api.github.com/repos/openucx/ucx/issues/871/comments","events_url":"https://api.github.com/repos/openucx/ucx/issues/871/events","html_url":"https://github.com/openucx/ucx/pull/871","id":163760230,"number":871,"title":"UCT/SM: pass /proc/<pid>/fd/<fd> instead of path for posix shared mem…","user":{"login":"xinzhao3","id":16946816,"avatar_url":"https://avatars.githubusercontent.com/u/16946816?v=3","gravatar_id":"","url":"https://api.github.com/users/xinzhao3","html_url":"https://github.com/xinzhao3","followers_url":"https://api.github.com/users/xinzhao3/followers","following_url":"https://api.github.com/users/xinzhao3/following{/other_user}","gists_url":"https://api.github.com/users/xinzhao3/gists{/gist_id}","starred_url":"https://api.github.com/users/xinzhao3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xinzhao3/subscriptions","organizations_url":"https://api.github.com/users/xinzhao3/orgs","repos_url":"https://api.github.com/users/xinzhao3/repos","events_url":"https://api.github.com/users/xinzhao3/events{/privacy}","received_events_url":"https://api.github.com/users/xinzhao3/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openucx/ucx/labels/enhancement","name":"enhancement","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":16,"created_at":"2016-07-05T02:17:01Z","updated_at":"2016-07-07T09:47:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openucx/ucx/pulls/871","html_url":"https://github.com/openucx/ucx/pull/871","diff_url":"https://github.com/openucx/ucx/pull/871.diff","patch_url":"https://github.com/openucx/ucx/pull/871.patch"},"body":"…ory.\r\n\r\nOriginally in posix shared memory implementation, every process passes\r\na path to open shared memory file. This has the problem of stale files\r\nwhen the host process exits early. To avoid this, in the patch, we pass\r\n/proc/<pid>/fd/<fd> instead of path on each process, and does following:\r\n\r\n1. Each process opens its own posix file\r\n2. Each process immediately unlinks it (but keeps it open)\r\n3. Each process mmaps the file\r\n4. Let other processes open /proc/<pid>/fd/<fd> and mmap to it\r\n5. In case of exit, the file will be automatically deleted, because no\r\n   directory entry points to it\r\n\r\nfixes #718"},"comment":{"url":"https://api.github.com/repos/openucx/ucx/issues/comments/231033100","html_url":"https://github.com/openucx/ucx/pull/871#issuecomment-231033100","issue_url":"https://api.github.com/repos/openucx/ucx/issues/871","id":231033100,"user":{"login":"yosefe","id":2255631,"avatar_url":"https://avatars.githubusercontent.com/u/2255631?v=3","gravatar_id":"","url":"https://api.github.com/users/yosefe","html_url":"https://github.com/yosefe","followers_url":"https://api.github.com/users/yosefe/followers","following_url":"https://api.github.com/users/yosefe/following{/other_user}","gists_url":"https://api.github.com/users/yosefe/gists{/gist_id}","starred_url":"https://api.github.com/users/yosefe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yosefe/subscriptions","organizations_url":"https://api.github.com/users/yosefe/orgs","repos_url":"https://api.github.com/users/yosefe/repos","events_url":"https://api.github.com/users/yosefe/events{/privacy}","received_events_url":"https://api.github.com/users/yosefe/received_events","type":"User","site_admin":false},"created_at":"2016-07-07T09:47:22Z","updated_at":"2016-07-07T09:47:22Z","body":"@xinzhao3 sorry for misunderstanding, but i've meant UCX runtime configuration option, not a compile-time configuration. See https://github.com/openucx/ucx/blob/master/src/uct/sm/mm/mm_posix.c#L28"}}
{"actor":{"display_login":"yosefe"}}