{"action":"created","issue":{"url":"https://api.github.com/repos/openucx/ucx/issues/661","repository_url":"https://api.github.com/repos/openucx/ucx","labels_url":"https://api.github.com/repos/openucx/ucx/issues/661/labels{/name}","comments_url":"https://api.github.com/repos/openucx/ucx/issues/661/comments","events_url":"https://api.github.com/repos/openucx/ucx/issues/661/events","html_url":"https://github.com/openucx/ucx/pull/661","id":137986933,"number":661,"title":"uct/ugni: always call GNI_CqGetEvent on TX CQ","user":{"login":"hppritcha","id":7818666,"avatar_url":"https://avatars.githubusercontent.com/u/7818666?v=3","gravatar_id":"","url":"https://api.github.com/users/hppritcha","html_url":"https://github.com/hppritcha","followers_url":"https://api.github.com/users/hppritcha/followers","following_url":"https://api.github.com/users/hppritcha/following{/other_user}","gists_url":"https://api.github.com/users/hppritcha/gists{/gist_id}","starred_url":"https://api.github.com/users/hppritcha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hppritcha/subscriptions","organizations_url":"https://api.github.com/users/hppritcha/orgs","repos_url":"https://api.github.com/users/hppritcha/repos","events_url":"https://api.github.com/users/hppritcha/events{/privacy}","received_events_url":"https://api.github.com/users/hppritcha/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-02T20:42:09Z","updated_at":"2016-03-02T20:43:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openucx/ucx/pulls/661","html_url":"https://github.com/openucx/ucx/pull/661","diff_url":"https://github.com/openucx/ucx/pull/661.diff","patch_url":"https://github.com/openucx/ucx/pull/661.patch"},"body":"Its not safe to use external book keeping to\r\nconditionally call ```GNI_CqGetEvent``` on TX CQs.\r\n\r\nParticularly if GNI SMSG functions\r\nare returning GNI_RC_NOT_DONE, that may imply\r\na depletion of internal CQ entry related resources\r\ninside GNI.  This can prevent things like return\r\nof back credits in one way SMSG traffic.\r\n\r\nThis commit unblocks one of the send-on-way\r\nam tests when using GNI uct.  The test subsequently\r\nfails with some kind of object-not-returned-to-mpool\r\nproblem, but that doesn't appear to be related to\r\nthe gni transport.\r\n\r\n@MattBBaker \r\n\r\nSigned-off-by: Howard Pritchard <howardp@lanl.gov>"},"comment":{"url":"https://api.github.com/repos/openucx/ucx/issues/comments/191420965","html_url":"https://github.com/openucx/ucx/pull/661#issuecomment-191420965","issue_url":"https://api.github.com/repos/openucx/ucx/issues/661","id":191420965,"user":{"login":"mellanox-github","id":9459943,"avatar_url":"https://avatars.githubusercontent.com/u/9459943?v=3","gravatar_id":"","url":"https://api.github.com/users/mellanox-github","html_url":"https://github.com/mellanox-github","followers_url":"https://api.github.com/users/mellanox-github/followers","following_url":"https://api.github.com/users/mellanox-github/following{/other_user}","gists_url":"https://api.github.com/users/mellanox-github/gists{/gist_id}","starred_url":"https://api.github.com/users/mellanox-github/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mellanox-github/subscriptions","organizations_url":"https://api.github.com/users/mellanox-github/orgs","repos_url":"https://api.github.com/users/mellanox-github/repos","events_url":"https://api.github.com/users/mellanox-github/events{/privacy}","received_events_url":"https://api.github.com/users/mellanox-github/received_events","type":"User","site_admin":false},"created_at":"2016-03-02T20:43:02Z","updated_at":"2016-03-02T20:43:02Z","body":"Can one of the admins verify this patch?"}}