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

Is the branch, 201811 (Broadcom SAI 3.3 GA release) supporting the Broadcom TD3 chip yet? #2370

Closed
pollyhsu2git opened this issue Dec 10, 2018 · 4 comments

Comments

@pollyhsu2git
Copy link
Contributor

pollyhsu2git commented Dec 10, 2018

Description
Based on the Broadcom SAI release note, the Broadcom SAI 3.3.X.X supports the Broadcom TD3 chp. However, based on our testing here on the branch, 201811, it seems NOT

We just want to clarify that:

  1. Whether SONiC has any plan to support the Broadcom TD3 chip?
  2. If YES, when will it be?

Steps to reproduce the issue:

  1. git clone the SONiC code base, and checkout the branch, 201811.
  2. Do the porting of our TD3 model (AS7726-32X) based on the SONiC porting guide.
  3. The syncd initialized failed with the following error messages:

Describe the results you received:
Nov 3 17:32:36.851910 sonic ERR syncd#syncd: sai_api_initialize:105 BRCM SAI ver: [3.3.3.1], OCP SAI ver: [1.3.3], SDK ver: [6.5.13]
Nov 3 17:32:39.041327 sonic INFO syncd#supervisord: syncd PCI unit 0: Dev 0xb870, Rev 0x01, Chip BCM56870_A0, Driver BCM56870_A0#015
Nov 3 17:32:39.041327 sonic INFO syncd#supervisord: syncd #15#015
Nov 3 17:32:39.041327 sonic INFO syncd#supervisord: syncd Initializing platform#015#015
Nov 3 17:32:39.148944 sonic INFO syncd#supervisord: syncd SOC unit 0 attached to PCI device BCM56870_A0#015
Nov 3 17:32:39.152045 sonic INFO syncd#supervisord: syncd Boot flags: Cold boot#015
Nov 3 17:32:39.153589 sonic INFO syncd#supervisord: syncd rc: unit 0 device BCM56870_A0#015
Nov 3 17:32:39.971735 sonic INFO syncd#supervisord: syncd 0:_cmd_cancun_file_load: Fail to open/load file bcm56870_a0_cmh.pkg. (Invalid parameter)#15
Nov 3 17:32:39.971735 sonic INFO syncd#supervisord: syncd 0:_cmd_cancun_file_load: Fail to open/load file bcm56870_a0_cch.pkg. (Invalid parameter)#15
Nov 3 17:32:39.971735 sonic INFO syncd#supervisord: syncd 0:_cmd_cancun_file_load: Fail to open/load file bcm56870_a0_ceh.pkg. (Invalid parameter)#15
Nov 3 17:32:40.003190 sonic INFO syncd#supervisord: syncd 0:soc_cancun_cch_reg_set: CCH file is not loaded#015
Nov 3 17:32:40.011619 sonic INFO syncd#supervisord: syncd 0:_cmd_cancun_file_load: Fail to open/load file bcm56870_a0_cih.pkg. (Invalid parameter)#15
....
root@sonic:/# cd /
root@sonic:/# find . -name *.pkg
root@sonic:/#

Describe the results you expected:

  1. syncd is initialized successfully.
  2. swss is initialized successfully
  3. the port / interface is up and running

Additional information you deem important (e.g. issue happens only occasionally):

root@sonic:/home/admin# show platform summary
Platform: x86_64-accton_as7726_32x-r0
HwSKU: Accton-AS7726-32X
ASIC: broadcom
root@sonic:/home/admin# show ver
SONiC Software Version: SONiC.201811.0-dirty-20181210.140616
Distribution: Debian 9.6
Kernel: 4.9.0-7-amd64
Build commit: ff9eb73
Build date: Mon Dec 10 06:10:31 UTC 2018
Built by: polly_hsu@Tainan-Linux-216

**Attach debug file `sudo generate_dump`:**

sonic_dump_sonic_20161103_174424.tar.gz

@pollyhsu2git pollyhsu2git changed the title Is the branch, 201811 (Broadcom SAI 3.3 GA release) on the Broadcom platform supporting the Broadcom TD3 chip yet? Is the branch, 201811 (Broadcom SAI 3.3 GA release) supporting the Broadcom TD3 chip yet? Dec 10, 2018
@PrabhuSreenivasan
Copy link
Contributor

Hi,
Below logs suggest cancun files are missing and that could be root cause of your issue. Are they available on SONiC? If not, you may want to get in touch with Broadcom marketing.

