Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IB/ADDRESS: pack MTU value for non 4K value - v1.10 #6277

Merged
merged 1 commit into from
Feb 5, 2021

Conversation

hoopoepg
Copy link
Contributor

@hoopoepg hoopoepg commented Feb 5, 2021

What

  • Syncronise PATH MTU value between EP's

Why ?

  • On clusters where combination Connect-X3/Connect-X4+ are used there could be used incorrect values of PATH MTU values which prevents communication between sych systems

How ?

  • Pack MTU value for HCA where non 4096 MTU are used

cherry picked from #6216 as a single commit

- pack MTU value for HCA where non 4096 MTU are used
- added gtest for non-unified MTU
@yosefe
Copy link
Contributor

yosefe commented Feb 5, 2021

@hoopoepg were there any conflicts?

@hoopoepg
Copy link
Contributor Author

hoopoepg commented Feb 5, 2021

no, merged because original PR was merged as 7 commits, most of them are failed in CI.
split it into original commit set?

@yosefe
Copy link
Contributor

yosefe commented Feb 5, 2021

no, merged because original PR was merged as 7 commits, most of them are failed in CI.
split it into original commit set?

No need. After merging the commits, were there any conflicts?

@hoopoepg
Copy link
Contributor Author

hoopoepg commented Feb 5, 2021

no conflicts were found

@shamisp shamisp added the Release Blocker Release Blocker label Feb 5, 2021
@yosefe yosefe merged commit 2b99702 into openucx:v1.10.x Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Blocker Release Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants