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

swss: Add support for dynamic gearbox tuning in orchagent #3437

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

j-mazurek
Copy link

What I did

  • Removed gearbox tx_taps on gearbox init
  • Added serdes attributes for line-side and system-side gearbox tx_taps
  • Added cases in doPortTask within orchagent to set tx_taps on gearbox ports based off serdes attributes read in from APPL_DB

Why I did it

These changes were added as part of the Dynamic Gearbox Tuning feature. They allow orchagent to dynamically set gearbox tunings using new serdes attributes read in from APPL_DB.

How I verified it

  • Set all gearbox tx_taps in port_table to 0
    • Verified that tx_taps had been set by orchagent to 0 on gearboxes via credo shell
    • Set tx_taps in port_table to real values
    • Verified that orchagent set real values again by using credo shell to verify tx_tap values
  • Existing unit tests in test_gearbox.py

* Remove gearbox tx_taps on gearbox init
* Add serdes attributes for line-side and system-side gearbox tx_taps
* Add cases in doPortTask within orchagent to set tx_taps on gearbox
  ports based off serdes attributes read in from APPL_DB

Signed-off-by: Jan Mazurek <[email protected]>
@j-mazurek j-mazurek requested a review from prsunny as a code owner December 20, 2024 03:24
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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