Nov 3 17:32:39.971735 sonic INFO syncd#supervisord: syncd 0:_cmd_cancun_file_load: Fail to open/load file bcm56870_a0_cmh.pkg. (Invalid parameter)#15
Nov 3 17:32:39.971735 sonic INFO syncd#supervisord: syncd 0:_cmd_cancun_file_load: Fail to open/load file bcm56870_a0_cch.pkg. (Invalid parameter)#15
Nov 3 17:32:39.971735 sonic INFO syncd#supervisord: syncd 0:_cmd_cancun_file_load: Fail to open/load file bcm56870_a0_ceh.pkg. (Invalid parameter)#15

@pollyhsu2git
Copy link
Contributor Author

pollyhsu2git commented Dec 13, 2018

@PrabhuSreenivasan
Um, On SONiC, the Broadcom SAI 3.3 GA release is built / released by MS co-working with Broadcom. We are trying to clarify whether the Broadcom TD3 chip family will be supported on SONiC (MS / Broadcom) by high-lighting the root cause upon our debugging.

@Peter5Lin
Copy link
Contributor

Peter5Lin commented Feb 11, 2019

Hi
I'm porting Broadcom TD3 in SAI 3.3.4.3-1 and get some error message after port was inserted

Feb 11 01:31:29.718115 sonic INFO syncd#supervisord: ledinit 0:soc_cmicx_led_link_status: Led msg id 0x0 send failed, Error Code -3#015
Feb 11 01:31:30.980540 sonic INFO syncd#supervisord: ledinit 0:soc_iproc_data_send_wait: No response for msg 0#015
Feb 11 01:31:30.980683 sonic INFO syncd#supervisord: ledinit 0:soc_cmicx_led_link_status: Led msg id 0x0 send failed, Error Code -3#015
Feb 11 01:31:32.239502 sonic INFO syncd#supervisord: ledinit 0:soc_iproc_data_send_wait: No response for msg 4#015
Feb 11 01:31:32.239502 sonic INFO syncd#supervisord: ledinit 0:soc_cmicx_led_speed: Led msg id 0x4 send failed, Error Code -3#015
Feb 11 01:31:32.245943 sonic INFO syncd#supervisord: ledinit #15#015
Feb 11 01:31:32.246061 sonic INFO syncd.sh[2265]: 2019-02-11 01:31:32,245 INFO exited: ledinit (exit status 0; expected)

I checked M0 firmware was loaded

Feb 11 01:30:48.090761 sonic INFO syncd#supervisord: ledinit /sonic/platform/led_proc_init.soc#015#015
Feb 11 01:30:48.104493 sonic INFO syncd#supervisord: ledinit Loading M0 Firmware located at /usr/share/sonic/hwsku/led-auto.bin#015

Have I met the same issue? or I'm missing some configuration need to set?

@yxieca
Copy link
Contributor

yxieca commented Sep 19, 2019

No. SAI 3.3 doesn't have TD3 support. Sorry for taking so long to answer.

