[syncd][bcm] Enable SAI bulk mode in config_syncd_bcm#1850
[syncd][bcm] Enable SAI bulk mode in config_syncd_bcm#1850vpandian-nokia wants to merge 1 commit intosonic-net:masterfrom
Conversation
Enable syncd bulk mode for Broadcom by appending `-l` in `config_syncd_bcm()`, matching the existing ASIC-specific pattern used by other platforms and keeping the change localized to the BCM syncd init path. Signed-off-by: Vijay Pandian <vijayaragavan.pandian@nokia.com>
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Additional DUT performance data for this change: The measured gain with this change is in the range of about 8% to 10% on a DUT with AMD EPYC 3251 8-Core 2.5GHz and 16 GB DRAM, using route-download workloads of 300K IPv4 or 300K IPv6 routes. |
|
@prsunny @prabhataravind for viz. |
|
please confirm sonic-mgmt tests/pfcwd related are passing with this change |
|
@vpandian-nokia could you provide more data on sonic-mgmt Nightly tests with this change? And also the CI is failing. please help check. |
Thanks for the note. We are working on the We are also checking the current CI failure in parallel. The |
|
/azp run |
|
Commenter does not have sufficient privileges for PR 1850 in repo sonic-net/sonic-sairedis |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
@deepak-singhal0408 : The rerun shows the current failure is in I also checked recent |
Summary
syncd -linconfig_syncd_bcm()so Broadcom platforms use SAI bulk mode by default.SAI PERFMONusage.Test plan
syncd/scripts/syncd_init_common.sh.syncdcommand line includes-l.syncdandswsscontainers remain stable after boot.docker logs syncdanddocker logs swss.