{"action":"created","issue":{"url":"https://api.github.com/repos/ofiwg/libfabric/issues/2261","repository_url":"https://api.github.com/repos/ofiwg/libfabric","labels_url":"https://api.github.com/repos/ofiwg/libfabric/issues/2261/labels{/name}","comments_url":"https://api.github.com/repos/ofiwg/libfabric/issues/2261/comments","events_url":"https://api.github.com/repos/ofiwg/libfabric/issues/2261/events","html_url":"https://github.com/ofiwg/libfabric/pull/2261","id":170570667,"number":2261,"title":"man: Updates to AV man page.","user":{"login":"bturrubiates","id":2322856,"avatar_url":"https://avatars.githubusercontent.com/u/2322856?v=3","gravatar_id":"","url":"https://api.github.com/users/bturrubiates","html_url":"https://github.com/bturrubiates","followers_url":"https://api.github.com/users/bturrubiates/followers","following_url":"https://api.github.com/users/bturrubiates/following{/other_user}","gists_url":"https://api.github.com/users/bturrubiates/gists{/gist_id}","starred_url":"https://api.github.com/users/bturrubiates/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bturrubiates/subscriptions","organizations_url":"https://api.github.com/users/bturrubiates/orgs","repos_url":"https://api.github.com/users/bturrubiates/repos","events_url":"https://api.github.com/users/bturrubiates/events{/privacy}","received_events_url":"https://api.github.com/users/bturrubiates/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ofiwg/libfabric/labels/documentation","name":"documentation","color":"0e8a16"},{"url":"https://api.github.com/repos/ofiwg/libfabric/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/ofiwg/libfabric/milestones/13","html_url":"https://github.com/ofiwg/libfabric/milestones/release%201.4.0","labels_url":"https://api.github.com/repos/ofiwg/libfabric/milestones/13/labels","id":1671739,"number":13,"title":"release 1.4.0","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":21,"closed_issues":86,"state":"open","created_at":"2016-03-29T12:35:33Z","updated_at":"2016-08-11T04:57:45Z","due_on":"2016-09-30T04:00:00Z","closed_at":null},"comments":8,"created_at":"2016-08-11T04:57:45Z","updated_at":"2016-08-11T17:20:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ofiwg/libfabric/pulls/2261","html_url":"https://github.com/ofiwg/libfabric/pull/2261","diff_url":"https://github.com/ofiwg/libfabric/pull/2261.diff","patch_url":"https://github.com/ofiwg/libfabric/pull/2261.patch"},"body":"- Clarify that failed the fi_addr buffer of corresponding failed\r\n  insertions will be set to FI_ADDR_NOTAVAIL\r\n\r\n@goodell @jsquyres @j-xiong @jithinjosepkl \r\n\r\nSigned-off-by: Ben Turrubiates <bturrubi@cisco.com>"},"comment":{"url":"https://api.github.com/repos/ofiwg/libfabric/issues/comments/239229155","html_url":"https://github.com/ofiwg/libfabric/pull/2261#issuecomment-239229155","issue_url":"https://api.github.com/repos/ofiwg/libfabric/issues/2261","id":239229155,"user":{"login":"bturrubiates","id":2322856,"avatar_url":"https://avatars.githubusercontent.com/u/2322856?v=3","gravatar_id":"","url":"https://api.github.com/users/bturrubiates","html_url":"https://github.com/bturrubiates","followers_url":"https://api.github.com/users/bturrubiates/followers","following_url":"https://api.github.com/users/bturrubiates/following{/other_user}","gists_url":"https://api.github.com/users/bturrubiates/gists{/gist_id}","starred_url":"https://api.github.com/users/bturrubiates/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bturrubiates/subscriptions","organizations_url":"https://api.github.com/users/bturrubiates/orgs","repos_url":"https://api.github.com/users/bturrubiates/repos","events_url":"https://api.github.com/users/bturrubiates/events{/privacy}","received_events_url":"https://api.github.com/users/bturrubiates/received_events","type":"User","site_admin":false},"created_at":"2016-08-11T17:20:52Z","updated_at":"2016-08-11T17:20:52Z","body":"> The application knows what index it would get if fi_addr were not NULL and uses that one.\r\n\r\nIt's easy for the simple case where only `fi_av_insert` has been called, then you could just assume sequential indexes starting from `0`. The issue I see is whenever an `fi_av_remove` has been called. The documentation for `fi_av_remove` says:\r\n\r\n> fi_av_remove removes a set of addresses from an address vector. All resources\r\nassociated with the indicated addresses are released, and no future references\r\nto either the mapped address (in the case of FI_AV_MAP) or index (FI_AV_TABLE)\r\nare allowed.\r\n\r\n> The use of fi_av_remove is an optimization that applications may use to free\r\nmemory allocated with addresses that will no longer be accessed. Inserted\r\naddresses are not required to be removed. fi_av_close will automatically\r\ncleanup any resources associated with addresses remaining in the AV when it is\r\ninvoked.\r\n\r\nSo this means that it becomes difficult for an application to determine what the resulting index will be. This is implementation specific.\r\n\r\n"}}
{"actor":{"display_login":"bturrubiates"}}