{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":151767322,"defaultBranch":"master","name":"aws-ofi-nccl","ownerLogin":"aws","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2018-10-05T19:17:39.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/2232217?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1713470233.0","currentOid":""},"activityList":{"items":[{"before":"332ade01f5236fe4114867c31bb5ec78f0feaeb3","after":"309d83495910db22a118d14f06f053d98bfc468c","ref":"refs/heads/master","pushedAt":"2024-05-22T18:46:43.000Z","pushType":"pr_merge","commitsCount":10,"pusher":{"login":"aws-nslick","name":"Nicholas Sielicki","path":"/aws-nslick","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/145174695?s=80&v=4"},"commit":{"message":"ci(github): add an al2023 build\n\nAdd an al2023 build (#282), alongside the recently added al2 build. This\nalso runs on codebuild \"self-hosted\" runners rather than on the gh\nhosted runners. To avoid installing the full toolkit, disable tests in\nthat build (see related commit b9c46d2).","shortMessageHtmlLink":"ci(github): add an al2023 build"}},{"before":"42edb20e89acbf4f81873ce0e70c38e9e12cd9c3","after":"b9a4f7a7e61bf9a5023a2f3f08ba31037577ca43","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-05-22T18:46:34.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"bwbarrett","name":"Brian Barrett","path":"/bwbarrett","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/15623965?s=80&v=4"},"commit":{"message":"[v1.9.x-aws].ci/aws: Remove unstable p3dn tests from Jenkins\n\nSigned-off-by: Seth Zegelstein \nSigned-off-by: Nicholas Sielicki \n(cherry picked from commit 78320f19fcddd3da4f1e6aaf39e2b1d34f37d0d8)","shortMessageHtmlLink":"[v1.9.x-aws].ci/aws: Remove unstable p3dn tests from Jenkins"}},{"before":"4c296bf474494205db3af490b40adc68fe0186d1","after":"332ade01f5236fe4114867c31bb5ec78f0feaeb3","ref":"refs/heads/master","pushedAt":"2024-05-21T06:47:13.000Z","pushType":"pr_merge","commitsCount":5,"pusher":{"login":"aws-nslick","name":"Nicholas Sielicki","path":"/aws-nslick","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/145174695?s=80&v=4"},"commit":{"message":"tuner/model: refactor, make better decisions\n\nmake this code cleaner in some respects, and messier in most others.\n\nFrom data collected internally, the decisions made through here are\noptimal for p5. Ring is fundamentally modeled incorrect here. Still\nneeded in the future: refactor this further to allow for parameters to\nbe specified per-platform, fix ring behavior on certain boundaries.\n\nSigned-off-by: Nicholas Sielicki ","shortMessageHtmlLink":"tuner/model: refactor, make better decisions"}},{"before":"6b52c23ceb42d6fa90552e38883b5981af2e8b10","after":"4c296bf474494205db3af490b40adc68fe0186d1","ref":"refs/heads/master","pushedAt":"2024-05-20T09:32:49.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"aws-nslick","name":"Nicholas Sielicki","path":"/aws-nslick","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/145174695?s=80&v=4"},"commit":{"message":"ci(github): add al2 build\n\nusing a codebuild self-hosted action runner, add a separate job for al2\nbuilds.","shortMessageHtmlLink":"ci(github): add al2 build"}},{"before":"12e1bb71ad01852ac78b9077a194b0e9dfc3c3f2","after":"42edb20e89acbf4f81873ce0e70c38e9e12cd9c3","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-05-17T14:15:23.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":"[v1.9.x-aws].ci/aws: Wait until ODCR has enough capacity to run\n\nThe p3dn ODCR is not immediately refilling the available instance count\nafter all the instances have been terminated. This has put a race\ncondition in our code which sometimes causes us to get ICE'ed. Attempt\nto fix the ICE by waiting till the ODCR has the required capacity before\nattempting to launch instances with it.\n\nSigned-off-by: Seth Zegelstein \n(cherry picked from commit 6b52c23ceb42d6fa90552e38883b5981af2e8b10)","shortMessageHtmlLink":"[v1.9.x-aws].ci/aws: Wait until ODCR has enough capacity to run"}},{"before":"31d2c083f8009a836bda95ff712fc1819e238a77","after":"6b52c23ceb42d6fa90552e38883b5981af2e8b10","ref":"refs/heads/master","pushedAt":"2024-05-17T06:43:30.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Wait until ODCR has enough capacity to run\n\nThe p3dn ODCR is not immediately refilling the available instance count\nafter all the instances have been terminated. This has put a race\ncondition in our code which sometimes causes us to get ICE'ed. Attempt\nto fix the ICE by waiting till the ODCR has the required capacity before\nattempting to launch instances with it.\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Wait until ODCR has enough capacity to run"}},{"before":"46202cfdd5e89cd26fa0f2ae4412de5d91d43fd5","after":"12e1bb71ad01852ac78b9077a194b0e9dfc3c3f2","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-05-16T21:10:32.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":"[v1.9.x-aws].ci/aws: Move p5 ODCR to af-south-1\n\nSigned-off-by: Seth Zegelstein \n(cherry picked from commit 31d2c083f8009a836bda95ff712fc1819e238a77)","shortMessageHtmlLink":"[v1.9.x-aws].ci/aws: Move p5 ODCR to af-south-1"}},{"before":"a8e549fe4d0b5aae0cfd4dfacb0a01ea19c40ecf","after":"31d2c083f8009a836bda95ff712fc1819e238a77","ref":"refs/heads/master","pushedAt":"2024-05-16T21:05:57.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Move p5 ODCR to af-south-1\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Move p5 ODCR to af-south-1"}},{"before":"96bef706ec547d3896dfb1bb0af042d37b0afb8d","after":"46202cfdd5e89cd26fa0f2ae4412de5d91d43fd5","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-05-16T16:07:51.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":"[v1.9.x-aws].ci/aws: Kill all clusters of same instance type/region on lock acquisition\n\nSigned-off-by: Seth Zegelstein \n(cherry picked from commit a8e549fe4d0b5aae0cfd4dfacb0a01ea19c40ecf)","shortMessageHtmlLink":"[v1.9.x-aws].ci/aws: Kill all clusters of same instance type/region o…"}},{"before":"87090771dfcfcaf50bb56a20a8932f03ee4505ac","after":"a8e549fe4d0b5aae0cfd4dfacb0a01ea19c40ecf","ref":"refs/heads/master","pushedAt":"2024-05-16T04:20:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Kill all clusters of same instance type/region on lock acquisition\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Kill all clusters of same instance type/region on lock acqui…"}},{"before":"8158c2b242460a89dd83cbf378aaf0e0a828e810","after":"87090771dfcfcaf50bb56a20a8932f03ee4505ac","ref":"refs/heads/master","pushedAt":"2024-05-16T01:09:27.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Add p3dn testing to Jenkins\n\nAdds ubuntu2004, and ubuntu2204 p3dn testing.\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Add p3dn testing to Jenkins"}},{"before":"3d88b88bdef0de37b2ac1d13e23d27487b3a7c17","after":"8158c2b242460a89dd83cbf378aaf0e0a828e810","ref":"refs/heads/master","pushedAt":"2024-05-15T04:45:10.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"aws-nslick","name":"Nicholas Sielicki","path":"/aws-nslick","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/145174695?s=80&v=4"},"commit":{"message":"fix: lttng: fix include define\n\nSigned-off-by: Nicholas Sielicki ","shortMessageHtmlLink":"fix: lttng: fix include define"}},{"before":"2369bc6ab991bdad7925eac60bdf08a9dffbb051","after":"96bef706ec547d3896dfb1bb0af042d37b0afb8d","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-05-11T00:09:59.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":"[v1.9.x-aws].ci/aws: Compile aws-ofi-nccl in debug\n\nCompile aws-ofi-nccl in debug to enable asserts, and make the CI more\neffective at catching bugs.\n\nSigned-off-by: Seth Zegelstein \n(cherry picked from commit 3d88b88bdef0de37b2ac1d13e23d27487b3a7c17)","shortMessageHtmlLink":"[v1.9.x-aws].ci/aws: Compile aws-ofi-nccl in debug"}},{"before":"69e53b025ea3d75231d210c7360934248bd14476","after":"3d88b88bdef0de37b2ac1d13e23d27487b3a7c17","ref":"refs/heads/master","pushedAt":"2024-05-10T18:45:15.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Compile aws-ofi-nccl in debug\n\nCompile aws-ofi-nccl in debug to enable asserts, and make the CI more\neffective at catching bugs.\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Compile aws-ofi-nccl in debug"}},{"before":"397d42a2d453e8b489438fa637d1dd39ddf3689c","after":"69e53b025ea3d75231d210c7360934248bd14476","ref":"refs/heads/master","pushedAt":"2024-05-10T04:18:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rauteric","name":"Eric Raut","path":"/rauteric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/10216922?s=80&v=4"},"commit":{"message":"fix(lttng): fix LTTNG support in master branch\n\n* m4: typo: `check_pkg_found` => `check_pkg_define`\n* tracepoint.c: typo in include path\n* tracing_impl/lttng.h: Fix header guards as required by LTTNG\n\nLTTNG requires that the header be included multiple times in some cases.\nA sample header shown in\nhttps://lttng.org/man/3/lttng-ust/v2.13/#doc-creating-tp. The\nLTTNG_UST_TRACEPOINT_HEADER_MULTI_READ macro should override the header\nguard.\n\nSigned-off-by: Eric Raut ","shortMessageHtmlLink":"fix(lttng): fix LTTNG support in master branch"}},{"before":"0f40297b31246c09baf938b25e1d671b23ab053f","after":"397d42a2d453e8b489438fa637d1dd39ddf3689c","ref":"refs/heads/master","pushedAt":"2024-05-10T01:51:57.000Z","pushType":"pr_merge","commitsCount":3,"pusher":{"login":"AmedeoSapio","name":"Amedeo Sapio","path":"/AmedeoSapio","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/10835281?s=80&v=4"},"commit":{"message":"rdma: typedef msg_type to uint16\n\nThe message type is a enum that goes on the wire in conn/conn_resp/ctrl messages.\nFor this reason we want the datatype size to be fixed, so we can typedef\nnccl_ofi_rdma_msg_type_t to uint16_t rather than explictly assign the enum to a\nuint16_t.\n\nSigned-off-by: Amedeo Sapio ","shortMessageHtmlLink":"rdma: typedef msg_type to uint16"}},{"before":"95dd1e4735199e77ece3f0569d613299303ae180","after":"2369bc6ab991bdad7925eac60bdf08a9dffbb051","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-05-03T12:48:50.000Z","pushType":"pr_merge","commitsCount":4,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":"[v1.9.x-aws] .ci/aws: Move static config parameter to config file\n\nSigned-off-by: Seth Zegelstein \n(cherry picked from commit 0f40297b31246c09baf938b25e1d671b23ab053f)","shortMessageHtmlLink":"[v1.9.x-aws] .ci/aws: Move static config parameter to config file"}},{"before":"d6a87b18cc332cb6c602d5520c8555bda740d8b7","after":"0f40297b31246c09baf938b25e1d671b23ab053f","ref":"refs/heads/master","pushedAt":"2024-05-02T22:44:33.000Z","pushType":"pr_merge","commitsCount":2,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Move static config parameter to config file\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Move static config parameter to config file"}},{"before":"cc4d8372d20ad244a292113b321db7f6ee3e0864","after":"d6a87b18cc332cb6c602d5520c8555bda740d8b7","ref":"refs/heads/master","pushedAt":"2024-05-02T22:08:56.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Add p5 testing to Jenkins\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Add p5 testing to Jenkins"}},{"before":"2c191c970ea518d7d95b95cc6bff7662211b52d8","after":"cc4d8372d20ad244a292113b321db7f6ee3e0864","ref":"refs/heads/master","pushedAt":"2024-04-30T04:18:17.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"a-szegel","name":"Seth Zegelstein","path":"/a-szegel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/97712042?s=80&v=4"},"commit":{"message":".ci/aws: Add Jenkins p4d CI tests\n\nSigned-off-by: Seth Zegelstein ","shortMessageHtmlLink":".ci/aws: Add Jenkins p4d CI tests"}},{"before":"2fe67477eac261a37dc2bf8e1e7b404b39137181","after":"95dd1e4735199e77ece3f0569d613299303ae180","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-04-24T16:36:52.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rauteric","name":"Eric Raut","path":"/rauteric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/10216922?s=80&v=4"},"commit":{"message":"1.9.x: Bump version to 1.9.2a1-aws\n\nWe've already made changes to this branch so it is no longer 1.9.1.\n\nSigned-off-by: Eric Raut ","shortMessageHtmlLink":"1.9.x: Bump version to 1.9.2a1-aws"}},{"before":"ed1861ee22c03ee10d11ac19449a169cfcfd61f5","after":"2fe67477eac261a37dc2bf8e1e7b404b39137181","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-04-19T18:15:42.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rajachan","name":"Raghu Raja","path":"/rajachan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/742736?s=80&v=4"},"commit":{"message":"fix(tuner): remove spurious deref from context arg\n\nThe context parameter to `nccl_ofi_tuner_get_coll_info` had an erroneous\ndereference.\n\nSigned-off-by: Eric Raut \n(cherry picked from commit 4c9a063e2a16c01e29ad08e9e83180d24d7ea5dc)","shortMessageHtmlLink":"fix(tuner): remove spurious deref from context arg"}},{"before":"4c9a063e2a16c01e29ad08e9e83180d24d7ea5dc","after":"2c191c970ea518d7d95b95cc6bff7662211b52d8","ref":"refs/heads/master","pushedAt":"2024-04-18T20:03:10.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rajachan","name":"Raghu Raja","path":"/rajachan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/742736?s=80&v=4"},"commit":{"message":"rdma: (fix) do not use cq_entry.len in send completions\n\nAccording to libfabric specs, the len field in a completion entry\nonly applies to completed receive operations. We were using it for send\ncompletions as well, which is currently working in the EFA provider,\nbut there is no guarantee that this will be true in the future and for\nother providers.\n\nSigned-off-by: Amedeo Sapio ","shortMessageHtmlLink":"rdma: (fix) do not use cq_entry.len in send completions"}},{"before":"70d8109200adbf8025fd5332962dc8b3e11f043d","after":null,"ref":"refs/heads/simple","pushedAt":"2024-04-18T19:57:13.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"aws-nslick","name":"Nicholas Sielicki","path":"/aws-nslick","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/145174695?s=80&v=4"}},{"before":null,"after":"70d8109200adbf8025fd5332962dc8b3e11f043d","ref":"refs/heads/simple","pushedAt":"2024-04-18T19:54:29.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"aws-nslick","name":"Nicholas Sielicki","path":"/aws-nslick","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/145174695?s=80&v=4"},"commit":{"message":"feat(rdma): delay fi_domain/fi_fabric calls\n\nlazily initializing here allows affinity to ensure handles are backed by\nlocal memory.","shortMessageHtmlLink":"feat(rdma): delay fi_domain/fi_fabric calls"}},{"before":"a3aea9ee5af6da9d382c292a51228a3abc1892ec","after":"4c9a063e2a16c01e29ad08e9e83180d24d7ea5dc","ref":"refs/heads/master","pushedAt":"2024-04-18T17:25:53.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rajachan","name":"Raghu Raja","path":"/rajachan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/742736?s=80&v=4"},"commit":{"message":"fix(tuner): remove spurious deref from context arg\n\nThe context parameter to `nccl_ofi_tuner_get_coll_info` had an erroneous\ndereference.\n\nSigned-off-by: Eric Raut ","shortMessageHtmlLink":"fix(tuner): remove spurious deref from context arg"}},{"before":"1006b3f0ec771781fd58790d6e200c86edf11f85","after":"a3aea9ee5af6da9d382c292a51228a3abc1892ec","ref":"refs/heads/master","pushedAt":"2024-04-17T03:39:21.000Z","pushType":"pr_merge","commitsCount":5,"pusher":{"login":"rauteric","name":"Eric Raut","path":"/rauteric","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/10216922?s=80&v=4"},"commit":{"message":"feat(tracing): add nvtx provider\n\nHook nvtx on existing lttng macros.\n\nWe figured out how to structure this in a way that\naligns the required usages of nvtx with cases\nlike NCCL_OFI_TRACE_SEND_WRITE_SEG COMPLETE/START. We use the NVTX\nstart/end API for ranges, and mark API for events.\n\nOnly supports RDMA protocol for now, SENDRECV protocol NVTX support will\nbe added in the future.\n\nSigned-off-by: Eric Raut ","shortMessageHtmlLink":"feat(tracing): add nvtx provider"}},{"before":"398b8537804ec30ee580218665b5be5908638720","after":"1006b3f0ec771781fd58790d6e200c86edf11f85","ref":"refs/heads/master","pushedAt":"2024-04-15T19:01:05.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"aws-nslick","name":"Nicholas Sielicki","path":"/aws-nslick","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/145174695?s=80&v=4"},"commit":{"message":"Fix log message for OFI_NCCL_DOMAIN_PER_THREAD setting.\n\nThe OFI_NCCL_DOMAIN_PER_THREAD variable and platform setting determines whether\nthe domain is created per process or per thread, but the associated print message\nreverses the output. Correct that, so that the correct relationship is printed in\nthe logs.","shortMessageHtmlLink":"Fix log message for OFI_NCCL_DOMAIN_PER_THREAD setting."}},{"before":"696975669c73ec5cbbb2af480ab973af35c3d2cc","after":"ed1861ee22c03ee10d11ac19449a169cfcfd61f5","ref":"refs/heads/v1.9.x-aws","pushedAt":"2024-04-14T04:21:38.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rajachan","name":"Raghu Raja","path":"/rajachan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/742736?s=80&v=4"},"commit":{"message":"rdma: (fix) memory leak: don't alloc `recv_conn_resp_req` on EAGAIN\n\nMove call to `prepare_recv_conn_resp_req` before entry point of\n`COMM_SEND_CONN` stage.\n\nThis resolves a potential memory leak where `prepare_recv_conn_resp_req`\nis called multiple times on EAGAIN.\n\nAlso fixup a similar case in `accept`, although there was no memory\nallocation there so no leak.\n\nSigned-off-by: Eric Raut \n(cherry picked from commit 398b8537804ec30ee580218665b5be5908638720)","shortMessageHtmlLink":"rdma: (fix) memory leak: don't alloc recv_conn_resp_req on EAGAIN"}},{"before":"b62f8997dc8571dc286ae45b076f470b6fedc964","after":"398b8537804ec30ee580218665b5be5908638720","ref":"refs/heads/master","pushedAt":"2024-04-13T03:24:47.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rajachan","name":"Raghu Raja","path":"/rajachan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/742736?s=80&v=4"},"commit":{"message":"rdma: (fix) memory leak: don't alloc `recv_conn_resp_req` on EAGAIN\n\nMove call to `prepare_recv_conn_resp_req` before entry point of\n`COMM_SEND_CONN` stage.\n\nThis resolves a potential memory leak where `prepare_recv_conn_resp_req`\nis called multiple times on EAGAIN.\n\nAlso fixup a similar case in `accept`, although there was no memory\nallocation there so no leak.\n\nSigned-off-by: Eric Raut ","shortMessageHtmlLink":"rdma: (fix) memory leak: don't alloc recv_conn_resp_req on EAGAIN"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEUUYepgA","startCursor":null,"endCursor":null}},"title":"Activity · aws/aws-ofi-nccl"}