Skip to content

[action] [PR:1858] vpp: get interface speed from vpp#123

Merged
mssonicbld merged 1 commit into
Azure:202603from
mssonicbld:cherry/msft-202603/1858
Apr 21, 2026
Merged

[action] [PR:1858] vpp: get interface speed from vpp#123
mssonicbld merged 1 commit into
Azure:202603from
mssonicbld:cherry/msft-202603/1858

Conversation

@mssonicbld

Copy link
Copy Markdown
Collaborator

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.

Signed-off-by: Sonic Build Admin sonicbld@microsoft.com

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

Signed-off-by: Sonic Build Admin <sonicbld@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator Author

Original PR: sonic-net/sonic-sairedis#1858

@mssonicbld

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@mssonicbld
mssonicbld merged commit 6415738 into Azure:202603 Apr 21, 2026
5 checks passed
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.

1 participant