{"action":"created","issue":{"url":"https://api.github.com/repos/open-mpi/ompi/issues/2307","repository_url":"https://api.github.com/repos/open-mpi/ompi","labels_url":"https://api.github.com/repos/open-mpi/ompi/issues/2307/labels{/name}","comments_url":"https://api.github.com/repos/open-mpi/ompi/issues/2307/comments","events_url":"https://api.github.com/repos/open-mpi/ompi/issues/2307/events","html_url":"https://github.com/open-mpi/ompi/pull/2307","id":185711443,"number":2307,"title":"btl/self: rewrite to decrease memory usage","user":{"login":"hjelmn","id":1226817,"avatar_url":"https://avatars.githubusercontent.com/u/1226817?v=3","gravatar_id":"","url":"https://api.github.com/users/hjelmn","html_url":"https://github.com/hjelmn","followers_url":"https://api.github.com/users/hjelmn/followers","following_url":"https://api.github.com/users/hjelmn/following{/other_user}","gists_url":"https://api.github.com/users/hjelmn/gists{/gist_id}","starred_url":"https://api.github.com/users/hjelmn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjelmn/subscriptions","organizations_url":"https://api.github.com/users/hjelmn/orgs","repos_url":"https://api.github.com/users/hjelmn/repos","events_url":"https://api.github.com/users/hjelmn/events{/privacy}","received_events_url":"https://api.github.com/users/hjelmn/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/open-mpi/ompi/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/open-mpi/ompi/labels/code%20cleanup","name":"code cleanup","color":"006b75"},{"url":"https://api.github.com/repos/open-mpi/ompi/labels/Target:%202.x","name":"Target: 2.x","color":"d4c5f9"}],"state":"closed","locked":false,"assignee":{"login":"bosilca","id":642701,"avatar_url":"https://avatars.githubusercontent.com/u/642701?v=3","gravatar_id":"","url":"https://api.github.com/users/bosilca","html_url":"https://github.com/bosilca","followers_url":"https://api.github.com/users/bosilca/followers","following_url":"https://api.github.com/users/bosilca/following{/other_user}","gists_url":"https://api.github.com/users/bosilca/gists{/gist_id}","starred_url":"https://api.github.com/users/bosilca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bosilca/subscriptions","organizations_url":"https://api.github.com/users/bosilca/orgs","repos_url":"https://api.github.com/users/bosilca/repos","events_url":"https://api.github.com/users/bosilca/events{/privacy}","received_events_url":"https://api.github.com/users/bosilca/received_events","type":"User","site_admin":false},"assignees":[{"login":"bosilca","id":642701,"avatar_url":"https://avatars.githubusercontent.com/u/642701?v=3","gravatar_id":"","url":"https://api.github.com/users/bosilca","html_url":"https://github.com/bosilca","followers_url":"https://api.github.com/users/bosilca/followers","following_url":"https://api.github.com/users/bosilca/following{/other_user}","gists_url":"https://api.github.com/users/bosilca/gists{/gist_id}","starred_url":"https://api.github.com/users/bosilca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bosilca/subscriptions","organizations_url":"https://api.github.com/users/bosilca/orgs","repos_url":"https://api.github.com/users/bosilca/repos","events_url":"https://api.github.com/users/bosilca/events{/privacy}","received_events_url":"https://api.github.com/users/bosilca/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/open-mpi/ompi/milestones/20","html_url":"https://github.com/open-mpi/ompi/milestones/v2.0.2","labels_url":"https://api.github.com/repos/open-mpi/ompi/milestones/20/labels","id":1945157,"number":20,"title":"v2.0.2","description":"","creator":{"login":"jsquyres","id":617316,"avatar_url":"https://avatars.githubusercontent.com/u/617316?v=3","gravatar_id":"","url":"https://api.github.com/users/jsquyres","html_url":"https://github.com/jsquyres","followers_url":"https://api.github.com/users/jsquyres/followers","following_url":"https://api.github.com/users/jsquyres/following{/other_user}","gists_url":"https://api.github.com/users/jsquyres/gists{/gist_id}","starred_url":"https://api.github.com/users/jsquyres/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsquyres/subscriptions","organizations_url":"https://api.github.com/users/jsquyres/orgs","repos_url":"https://api.github.com/users/jsquyres/repos","events_url":"https://api.github.com/users/jsquyres/events{/privacy}","received_events_url":"https://api.github.com/users/jsquyres/received_events","type":"User","site_admin":false},"open_issues":10,"closed_issues":62,"state":"open","created_at":"2016-08-16T14:31:37Z","updated_at":"2016-10-27T16:34:55Z","due_on":"2016-12-31T00:00:00Z","closed_at":null},"comments":5,"created_at":"2016-10-27T16:01:51Z","updated_at":"2016-10-27T17:49:07Z","closed_at":"2016-10-27T16:34:54Z","pull_request":{"url":"https://api.github.com/repos/open-mpi/ompi/pulls/2307","html_url":"https://github.com/open-mpi/ompi/pull/2307","diff_url":"https://github.com/open-mpi/ompi/pull/2307.diff","patch_url":"https://github.com/open-mpi/ompi/pull/2307.patch"},"body":"This commit rewrites much of the btl/self component to fix a long\r\nstanding memory usage bug. Before this commit the prepare_src path\r\nwould always allocate a max send fragment (256kB). This caused the\r\nrank to allocate 32 * 256k useless buffers from one send. This commit\r\nmakes the following changes:\r\n\r\n - Add the MCA_BTL_FLAGS_GET flag by default. No reason not to set it.\r\n\r\n - Reduce the eager limit, max send size, buffers per allocation, and\r\n   maximum buffer count per fragment size. These changes should have\r\n   no noticible affect on performance but should greatly reduce the\r\n   memory usage of the component.\r\n\r\n - Implement the sendi function. This should reduce self send latency\r\n   somewhat.\r\n\r\n - Rewrite prepare_src to never allocate a eager or max send fragment\r\n   for contiguous data.\r\n\r\n - add_procs needs to return something in the peer array for the proc\r\n   self not just set the reachability bit. Now stores (void *) 1.\r\n\r\n - Various cleanups. Removed and unused file.\r\n\r\nSigned-off-by: Nathan Hjelm <hjelmn@lanl.gov>"},"comment":{"url":"https://api.github.com/repos/open-mpi/ompi/issues/comments/256719402","html_url":"https://github.com/open-mpi/ompi/pull/2307#issuecomment-256719402","issue_url":"https://api.github.com/repos/open-mpi/ompi/issues/2307","id":256719402,"user":{"login":"jsquyres","id":617316,"avatar_url":"https://avatars.githubusercontent.com/u/617316?v=3","gravatar_id":"","url":"https://api.github.com/users/jsquyres","html_url":"https://github.com/jsquyres","followers_url":"https://api.github.com/users/jsquyres/followers","following_url":"https://api.github.com/users/jsquyres/following{/other_user}","gists_url":"https://api.github.com/users/jsquyres/gists{/gist_id}","starred_url":"https://api.github.com/users/jsquyres/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsquyres/subscriptions","organizations_url":"https://api.github.com/users/jsquyres/orgs","repos_url":"https://api.github.com/users/jsquyres/repos","events_url":"https://api.github.com/users/jsquyres/events{/privacy}","received_events_url":"https://api.github.com/users/jsquyres/received_events","type":"User","site_admin":false},"created_at":"2016-10-27T17:49:07Z","updated_at":"2016-10-27T17:49:07Z","body":"Could you PR it to v2.1.0 for now?  Let me talk this over with Howard to see if we want it in v2.0.2.  It might be ok to push people in the direction of v2.1.0."}}
{"actor":{"display_login":"jsquyres"}}