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

Add XcvrApi for configuring application 0 #474

Closed
wants to merge 2 commits into from

Conversation

noaOrMlnx
Copy link
Collaborator

Description

When NOS configures the CMIS modules, we need to be aware to a case where some of the lanes are disabled.
According to CMIS spec, in that case, an application 0 is needed to make sure all disabled lanes are zeroed.
This function sets "0" in all lanes before configuring the module, so de-activated lanes will remain with 0.

Motivation and Context

This was done according to CMIS specification document.

How Has This Been Tested?

This addition to CMIS thread was tested with a CMIS active cable, with only 4 active lanes (and 4 de-active).
We made sure the link is up, right places are zeroed, and have traffic.

Additional Information (Optional)

@prgeor
Copy link
Collaborator

prgeor commented Jun 10, 2024

@noaOrMlnx please check this API (decommsion_all_datapaths) which is invoked by CMIS manager thread at the beginning of configuring any speed to put all lanes with 0 application to start with

def decommission_all_datapaths(self):

@noaOrMlnx
Copy link
Collaborator Author

@prgeor thanks, I checked it and indeed this works and solved the issue. I will close 2 relevant PRs.

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