Skip to content

UCT/DC: Adjust DC FHS latency#11498

Open
shasson5 wants to merge 10 commits into
openucx:masterfrom
shasson5:fhs
Open

UCT/DC: Adjust DC FHS latency#11498
shasson5 wants to merge 10 commits into
openucx:masterfrom
shasson5:fhs

Conversation

@shasson5

@shasson5 shasson5 commented May 27, 2026

Copy link
Copy Markdown
Contributor

What?

Adjust DC FHS latency

Why?

Impove perf by better selection of RC/DC switching point

How?

Distinguish AR enabled and forced FHS scenraios and add extra round trip latency

@shasson5 shasson5 added the WIP-DNM Work in progress / Do not review label May 27, 2026
@shasson5 shasson5 marked this pull request as ready for review May 27, 2026 18:42
@shasson5

Copy link
Copy Markdown
Contributor Author

@svc-nvidia-pr-review

@svc-nvidia-pr-review

Copy link
Copy Markdown

🤖 Starting review — findings will be posted here when done.

Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c Outdated
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c Outdated
Comment thread buildlib/tools/check_tls_perf_caps.sh
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
@shasson5

Copy link
Copy Markdown
Contributor Author

@svc-nvidia-pr-review

@svc-nvidia-pr-review

Copy link
Copy Markdown

🤖 Starting review — findings will be posted here when done.

Comment thread buildlib/tools/check_tls_perf_caps.sh
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
@shasson5

Copy link
Copy Markdown
Contributor Author

@svc-nvidia-pr-review

@svc-nvidia-pr-review

Copy link
Copy Markdown

🤖 Starting review — findings will be posted here when done.

Comment thread buildlib/tools/check_tls_perf_caps.sh
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c
Comment thread src/uct/ib/mlx5/dc/dc_mlx5.c Outdated
@openucx openucx deleted a comment from svc-nvidia-pr-review Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WIP-DNM Work in progress / Do not review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants