Skip to content

vpp: support setting interface speed via sai - #1961

Merged
yue-fred-gao merged 4 commits into
sonic-net:masterfrom
yue-fred-gao:set_interface_speed
Aug 4, 2026
Merged

vpp: support setting interface speed via sai#1961
yue-fred-gao merged 4 commits into
sonic-net:masterfrom
yue-fred-gao:set_interface_speed

Conversation

@yue-fred-gao

Copy link
Copy Markdown
Contributor

Description of PR

Summary: support setting interface speed in vpp

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Documentation update
  • Test improvement

Approach

What is the motivation for this PR?

Work item tracking
  • Microsoft ADO (number only):

How did you do it?

  1. extend vpp to support setting interface speed through api via PR: https://gerrit.fd.io/r/c/vpp/+/45800
  2. implement vpp_set_port_speed on top of the new vpp api
  3. hook up SAI_PORT_ATTR_SPEED attribute in sai UpdatePort to vpp_set_port_speed

How did you verify/test it?

Run test with Marvel octeon driver

Any platform specific information?

vpp specific

Documentation

Signed-off-by: Yue Gao <yuega2@cisco.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Yue Gao <yuega2@cisco.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@janpal23 janpal23 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verified end-to-end; interface speed changes between 1G and 10G work as expected with this change.

@mssonicbld

Copy link
Copy Markdown
Collaborator

Cherry-pick PR to msft-202603: Azure/sonic-sairedis.msft#135

@mssonicbld

Copy link
Copy Markdown
Collaborator

This PR has backport request label(s) for branch(es): msft-202603,202605, but is missing required test information. Please make sure you tick the tested branch(es) in the Tested branch section and provide test evidence (e.g., 202603: <test result>) in the Test result section as well in your PR description.

---Powered by SONiC BuildBot

@yejianquan

Copy link
Copy Markdown
Contributor

don't have plan to onboard C0 with 202605, might to go with 202611, we can skip on this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants