Skip to content

vpp: get interface speed from vpp#1858

Merged
yue-fred-gao merged 2 commits into
sonic-net:masterfrom
yue-fred-gao:get_interface_speed
Apr 20, 2026
Merged

vpp: get interface speed from vpp#1858
yue-fred-gao merged 2 commits into
sonic-net:masterfrom
yue-fred-gao:get_interface_speed

Conversation

@yue-fred-gao

Copy link
Copy Markdown
Contributor

why

currently saivpp doesn't support get interface speed. It returns NOT_IMPLEMENTED

what this PR does

interface speed is available in vpp vl_api_sw_interface_dump_t api. We save it in a map with sw_if_index as key. When syncd queries port speed, get the speed from the map if it is available. Otherwise, returns configured rate or 0 if the port is down.

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

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

@AkeelAli AkeelAli 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.

LGTM. Minor formatting comment.

Comment thread vslib/vpp/vppxlate/SaiVppXlate.c Outdated
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

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

yue-fred-gao and others added 2 commits April 20, 2026 11:32
Signed-off-by: Yue Gao <yuega2@cisco.com>
Adjust indentation

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).

@yue-fred-gao
yue-fred-gao merged commit c619f4c into sonic-net:master Apr 20, 2026
19 checks passed
@mssonicbld

Copy link
Copy Markdown
Collaborator

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

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.

4 participants