Skip to content

Introduce a new role for DPU-NPU Interconnect#92

Closed
vivekrnv wants to merge 3 commits intomasterfrom
ss_int_ports
Closed

Introduce a new role for DPU-NPU Interconnect#92
vivekrnv wants to merge 3 commits intomasterfrom
ss_int_ports

Conversation

@vivekrnv
Copy link
Copy Markdown
Owner

@vivekrnv vivekrnv commented Mar 19, 2024 •
edited
Loading

Why I did it

  • Platform api related to xcvr and sfp should not be called for internal ports of SmartSwitch
  • To avoid doing this from platform agnostic code of SONiC, a new role called 'Dpc' is added, which would mean DPU-NPU Data Port
  • Any new sku which is used as a smartswitch should add this argument to hwsku.json for the ports that fit this category
Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

root@mtvr-leopard-01:/home/admin# show interfaces status
  Interface                            Lanes    Speed    MTU    FEC    Alias    Vlan    Oper    Admin                                             Type    Asym PFC
-----------  -------------------------------  -------  -----  -----  -------  ------  ------  -------  -----------------------------------------------  ----------
  Ethernet0                  0,1,2,3,4,5,6,7     400G   9100    N/A     etp1  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
  Ethernet8            8,9,10,11,12,13,14,15     400G   9100    N/A     etp2  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
 Ethernet16          16,17,18,19,20,21,22,23     400G   9100    N/A     etp3  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
 Ethernet24          24,25,26,27,28,29,30,31     400G   9100    N/A     etp4  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
 Ethernet32          32,33,34,35,36,37,38,39     400G   9100    N/A     etp5  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
 Ethernet40          40,41,42,43,44,45,46,47     400G   9100    N/A     etp6  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
 Ethernet48          48,49,50,51,52,53,54,55     400G   9100    N/A     etp7  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
 Ethernet56          56,57,58,59,60,61,62,63     400G   9100    N/A     etp8  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
 Ethernet64          64,65,66,67,68,69,70,71     400G   9100    N/A     etp9  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
..........................
Ethernet208  208,209,210,211,212,213,214,215     400G   9100    N/A    etp27  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
Ethernet216  216,217,218,219,220,221,222,223     400G   9100    N/A    etp28  routed    down       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
Ethernet224  224,225,226,227,228,229,230,231     400G   9100    N/A    etp29  routed     N/A       up                                DPU-NPU Data Port         N/A
Ethernet232  232,233,234,235,236,237,238,239     400G   9100    N/A    etp30  routed     N/A       up                                DPU-NPU Data Port         N/A
Ethernet240  240,241,242,243,244,245,246,247     400G   9100    N/A    etp31  routed     N/A       up                                DPU-NPU Data Port         N/A
Ethernet248  248,249,250,251,252,253,254,255     400G   9100    N/A    etp32  routed     N/A       up                                DPU-NPU Data Port         N/A

root@mtvr-leopard-01:/home/admin# sfputil show presence
Port         Presence
-----------  ----------
Ethernet0    Present
Ethernet8    Present
Ethernet16   Present
Ethernet24   Present
Ethernet32   Present
Ethernet40   Present
Ethernet48   Present
Ethernet56   Present
Ethernet64   Present
Ethernet72   Present
Ethernet80   Present
Ethernet88   Present
Ethernet96   Present
Ethernet104  Present
Ethernet112  Present
Ethernet120  Present
Ethernet128  Present
Ethernet136  Present
Ethernet144  Present
Ethernet152  Present
Ethernet160  Present
Ethernet168  Present
Ethernet176  Present
Ethernet184  Present
Ethernet192  Present
Ethernet200  Present
Ethernet208  Present
Ethernet216  Present

Nothing related to Dpc ports is populated in the XCVRD tables

root@mtvr-leopard-01:/home/admin# redis-cli -n 6 keys "TRANSCEIVER_INFO*"
 1) "TRANSCEIVER_INFO|Ethernet112"
 2) "TRANSCEIVER_INFO|Ethernet88"
 3) "TRANSCEIVER_INFO|Ethernet160"
 4) "TRANSCEIVER_INFO|Ethernet192"
 5) "TRANSCEIVER_INFO|Ethernet0"
 6) "TRANSCEIVER_INFO|Ethernet72"
 7) "TRANSCEIVER_INFO|Ethernet16"
 8) "TRANSCEIVER_INFO|Ethernet64"
 9) "TRANSCEIVER_INFO|Ethernet40"
10) "TRANSCEIVER_INFO|Ethernet208"
11) "TRANSCEIVER_INFO|Ethernet104"
12) "TRANSCEIVER_INFO|Ethernet152"
13) "TRANSCEIVER_INFO|Ethernet48"
14) "TRANSCEIVER_INFO|Ethernet184"
15) "TRANSCEIVER_INFO|Ethernet176"
16) "TRANSCEIVER_INFO|Ethernet56"
17) "TRANSCEIVER_INFO|Ethernet200"
18) "TRANSCEIVER_INFO|Ethernet128"
19) "TRANSCEIVER_INFO|Ethernet120"
20) "TRANSCEIVER_INFO|Ethernet96"
21) "TRANSCEIVER_INFO|Ethernet216"
22) "TRANSCEIVER_INFO|Ethernet8"
23) "TRANSCEIVER_INFO|Ethernet144"
24) "TRANSCEIVER_INFO|Ethernet32"
25) "TRANSCEIVER_INFO|Ethernet168"
26) "TRANSCEIVER_INFO|Ethernet24"
27) "TRANSCEIVER_INFO|Ethernet136"
28) "TRANSCEIVER_INFO|Ethernet80"

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@vivekrnv vivekrnv requested a review from dgsudharsan as a code owner March 19, 2024 23:05
dgsudharsan
dgsudharsan reviewed Mar 21, 2024
View reviewed changes
@vivekrnv
Copy link
Copy Markdown
Owner Author

vivekrnv commented Mar 21, 2024

@dgsudharsan FYI added this check to the api

if not port.startswith(front_panel_prefix()):
        return False

@vivekrnv vivekrnv closed this Mar 25, 2024
vivekrnv pushed a commit that referenced this pull request Jul 29, 2025
@mssonicbld
[submodule] Update submodule sonic-dash-ha to the latest HEAD automat…
2855448 
…ically (sonic-net#23472)

#### Why I did it
src/sonic-dash-ha
```
* 3d9b38f - (HEAD -> master, origin/master, origin/HEAD) Rename disable to disabled in ha_scope table (#93) (14 hours ago) [yue-fred-gao]
* 01f9bf5 - Add semgrep static analysis tool to find security vulnerabilities. (#90) (25 hours ago) [Mai Bui]
* 4121247 - Use vnet_name from DASH_HA_GLOBAL_CONFIG to create vnet route (#92) (30 hours ago) [yue-fred-gao]
```
#### How I did it
#### How to verify it
#### Description for the changelog
vivekrnv pushed a commit that referenced this pull request Feb 17, 2026
@mssonicbld
[submodule] Update submodule dhcprelay to the latest HEAD automatical…
1ef0d24 
…ly (sonic-net#25509)

#### Why I did it
src/dhcprelay
```
* 8987cbb - (HEAD -> master, origin/master, origin/HEAD) Add .github/copilot-instructions.md for AI-assisted development (#92) (31 hours ago) [rustiqly]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dgsudharsan dgsudharsan dgsudharsan approved these changes

+1 more reviewer

@oleksandrivantsiv oleksandrivantsiv oleksandrivantsiv approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vivekrnv @dgsudharsan @oleksandrivantsiv