[sonic-device-data]: Update BRCM Tunnel/ECMP Parameter For 7050cx3 48x50G+8x100G #6415
Conversation
…x50G+8x100G Update Tunnel and ECMP paramaters for brcm 7050cx3 48x50G+8x100G SKU. signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
| @@ -1,3 +1,13 @@ | |||
| host_as_route_disable=1 | |||
| sai_trap_group_priority=1000 | |||
There was a problem hiding this comment.
looks like based on latest conversation with BCM, this attribute is not required to be set. @gechiang ?
There was a problem hiding this comment.
Yes. that is also my understanding based on yesterday's meeting with Jingping Xie of BRCM.
He mentioned that on their next 4.3 deliverables is where we need NOT set this any more and the way we configure the ACL using DROP action will achieve the outcome we need (Drop FWD path packets but allow Control packets still trapped to CPU). There is a way to drop both control plane and forwarding place pacekts if we choose to use DENY action to drop the packet when programming the ACL.
|
@tahmed-dev , could you also update this file as well, else some duts may have issue |
| @@ -1,3 +1,13 @@ | |||
| host_as_route_disable=1 | |||
There was a problem hiding this comment.
For all TD3 config.bcm files will also require a line similar to the one here:
sai_load_hw_config=/etc/bcm56870_a0/b870.6.6.1/
I will disclose the specific path once we have validate this change.
| @@ -1,3 +1,13 @@ | |||
| host_as_route_disable=1 | |||
There was a problem hiding this comment.
For all TD3 config.bcm files will also require a line similar to the one here:
sai_load_hw_config=/etc/bcm/flex/bcm56870_a0/b870.6.6.1/
We need to sort out the upgrade case with BRCM later. For now above path is good.
There was a problem hiding this comment.
@tahmed-dev SAI4.3 PR merged today. This PR should be able to go as well.
Please make sure you have committed the additional soc property on all TD3 config.bcm file with the following:
sai_load_hw_config=/etc/bcm/flex/bcm56870_a0/b870.6.6.1/
The last "/" is important. missing it will cause syncd on TD3 to crash...
…Us (sonic-net#6415) Update Tunnel and ECMP parameters for brcm 7050cx3 48x50G+8x100G and 32x100G SKUs. signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
…Us (#6415) Update Tunnel and ECMP parameters for brcm 7050cx3 48x50G+8x100G and 32x100G SKUs. signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
Update Tunnel and ECMP paramaters for brcm 7050cx3 48x50G+8x100G SKU.
signed-off-by: Tamer Ahmed tamer.ahmed@microsoft.com
- Why I did it
To enable new IPinIP tunnel feature for dual ToR
- How I did it
Add new parameter for brcm 7050cx3 48x50G+8x100G SKU
- How to verify it
Verified on target and tunnel are created.
- Which release branch to backport (provide reason below if selected)
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)