Skip to content

[aclorch]: Add ICMP type/code match for v4/v6#868

Merged
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:icmp
May 8, 2019
Merged

[aclorch]: Add ICMP type/code match for v4/v6#868
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:icmp

Conversation

@stcheng
Copy link
Copy Markdown
Contributor

@stcheng stcheng commented Apr 30, 2019

Support the following matches:
SAI_ACL_TABLE_ATTR_FIELD_ICMP_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMP_CODE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_CODE

Signed-off-by: Shu0T1an ChenG [email protected]

Support the following matches:
SAI_ACL_TABLE_ATTR_FIELD_ICMP_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMP_CODE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_CODE

Signed-off-by: Shu0T1an ChenG <[email protected]>
@stcheng stcheng merged commit f889f80 into sonic-net:master May 8, 2019
@stcheng stcheng deleted the icmp branch May 8, 2019 16:14
stcheng pushed a commit to stcheng/sonic-swss that referenced this pull request Jul 1, 2019
Support the following matches:
SAI_ACL_TABLE_ATTR_FIELD_ICMP_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMP_CODE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_CODE

Signed-off-by: Shu0T1an ChenG <[email protected]>
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
When watchdog thread will exceed 30sec it will print currently executing event like create/remove/get/set. This data can be also found in sairedis.rec, but this PR will also print this in syslog for convenience.
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
Support the following matches:
SAI_ACL_TABLE_ATTR_FIELD_ICMP_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMP_CODE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_TYPE
SAI_ACL_TABLE_ATTR_FIELD_ICMPV6_CODE

Signed-off-by: Shu0T1an ChenG <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants