Add switch ASIC vendor and platform for Nephos #1082

kaiyu22 · 2017-10-26T10:58:14Z

- What I did
Add switch ASIC vendor: Nephos
Add Nephos platform: Ingrasys S9130-32X

- How I did it
Add platform/nephos files
Add platform/nephos/sonic-platform-modules-ingrasys submodule
Add device/ingrasys/x86_64-ingrasys_s9130_32x-r0 files
Add SONiC to support Nephos platform

- How to verify it
To build SONiC installer image and docker images, run the following commands:
make configure PLATFORM=nephos
make target/sonic-nephos.bin
Check system and network feature is worked as well

- Description for the changelog
Add switch ASIC vendor and platform for Nephos

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

Signed-off-by: Sam Yang [email protected]

-What I did Add switch ASIC vendor: Nephos Add Nephos platform: Ingrasys S9130-32X - How I did it Add platform/nephos files Add platform/nephos/sonic-platform-modules-ingrasys submodule Add device/ingrasys/x86_64-ingrasys_s9130_32x-r0 files Add SONiC to support Nephos platform - How to verify it To build SONiC installer image and docker images, run the following commands: make configure PLATFORM=nephos make target/sonic-nephos.bin Check system and network feature is worked as well - Description for the changelog Add switch ASIC vendor and platform for Nephos Signed-off-by: Sam Yang <[email protected]>

lguohan · 2017-10-26T14:31:42Z

platform/nephos/rules.mk

+include $(PLATFORM_PATH)/docker-ptf-nephos.mk
+
+NPX_DIAG = npx_diag
+$(NPX_DIAG)_URL = "ftp://sonic:[email protected]/lib/nephos/npx_diag_0.0.0-20171002"


this is private ip address, we cannot connect to it.

lguohan · 2017-10-26T14:32:10Z

slave.mk

