Skip to content

Commit

Permalink
Revert "[aclorch] Fixed issue sonic-net#2204.Support IN_PORTS qualife…
Browse files Browse the repository at this point in the history
…r in MIRRORV6 table. (sonic-net#2668)" (sonic-net#2687)

Reverts sonic-net#2668

which cause ACL creation failure on Mellanox platform with below err msg.
2023-03-02T11:25:00.2743784Z E Mar 2 11:22:51.042274 str2-msn2700-spy-2 ERR syncd#SDK: [ACL.ERR] Failed calculating key blocks - too many keys selected.
2023-03-02T11:25:00.2744673Z E
2023-03-02T11:25:00.2745732Z E Mar 2 11:22:51.043026 str2-msn2700-spy-2 ERR swss#orchagent: :- create: create status: SAI_STATUS_FAILURE
2023-03-02T11:25:00.2746583Z E
2023-03-02T11:25:00.2747724Z E Mar 2 11:22:51.043196 str2-msn2700-spy-2 ERR swss#orchagent: :- addAclTable: Failed to create ACL table EVERFLOWV6
2023-03-02T11:25:00.2748625Z E
2023-03-02T11:25:00.2749932Z E Mar 2 11:22:51.043384 str2-msn2700-spy-2 ERR syncd#SDK: [SAI_ACL.ERR] mlnx_sai_acl.c[13937]- mlnx_create_acl_table: Failed to create flex key - Internal Error.
  • Loading branch information
StormLiangMS authored Mar 4, 2023
1 parent ebe8de7 commit 309df59
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
3 changes: 1 addition & 2 deletions orchagent/aclorch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3217,7 +3217,7 @@ void AclOrch::initDefaultTableTypes()
* | -----------------------------------------------------------------|
* | MATCH_ETHERTYPE | √ | √ | |
* |------------------------------------------------------------------|
* | MATCH_IN_PORTS | √ | √ | |
* | MATCH_IN_PORTS | √ | √ | |
* |------------------------------------------------------------------|
*/

Expand Down Expand Up @@ -3278,7 +3278,6 @@ void AclOrch::initDefaultTableTypes()
.withMatch(make_shared<AclTableMatch>(SAI_ACL_TABLE_ATTR_FIELD_L4_SRC_PORT))
.withMatch(make_shared<AclTableMatch>(SAI_ACL_TABLE_ATTR_FIELD_L4_DST_PORT))
.withMatch(make_shared<AclTableMatch>(SAI_ACL_TABLE_ATTR_FIELD_TCP_FLAGS))
.withMatch(make_shared<AclTableMatch>(SAI_ACL_TABLE_ATTR_FIELD_IN_PORTS))
.withMatch(make_shared<AclTableMatch>(SAI_ACL_TABLE_ATTR_FIELD_DSCP))
.withMatch(make_shared<AclTableRangeMatch>(set<sai_acl_range_type_t>{
{SAI_ACL_RANGE_TYPE_L4_SRC_PORT_RANGE, SAI_ACL_RANGE_TYPE_L4_DST_PORT_RANGE}}))
Expand Down
1 change: 0 additions & 1 deletion tests/test_mirror_ipv6_separate.py
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,6 @@ def test_MirrorV6TableCreation(self, dvs, testlog):
"SAI_ACL_TABLE_ATTR_FIELD_L4_DST_PORT",
"SAI_ACL_TABLE_ATTR_FIELD_TCP_FLAGS",
"SAI_ACL_TABLE_ATTR_FIELD_DSCP",
"SAI_ACL_TABLE_ATTR_FIELD_IN_PORTS",
"SAI_ACL_TABLE_ATTR_FIELD_OUTER_VLAN_ID"
]

Expand Down

0 comments on commit 309df59

Please sign in to comment.