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

NDR support and backward compatibility rate support #25

Merged
merged 2 commits into from
Apr 25, 2021

Conversation

ornechemia
Copy link

No description provided.

@ornechemia ornechemia force-pushed the ndr_support branch 4 times, most recently from 4b5c54d to c3435fa Compare April 13, 2021 12:11

return i ? i : IB_MIN_RATE;
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add to header of ib_path_rate_max_12xedr that the function is deprecated and should use
function osm_sa_limit_rate instead.

@ornechemia ornechemia force-pushed the ndr_support branch 3 times, most recently from a061789 to 073423c Compare April 22, 2021 07:43
NDR speed is 106.25Gbps (IB Specification volume 1.5)

Signed-off-by: Or Nechemia <orn@nvidia.com>
New configuration parameter (max_rate_enum) allows to manually limit the
rate of subnet when using old drivers which support up to a limited rate.
The value it takes is Rate Enumeration, defined at IB specification chapter
15.2.5.16.1.

Previously, a boolean parameter (use_original_extended_sa_rates_only) was
used to indicate rate limit of up to 12xEDR. However it cannot be used to
indicate of other rate limitations, thus max_rate_enum is required.

In case use_original_extended_sa_rates_only is TRUE (equivalent to limited
rate enumeration of 18), SA uses the minimum between 18 (up to 12xEDR) and
max_rate_enum as the rate limitation.

Signed-off-by: Or Nechemia <orn@nvidia.com>
@tamirronen tamirronen merged commit ca0a786 into linux-rdma:master Apr 25, 2021
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.

2 participants