@@ -379,7 +379,7 @@ $(DOCKER_LOAD_TARGETS) : $(TARGET_PATH)/%.gz-load : .platform docker-start $$(TA
 ###############################################################################

 # targets for building installers with base image
-$(addprefix $(TARGET_PATH)/, $(SONIC_INSTALLERS)) : $(TARGET_PATH)/% : .platform onie-image.conf $$(addprefix $(DEBS_PATH)/,$$($$*_DEPENDS)) $$(addprefix $(DEBS_PATH)/,$$($$*_INSTALLS)) $$(addprefix $(FILES_PATH)/,$$($$*_FILES)) $(addprefix $(DEBS_PATH)/,$(INITRAMFS_TOOLS) $(LINUX_KERNEL) $(IGB_DRIVER) $(SONIC_DEVICE_DATA) $(SONIC_UTILS)) $$(addprefix $(TARGET_PATH)/,$$($$*_DOCKERS)) $$(addprefix $(PYTHON_WHEELS_PATH)/,$(SONIC_CONFIG_ENGINE))
+$(addprefix $(TARGET_PATH)/, $(SONIC_INSTALLERS)) : $(TARGET_PATH)/% : .platform onie-image.conf $$(addprefix $(DEBS_PATH)/,$$($$*_DEPENDS)) $$(addprefix $(DEBS_PATH)/,$$($$*_INSTALLS)) $$(addprefix $(FILES_PATH)/,$$($$*_FILES)) $(addprefix $(DEBS_PATH)/,$(INITRAMFS_TOOLS) $(LINUX_KERNEL) $(IGB_DRIVER) $(IXGBE_DRIVER)  $(SONIC_DEVICE_DATA) $(SONIC_UTILS)) $$(addprefix $(TARGET_PATH)/,$$($$*_DOCKERS)) $$(addprefix $(PYTHON_WHEELS_PATH)/,$(SONIC_CONFIG_ENGINE))


extra whitespace.

jleveque · 2017-10-26T17:24:24Z

device/ingrasys/x86_64-ingrasys_s9130_32x-r0/INGRASYS-S9130-32X/port_config.ini

@@ -0,0 +1,33 @@
+# name          lanes             alias             port


Change final column title from port to index.

Refer to https://github.com/Azure/SONiC/wiki/Porting-Guide.

1. Fix private ip to public ip 2. Remove extra whitespace 3. Change final column title from 'port' to 'index' Signed-off-by: Sam Yang <[email protected]>

stcheng

please check the comments

stcheng · 2017-10-27T07:26:18Z

platform/nephos/one-aboot.mk

@@ -0,0 +1,2 @@
+# sonic broadcom one image installer


if only .bin is created, this file is not needed.

thanks, I will update it.

stcheng · 2017-10-27T07:26:30Z

platform/nephos/one-image.mk

@@ -0,0 +1,9 @@
+# sonic broadcom one image installer


change the platform name

thanks, I will update it.

stcheng · 2017-10-27T07:26:58Z

platform/nephos/rules.mk

+include $(PLATFORM_PATH)/docker-syncd-nephos.mk
+include $(PLATFORM_PATH)/docker-syncd-nephos-rpc.mk
+include $(PLATFORM_PATH)/one-image.mk
+include $(PLATFORM_PATH)/one-aboot.mk


aboot is not needed.

thanks, I will update it.

stcheng · 2017-10-27T07:27:22Z

platform/nephos/rules.mk

+NPX_DIAG = npx_diag
+$(NPX_DIAG)_URL = "https://github.com/NephosInc/SONiC/raw/master/sdk/npx_diag_2.0.1-20171020"
+
+DSSERVE = dsserve


is this needed?

Yes, this is needed.
Nephos support the npx_diag utility like bcmcmd to show/debug the ASIC information.

1. Remove aboot related files 2. Rename platform name Signed-off-by: Sam Yang <[email protected]>

4628a5a [vs tests] Refactor fdb_update tests to be consistent with vs test suite (#1185) 9594de0 Enable MIRRORv6 for BFN platform (#1177) a35afac [portsorch] use ingress/egress disable for LAG member enable/disable (#1166) faab3e1 remove the obsoleted platforms build links (#1169) 97abec5 [restore_neighbors.py] build arp packet with correct hwsrc and psrc (#1158) 97a22ca [vs tests] Mark VLAN and warm reboot tests as xfail (#1183) b453797 [Teamdmgrd]Fix "teamd cannot start when port channel was down before warm reboot" issue (#1171) 3732e74 [cfgmgr] clear loopback and vrf in kernel if not warmstart (#1141) 51315c1 [debugcounterorch] Add checks for supported counter types and drop reasons (#1173) 10f091d [vs tests] Disable NAT tests to unblock Jenkins (#1179) 698ca2a [portsorch] Refactor portsorch to use FlexCounterManager to setup port and queue stats (#1170) ea8b1da Merge pull request #1125 from AkhileshSamineni/natorch_changes_in_sonic bbd77fc Merge pull request #1126 from AkhileshSamineni/natsyncd_changes_in_sonic 419d9d2 Fix bug: shellquote the inner string (#1168) d507a93 [aclorch] Validate that provided IN/OUT_PORTS are physical interfaces (#1156) f24081b [AclOrch] add validation for check CRM (#1082) 85036df Merge pull request #1059 from AkhileshSamineni/nat_changes_in_sonic 1eac91e [portsorch] process PortsOrch tables in specific order (#1108) 80e01c0 Teamd :: fix for cleaning up the teamd processes correctly on teamd docker stop (#1159) 77fa5a4 Move away sairedis logrotate from signal handler (#1153) 5ad47af Merge branch 'master' into natorch_changes_in_sonic 982abd6 Fixed compilation issue after forced push. 286ca21 Fixed compilation issue. 3350b85 Added pytest testcases for NAT. 63ebb3a Added platform changes to avoid nexthop traching in OA. f9a8302 Addressed review comments. 946e53d Fixed compilation issue. 3d2b900 Addressed review comments and added 'nat miss' traps in copp json. ff2c78c Added changes to verify the NAT is supported before setting nat zone. 346f59a Addressed review comments. 8246acb Removed debug framework change to avoid compilation error f59ec87 Fixed compilation issue. e8b9dab Added aclorch changes for "do_not_nat" action. 88fe411 Orchagent changes in sonic-swss submodule to support NAT feature. 3ca530c Addressed review comments e4b7724 Addressed Review comments 72750ac Natsyncd changes in sonic-swss submodule to support NAT feature. 2086ad2 Fixed compilation issue after forced push. 3fcfea3 Fix in cleanup. 4028803 Added missed check ed405bf Addressed review comments ae6eb80 Compilation fix to pick proper ACL Table name - CFG_ACL_TABLE_TABLE_NAME d750132 Fix for string compilation issue 904e80b Corrected Makefile.am change. 5b09c41 Removed pytest related files and added those to PR #1125 fd733b8 Removed the Orchagent and Natsyncd related files as those are covered as part of PRs #1125 and #1126 b53c888 Loopback related changes for NAT. 29bb539 Added neighsync changes to support warm reboot for NAT. 4e0d0a1 Changes in sonic-swss submodule to support NAT feature. Signed-off-by: Stepan Blyschak <[email protected]>

- Why I did it Update sonic-sairedis submodule pointer to include the following: 2585a1f [Support gearbox SAI_PORT_ATTR_PORT_SERDES_ID on vs (#1082) fd9bc84 [SAI NAT aging notification (#987) 3fa8f34 [[doc]: Update README.md (#1122) 157e573 [[lgtm] Fix libyang missing in lgtm validation issue (#1135) af80caa Add Voqs to Virtual Switch (#1061) f9008ad [fastboot] fastboot enhancement: Use warm-boot infrastructure for fast-boot (#1100) - How I did it Advance sonic-sairedis pointer Signed-off-by: dprital <[email protected]>

lguohan reviewed Oct 26, 2017

View reviewed changes

jleveque suggested changes Oct 26, 2017

View reviewed changes

Updated patch for PR#1082

2e96bbc

1. Fix private ip to public ip 2. Remove extra whitespace 3. Change final column title from 'port' to 'index' Signed-off-by: Sam Yang <[email protected]>

stcheng suggested changes Oct 27, 2017

View reviewed changes

stcheng added the Enhancement ➕ label Oct 27, 2017

Update some typo

9e93942

1. Remove aboot related files 2. Rename platform name Signed-off-by: Sam Yang <[email protected]>

stcheng approved these changes Oct 27, 2017

View reviewed changes

jleveque approved these changes Oct 28, 2017

View reviewed changes

jleveque merged commit b945010 into sonic-net:201709 Oct 28, 2017

stepanblyschak mentioned this pull request Jan 28, 2020

[swss] update swss pointer #4077

Merged

madhanmellanox pushed a commit to madhanmellanox/sonic-buildimage that referenced this pull request Mar 23, 2020

[AclOrch] add validation for check CRM (sonic-net#1082)

e3af8ff

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add switch ASIC vendor and platform for Nephos #1082

Add switch ASIC vendor and platform for Nephos #1082

kaiyu22 commented Oct 26, 2017

lguohan Oct 26, 2017

lguohan Oct 26, 2017

jleveque Oct 26, 2017

stcheng left a comment

stcheng Oct 27, 2017

kaiyu22 Oct 27, 2017

stcheng Oct 27, 2017

kaiyu22 Oct 27, 2017

stcheng Oct 27, 2017

kaiyu22 Oct 27, 2017

stcheng Oct 27, 2017

kaiyu22 Oct 27, 2017

stcheng Oct 27, 2017

Add switch ASIC vendor and platform for Nephos #1082

Add switch ASIC vendor and platform for Nephos #1082

Conversation

kaiyu22 commented Oct 26, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

stcheng left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment