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

Celestica innovium support for Midstone100x and SilverstoneX #13230

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

skannan-sonic
Copy link
Contributor

@skannan-sonic skannan-sonic commented Jan 3, 2023

Why I did it

Innovium ASIC is not supported in master branch.
This pull request adds support for Innovium ASIC and along with 2 Celestica platforms [Silverstone-X and Midstone-100X]

How I did it

1.docker-syncd-invm is modified to compile for buster and bullseye from stretch.
2.Added platform silverstone-x/Midstone-100x

How to verify it

  1. Verified the full compilation.
  2. Validated the image in Silverstone-X box
  3. Show interface status and show version
    Interface Lanes Speed MTU FEC Alias Vlan Oper Admin Type Asym PFC

Ethernet0 217,218,219,220,221,222,223,224 400G 9126 rs Eth1 routed down up N/A N/A
Ethernet8 209,210,211,212,213,214,215,216 400G 9126 rs Eth2 routed down up N/A N/A
Ethernet16 201,202,203,204,205,206,207,208 400G 9126 rs Eth3 routed down up N/A N/A
Ethernet24 193 25G 9126 none Eth4 routed down up N/A N/A
Ethernet25 194 25G 9126 none Eth4 routed down down N/A N/A
Ethernet26 195 25G 9126 none Eth4 routed down down N/A N/A
Ethernet27 196 25G 9126 none Eth4 routed down down N/A N/A
Ethernet28 197 25G 9100 none Eth4 routed down down N/A N/A
Ethernet29 198 25G 9126 none Eth4 routed down down N/A N/A
Ethernet30 199 25G 9126 none Eth4 routed down down N/A N/A
Ethernet31 200 25G 9126 none Eth4 routed down up N/A N/A
Ethernet32 185,186,187,188,189,190,191,192 400G 9126 rs Eth5 routed down up N/A N/A
Ethernet40 177 25G 9100 rs Eth6 PortChannel2 up up N/A N/A
Ethernet41 178 25G 9100 rs Eth6 PortChannel2 up up N/A N/A

root@silverstonex:/lib/modules/5.10.0-18-2-amd64/kernel/drivers# show version

SONiC Software Version: SONiC.celestica-innovium-master.10-018ad4365
Distribution: Debian 11.5
Kernel: 5.10.0-18-2-amd64
Build commit: 018ad43
Build date: Mon Nov 21 09:22:36 UTC 2022
Built by: skannan@AZUHPSP02

Platform: x86_64-cel_silverstone-x-r0
HwSKU: Silverstone-x
ASIC: innovium
ASIC Count: 1
Serial Number: R3240F2B121A06GD200038
Model Number: N/A
Hardware Revision: N/A
Uptime: 11:57:16 up 8 min, 1 user, load average: 1.75, 1.11, 0.57
Date: Mon 21 Nov 2022 11:57:16

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

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

Description for the changelog

Innovium ASIC is not supported in master branch.
This pull request adds support for Innovium ASIC and along with 2 Celestica platforms [Silverstone-X and Midstone-100X]

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

None

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

@skannan-sonic skannan-sonic marked this pull request as ready for review January 3, 2023 06:17
@skannan-sonic skannan-sonic requested a review from lguohan as a code owner January 3, 2023 06:17
@zhangyanzhao
Copy link
Collaborator

@kselvaraj2 202111 is the earliest release to support this platform, technically, all later releases 202205, 202211, master should be supported as well. Can you please update this PR by adding 202211? Thanks.

@skannan-sonic
Copy link
Contributor Author

Thanks @zhangyanzhao updated 202211 along with other releases.

@skannan-sonic
Copy link
Contributor Author

Hi @zhangyanzhao,@yxieca,@StormLiangMS,

Gentle remainder to review this code request.

Thanks,
Kannan.S

@lguohan lguohan added the device label Sep 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants