[device]: Add SAI checksum verify to TD3 config#8886
Merged
theasianpianist merged 1 commit intosonic-net:202012from Oct 4, 2021
Merged
[device]: Add SAI checksum verify to TD3 config#8886theasianpianist merged 1 commit intosonic-net:202012from
theasianpianist merged 1 commit intosonic-net:202012from
Conversation
Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
Contributor
|
Why to 202012? Not required in master? |
Collaborator
@prsunny There is a separate PR for master branch: The reason for this to be separate PR is because the BRCM SAI support is only available in 4.3.5.1 for now and not expect on REL 5.0 for some time. Since this fix is needed in 20201231, I asked Lawrence to raise a separate PR so that it can be moved into 20201231 independently and sooner... hope this helps clarify it... |
renukamanavalan
pushed a commit
to renukamanavalan/sonic-buildimage
that referenced
this pull request
Oct 11, 2021
Why I did it Update FRR 7.2.1 head. The following is a list of new commits. 5ae667a1f Merge pull request sonic-net#9335 from FRRouting/mergify/bp/stable/7.2/pr-9214 eb679e8a1 zebra: bugfix of error quit of zebra, due to no nexthop ACTIVE 80d2eaa98 Merge pull request sonic-net#8886 from FRRouting/mergify/bp/stable/7.2/pr-8876 1eeab2c1e lib: remove pure attribute from functions that modify memory eb00dc4ec Merge pull request sonic-net#6944 from LabNConsulting/working/lb/7.2/valgrind-supp-libyang b9d6d05bf bgpd: suppress new libyang_1.0 related loss reports 8c26a71eb Merge pull request sonic-net#6562 from ton31337/fix/configuration_for_labeled_unicast_in_place_7.2 386a1719c bgpd: Make sure network/aggregate-address commands lay down under labeled safi b01c8bf28 Merge pull request sonic-net#6526 from ton31337/fix/set_ipv6_ll_if_global_zero_7.2 c382833e8 bgpd: Use IPv6 LL address as nexthop if global was set to ::/LL 99509b835 Merge pull request sonic-net#6395 from opensourcerouting/7.2/init-config-perms 7eef8f7b1 build: use configfile mode in init script 4cbe07705 Merge pull request sonic-net#6360 from opensourcerouting/7.2/fix-warnings 84bb11785 nhrpd: clean up SA warning aac726476 nhrpd: be more careful with linked lists 3a4b6d654 debian: Fix spelling error 756c67c6c Merge pull request sonic-net#6284 from opensourcerouting/7.2/gcc-10 65a116a64 Merge pull request sonic-net#6354 from ton31337/fix/communities_bgpd_crash_7.2 f7a00fd67 bgpd: Check to ensure community attributes exist before freeing them a960f99c2 vrrpd: fix build on Fedora Rawhide d4caff99f babeld: GCC complaining about no return in non-void function a014c27ae babeld: fix build on Fedora Rawhide 79ff55b5b bgpd: remove unused variable ff343e588 pimd: Make frr able to be built by gcc 10 9a3cf1ba2 ldpd: remove multiple definitions of thread_master a19515bfe ldpd: fix another linking issue with GCC-10 b4c8de38c tests: fix build with GCC 10 4f27e8c85 ldpd: Fix linking error on Fedora Rawhide with GCC 10 How I did it Update FRR 7.2 pointer and create a tag frr-7.2.1-s4.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Lawrence Lee lawlee@microsoft.com
Why I did it
A new config option was added to control the value of IPV4_INCR_CHECKSUM_ORIGINAL_VALUE_VERIFY in the EGR_FLEX_CONFIG control register (this prevents checksums of 0xffff from being propagated to other devices)
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)