@yxieca yxieca closed this as completed Sep 19, 2019
dprital added a commit to dprital/sonic-buildimage that referenced this issue Jul 7, 2022
Update sonic-swss submodule pointer to include the following:
* Port configuration incremental update support ([sonic-net#2305](sonic-net/sonic-swss#2305))
* [VS Test] Skip failing subport tests ([sonic-net#2370](sonic-net/sonic-swss#2370))
* [teammgr]: Waiting MACsec ready before doLagMemberTask ([sonic-net#2286](sonic-net/sonic-swss#2286))
* [vnetorch] [vxlanorch] fix a set of memory usage issues ([sonic-net#2352](sonic-net/sonic-swss#2352))
* [tests] [asan] add graceful stop flag ([sonic-net#2347](sonic-net/sonic-swss#2347))
* [asan] suppress the static variable leaks ([sonic-net#2354](sonic-net/sonic-swss#2354))
* Add support for IP interface loopback action ([sonic-net#2307](sonic-net/sonic-swss#2307))
* [orchagent]: srv6orch support for uSID ([sonic-net#2335](sonic-net/sonic-swss#2335))

Signed-off-by: dprital <drorp@nvidia.com>
liat-grozovik pushed a commit that referenced this issue Jul 14, 2022
Update sonic-swss submodule pointer to include the following:
* Port configuration incremental update support ([#2305](sonic-net/sonic-swss#2305))
* [VS Test] Skip failing subport tests ([#2370](sonic-net/sonic-swss#2370))
* [teammgr]: Waiting MACsec ready before doLagMemberTask ([#2286](sonic-net/sonic-swss#2286))
* [vnetorch] [vxlanorch] fix a set of memory usage issues ([#2352](sonic-net/sonic-swss#2352))
* [tests] [asan] add graceful stop flag ([#2347](sonic-net/sonic-swss#2347))
* [asan] suppress the static variable leaks ([#2354](sonic-net/sonic-swss#2354))
* Add support for IP interface loopback action ([#2307](sonic-net/sonic-swss#2307))
* [orchagent]: srv6orch support for uSID ([#2335](sonic-net/sonic-swss#2335))

Signed-off-by: dprital <drorp@nvidia.com>
vivekrnv pushed a commit to vivekrnv/sonic-buildimage that referenced this issue Aug 26, 2022
@zbud-msft zbud-msft mentioned this issue Sep 15, 2022
7 tasks
theasianpianist added a commit to theasianpianist/sonic-buildimage that referenced this issue Oct 5, 2022
Include:

df92fb7 Improve verbosity level and provide more info in the log (sonic-net#2472)
e81ed20 [intfmgr]: Enable `accept_untracked_na` kernel param (sonic-net#2436)
24d29f1 [orchdaemon]: Fixed sairedis record file rotation (sonic-net#2299)
b8ee07d [build] add missing package libyang-dev in lgtm.yml (sonic-net#2475)
e46dd29 [crm] Fix issue with continues EXCEEDED and CLEAR logs for ACL group/table counters (sonic-net#2463)
b61d24c [doc]: Update README.md (sonic-net#2456)
b9ade5d [orchagent] Fix issue: ip prefix shall be inited even if VRF/VNET is not ready (sonic-net#2461)
f0f1eb4 Revert "[counters] Improve performance by polling only configured ports buffer queue/pg counters (sonic-net#2360)" (sonic-net#2458)
3d757a8 [ci][asan] add DVS tests run with ASAN (sonic-net#2441)
04fbc8e [ci] Only when test stage succeeded or succeededwithissues, PR run Gcov (sonic-net#2460)
7cc035f [orchagent]: Publish identified events via structured-events channel (sonic-net#2446)
efa0f01 [QoS] Enforce drop probability only for colors whose WRED are enabled (sonic-net#2422)
05c5c2f [swss] Replace memset functions (sonic-net#2423)
9ff993d Modified the test file to remove click commands and do the REDIS-DB u… (sonic-net#2264)
9e376af Install libyang in azure pipeline. (sonic-net#2445)
c1eb99a check state_db for po before sending ARP/ND pkts (sonic-net#2444)
43cc486 [portmgr] Fixed the orchagent crash due to late arrival of notif (sonic-net#2431)
b62c716 Enhance orchagent and buffer manager in error handling (sonic-net#2414)
13bda3c [Everflow/ERSPAN] Set correct destination port and mac address when the nexthop is updated for ERSPAN mirror destination (sonic-net#2392)
0ccb315 Revert "[VS Test] Skip failing subport tests (sonic-net#2370)" (sonic-net#2421)
ac8a83f [UT] [Portsyncd] Added Unit Tests for portsyncd (sonic-net#2297)
83a186a Change the log messages in addKernelNeigh/Route from ERROR to INFO (sonic-net#2437)
9c23389 [BFD]Clean up state_db BFD entries on swss restart (sonic-net#2434)
d41aebf EntityBulker SIGSEGV when create_entry attr_count 0 (sonic-net#2224)
f52a7b1 Fix the Fec Mode Setting of gbsyncd (sonic-net#2430)
8cc0a45 [neighsyncd] Enabling ipv4 link local entries for non-dualtor (sonic-net#2427)
5624e87 Revert "[ci][asan] add DVS tests run with ASAN (sonic-net#2363)" (sonic-net#2433)
a26b26a Dynamic port configuration - add port buffer cfg to the port ref counter (sonic-net#2194)
486939a tlm_teamd: Filter portchannel subinterface events from STATE_DB LAG_TABLE (sonic-net#2408)
a4b8992 [counters] Improve performance by polling only configured ports buffer queue/pg counters (sonic-net#2360)
4aaeec9 added support for Xsight platform (sonic-net#2426)
ca9edca [ci][asan] add DVS tests run with ASAN (sonic-net#2363)
dec4570 Handle dual ToR neighbor miss scenario (sonic-net#2151)
9eb4422 Upstream new development on p4orch (sonic-net#2237)
e9be2c0 [lgtm] Fix dependency (sonic-net#2419)
c0168f3 [muxorch] Returning true if nbr in skip_neighbor_ in isNeighborActive() (sonic-net#2415)
cfcf3d8 [macsec]: Set MTU for MACsec (sonic-net#2398)
8346034 Delete Invalid if condition in intfsorch.cpp (sonic-net#2411)

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
theasianpianist added a commit that referenced this issue Oct 7, 2022
Include:

df92fb7 Improve verbosity level and provide more info in the log (#2472)
e81ed20 [intfmgr]: Enable `accept_untracked_na` kernel param (#2436)
24d29f1 [orchdaemon]: Fixed sairedis record file rotation (#2299)
b8ee07d [build] add missing package libyang-dev in lgtm.yml (#2475)
e46dd29 [crm] Fix issue with continues EXCEEDED and CLEAR logs for ACL group/table counters (#2463)
b61d24c [doc]: Update README.md (#2456)
b9ade5d [orchagent] Fix issue: ip prefix shall be inited even if VRF/VNET is not ready (#2461)
f0f1eb4 Revert "[counters] Improve performance by polling only configured ports buffer queue/pg counters (#2360)" (#2458)
3d757a8 [ci][asan] add DVS tests run with ASAN (#2441)
04fbc8e [ci] Only when test stage succeeded or succeededwithissues, PR run Gcov (#2460)
7cc035f [orchagent]: Publish identified events via structured-events channel (#2446)
efa0f01 [QoS] Enforce drop probability only for colors whose WRED are enabled (#2422)
05c5c2f [swss] Replace memset functions (#2423)
9ff993d Modified the test file to remove click commands and do the REDIS-DB u… (#2264)
9e376af Install libyang in azure pipeline. (#2445)
c1eb99a check state_db for po before sending ARP/ND pkts (#2444)
43cc486 [portmgr] Fixed the orchagent crash due to late arrival of notif (#2431)
b62c716 Enhance orchagent and buffer manager in error handling (#2414)
13bda3c [Everflow/ERSPAN] Set correct destination port and mac address when the nexthop is updated for ERSPAN mirror destination (#2392)
0ccb315 Revert "[VS Test] Skip failing subport tests (#2370)" (#2421)
ac8a83f [UT] [Portsyncd] Added Unit Tests for portsyncd (#2297)
83a186a Change the log messages in addKernelNeigh/Route from ERROR to INFO (#2437)
9c23389 [BFD]Clean up state_db BFD entries on swss restart (#2434)
d41aebf EntityBulker SIGSEGV when create_entry attr_count 0 (#2224)
f52a7b1 Fix the Fec Mode Setting of gbsyncd (#2430)
8cc0a45 [neighsyncd] Enabling ipv4 link local entries for non-dualtor (#2427)
5624e87 Revert "[ci][asan] add DVS tests run with ASAN (#2363)" (#2433)
a26b26a Dynamic port configuration - add port buffer cfg to the port ref counter (#2194)
486939a tlm_teamd: Filter portchannel subinterface events from STATE_DB LAG_TABLE (#2408)
a4b8992 [counters] Improve performance by polling only configured ports buffer queue/pg counters (#2360)
4aaeec9 added support for Xsight platform (#2426)
ca9edca [ci][asan] add DVS tests run with ASAN (#2363)
dec4570 Handle dual ToR neighbor miss scenario (#2151)
9eb4422 Upstream new development on p4orch (#2237)
e9be2c0 [lgtm] Fix dependency (#2419)
c0168f3 [muxorch] Returning true if nbr in skip_neighbor_ in isNeighborActive() (#2415)
cfcf3d8 [macsec]: Set MTU for MACsec (#2398)
8346034 Delete Invalid if condition in intfsorch.cpp (#2411)

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
dprital added a commit to dprital/sonic-buildimage that referenced this issue Oct 28, 2022
Update sonic-utilities submodule pointer to include the following:
* 4a3d49d Fix exception in adding mirror_session when gre_type is absent ([sonic-net#2458](sonic-net/sonic-utilities#2458))
* 7e7d05c Update the DBmigrator to support persistent loglevel during warm-upgrade ([sonic-net#2370](sonic-net/sonic-utilities#2370))
* c2841b8 [doc]: Update Command-Reference.md ([sonic-net#2444](sonic-net/sonic-utilities#2444))
* 254cafc Event Counters CLI ([sonic-net#2449](sonic-net/sonic-utilities#2449))
* 2dab0d0 [techsupport] Adding FRR EVPN dumps ([sonic-net#2442](sonic-net/sonic-utilities#2442))
* 3c0aece [show][muxcable] add support for show mux firmware version all ([sonic-net#2441](sonic-net/sonic-utilities#2441))

Signed-off-by: dprital <drorp@nvidia.com>
liat-grozovik pushed a commit that referenced this issue Oct 31, 2022
Update sonic-utilities submodule pointer to include the following:
4a3d49d Fix exception in adding mirror_session when gre_type is absent (#2458)
7e7d05c Update the DBmigrator to support persistent loglevel during warm-upgrade (#2370)
c2841b8 [doc]: Update Command-Reference.md (#2444)
254cafc Event Counters CLI (#2449)
2dab0d0 [techsupport] Adding FRR EVPN dumps (#2442)
3c0aece [show][muxcable] add support for show mux firmware version all (#2441)

Signed-off-by: dprital <drorp@nvidia.com>
tshalvi pushed a commit to tshalvi/sonic-buildimage that referenced this issue Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants