Skip to content

UCT/GGA: Backport fix #11507 to 1.21.x#11519

Open
ZhenlongMa wants to merge 1 commit into
openucx:v1.21.xfrom
ZhenlongMa:backport-fix-to-1.21.x
Open

UCT/GGA: Backport fix #11507 to 1.21.x#11519
ZhenlongMa wants to merge 1 commit into
openucx:v1.21.xfrom
ZhenlongMa:backport-fix-to-1.21.x

Conversation

@ZhenlongMa
Copy link
Copy Markdown
Contributor

What?

Port changes #11507 to 1.21.x branch to support NIXL. The changes have been approved and merged into the master branch.

Why?

The motivation of the changes can be seen in #11507.

The changes are to support NIXL, which currently uses version 1.21.x.

How?

Cherry-picked the merged changes from #11507 onto the 1.21.x branch.

@ZhenlongMa ZhenlongMa changed the title UCT: Add option to disable gga (#11507) UCT/GGA: Backport fix #11507 to 1.21.x Jun 4, 2026
@ZhenlongMa
Copy link
Copy Markdown
Contributor Author

I initially submitted a PR trying to merge the commit into branch v1.20.x by mistake. Then I closed it and recommitted this new PR. But this PR shows that perf checks were skipped "due to merge conflicts". When I check the details, it jumps to the previous PR. I am confused. I don't know why this happens and where the conflicts are.

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 4 pipeline(s), but failed to run 1 pipeline(s).

@Alexey-Rivkin
Copy link
Copy Markdown
Contributor

@ZhenlongMa Could you force-push an empty amend commit?

* UCT/IB/MLX5: Add option to disable GGA transport

Add a --with-gga/--without-gga configure option to control building the GGA (DPU) MLX5 transport, and gate gga/gga_mlx5.c on a new HAVE_TL_GGA conditional instead of HAVE_MLX5_MMO. Also add the author to AUTHORS.

Signed-off-by: Zhenlong Ma <zhenlongm@nvidia.com>

* UCT/IB/MLX5: Address review on GGA configure option

Signed-off-by: Zhenlong Ma <zhenlongm@nvidia.com>

* UCT/IB/MLX5: Change with_gga checking location

Signed-off-by: Zhenlong Ma <zhenlongm@nvidia.com>

---------

Signed-off-by: Zhenlong Ma <zhenlongm@nvidia.com>
(cherry picked from commit 10de7c0)
@ZhenlongMa ZhenlongMa force-pushed the backport-fix-to-1.21.x branch from 3f374d1 to c8e2f84 Compare June 4, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants