Skip to content

Commit e800902

Browse files
authored
Add link requirement per sku before enabling traffic on a fap (#21865)
With the current fap initialization sequence, multiple faps in a modular system could come up with stand alone mode. This is not a recommended configuration. It might mess up the global clock and might ends up with traffic issue. #21866 This change adds a required minimum number of links have to be ready before enabling traffic on a fap, which based on the guidance should be able to guarantee the faps in a modular system can form a proper topology.
1 parent d4ce7ce commit e800902

File tree

8 files changed

+16
-0
lines changed

8 files changed

+16
-0
lines changed

device/arista/x86_64-arista_7800r3_48cq2_lc/Arista-7800R3-48CQ2-C48/jr2-a7280cr3-32d4-40x100G.config.bcm

+2
Original file line numberDiff line numberDiff line change
@@ -863,3 +863,5 @@ dma_desc_aggregator_enable_specific_MDB_FEC.BCM8869X=1
863863
sai_pfc_dlr_init_capability=0
864864
sai_default_cpu_tx_tc=7
865865
sai_disable_srcmacqedstmac_ctrl=1
866+
appl_param_active_links_thr_high=58
867+
appl_param_active_links_thr_low=1

device/arista/x86_64-arista_7800r3a_36d2_lc/Arista-7800R3A-36D2-C36/0/j2p-a7800r3a-36d-36x400G.config.bcm

+2
Original file line numberDiff line numberDiff line change
@@ -1017,3 +1017,5 @@ xflow_macsec_secure_chan_to_num_secure_assoc_decrypt=2
10171017
sai_pfc_dlr_init_capability=0
10181018
sai_default_cpu_tx_tc=7
10191019
sai_disable_srcmacqedstmac_ctrl=1
1020+
appl_param_active_links_thr_high=91
1021+
appl_param_active_links_thr_low=1

device/arista/x86_64-arista_7800r3a_36d2_lc/Arista-7800R3A-36D2-C36/1/j2p-a7800r3a-36d-36x400G.config.bcm

+2
Original file line numberDiff line numberDiff line change
@@ -1017,3 +1017,5 @@ xflow_macsec_secure_chan_to_num_secure_assoc_decrypt=2
10171017
sai_pfc_dlr_init_capability=0
10181018
sai_default_cpu_tx_tc=7
10191019
sai_disable_srcmacqedstmac_ctrl=1
1020+
appl_param_active_links_thr_high=91
1021+
appl_param_active_links_thr_low=1

device/arista/x86_64-arista_7800r3a_36d2_lc/Arista-7800R3A-36D2-C72/0/j2p-a7800r3a-36d-36x400G.config.bcm

+2
Original file line numberDiff line numberDiff line change
@@ -1034,3 +1034,5 @@ xflow_macsec_secure_chan_to_num_secure_assoc_decrypt=4
10341034
sai_pfc_dlr_init_capability=0
10351035
sai_default_cpu_tx_tc=7
10361036
sai_disable_srcmacqedstmac_ctrl=1
1037+
appl_param_active_links_thr_high=91
1038+
appl_param_active_links_thr_low=1

device/arista/x86_64-arista_7800r3a_36d2_lc/Arista-7800R3A-36D2-C72/1/j2p-a7800r3a-36d-36x400G.config.bcm

+2
Original file line numberDiff line numberDiff line change
@@ -1034,3 +1034,5 @@ xflow_macsec_secure_chan_to_num_secure_assoc_decrypt=4
10341034
sai_pfc_dlr_init_capability=0
10351035
sai_default_cpu_tx_tc=7
10361036
sai_disable_srcmacqedstmac_ctrl=1
1037+
appl_param_active_links_thr_high=91
1038+
appl_param_active_links_thr_low=1

device/arista/x86_64-arista_7800r3a_36d2_lc/Arista-7800R3A-36D2-D36/0/j2p-a7800r3a-36d-36x400G.config.bcm

+2
Original file line numberDiff line numberDiff line change
@@ -1054,3 +1054,5 @@ xflow_macsec_secure_chan_to_num_secure_assoc_decrypt=4
10541054
sai_pfc_dlr_init_capability=0
10551055
sai_default_cpu_tx_tc=7
10561056
sai_disable_srcmacqedstmac_ctrl=1
1057+
appl_param_active_links_thr_high=91
1058+
appl_param_active_links_thr_low=1

device/arista/x86_64-arista_7800r3a_36d2_lc/Arista-7800R3A-36D2-D36/1/j2p-a7800r3a-36d-36x400G.config.bcm

+2
Original file line numberDiff line numberDiff line change
@@ -1054,3 +1054,5 @@ xflow_macsec_secure_chan_to_num_secure_assoc_decrypt=4
10541054
sai_pfc_dlr_init_capability=0
10551055
sai_default_cpu_tx_tc=7
10561056
sai_disable_srcmacqedstmac_ctrl=1
1057+
appl_param_active_links_thr_high=91
1058+
appl_param_active_links_thr_low=1

src/sonic-device-data/tests/permitted_list

+2
Original file line numberDiff line numberDiff line change
@@ -353,3 +353,5 @@ flowtracker_ipfix_observation_domain_id
353353
flowtracker_num_unique_user_entry_keys
354354
sai_mmu_tc_to_pg_config
355355
sai_hostif_netif_iff_up_set
356+
appl_param_active_links_thr_high
357+
appl_param_active_links_thr_low

0 commit comments

Comments
 (0)